Discussion :

[Demirdiouff]

Bonjour à toutes et à tous.

Je me permets de faire appel à vos lumières pour tenter de m'éclaircir sur un point sur lequel je bloque.
(Je pourrais éventuellement procéder autrement en terme d'HTML, mais je souhaite résoudre le problème ne serait-ce que pour le savoir-faire...)

J'ai un formulaire assez long dont je ne pense pas lister l'intégralité ici, mais à l'intérieur de celui-ci, il y a des champs checkbox, dont j'aimerais qu'ils comportent le même "name" voici l'exemple ci-dessous :

View:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
				<label for="inputIsoType">Type de chauffage</label>
				<div class="row">
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeGaz">
							</span><label for="inputHeatTypeGaz" class="form-control">Gaz</label>
						</div>
					</div>
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeFioul">
							</span><label for="inputHeatTypeFioul" class="form-control">Fioul</label>
						</div>
					</div>
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeElectricite">
							</span><label for="inputHeatTypeElectricite" class="form-control">Electricité</label>
						</div>
					</div>
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeBois">
							</span><label for="inputHeatTypeBois" class="form-control">Bois</label>
						</div>
					</div>
				</div>

J'ai bien saisi que pour pouvoir stocker plusieurs checkbox dans un même "name" il fallait les stocker sous forme de tableau, d'où les "[]".

Cependant, le problème survient à l'étape d'après;
D'habitude, pour pouvoir récupérer ces valeurs, je procède ainsi :

Controlleur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
    public function addCustomerIso()
    {
        if (isset($_POST['addCustomerIso']))
        {
            $customerIso = new CustomerIso([
                'heat_type' => $_POST['customerIsoHeatType']
            ]);
            $this->customerIsoManager->add($customerIso);
        } else {
            echo 'Une erreur est survenue';
        }
    }

Toujours d'après les recherches Google, il faudrait effectuer un foreach pour récupérer les plusieurs checkbox si il y en a et ensuite effectuer le $_POST de celui-ci, mais c'est la que le problème intervient... je n'arrive pas à visualiser cette partie du code.
Certains parlent même d'"implode" pour rassembler les éléments du tableau en string, sur papier, ça me paraît tout juste, mais à retranscrire c'est plus compliqué.

Le modèle quant à lui devrait rester pareil ? Il n'aura aucun impact pour la requête BDD ?


Je vous remercie énormément pour votre aide.

Cdt

[andry.aime]

Bonjour,

1- Il te manque l'attribut value dans les checkbox.
2- Un id doit être unique dans un document.
3- Seul les cases cochées seront envoyées, donc ne t'attend pas à ce que les index seront identiques par rapport aux lignes de données.

A+.

[Demirdiouff]

Merci pour ta réponse,

C'est réglé pour les attributs "value".
Je n'ai pas bien compris concernant les id ? Je n'ai pas l'impression d'avoir de doublon ici ?

Cependant le problème qui se pose ici et que je ne sais pas comment résoudre dans le contexte actuel est le suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Notice: Array to string conversion

[andry.aime]

Je n'ai pas bien compris concernant les id ? Je n'ai pas l'impression d'avoir de doublon ici ?

Désolé, je croyais que c'était dans une boucle, je ne faisais qu'une lecture diagonale.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Notice: Array to string conversion

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$customerIso = new CustomerIso([
                'heat_type' => $_POST['customerIsoHeatType']
            ])

$_POST['customerIsoHeatType'] est un array

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
$customerIsoHeatTypes = $_POST['customerIsoHeatType'];
foreach ($customerIsoHeatTypes  as $value){
    $customerIso = new CustomerIso([
                'heat_type' =>  $value
            ]);
 $this->customerIsoManager->add($customerIso);
}

A+.

[Demirdiouff]

Ok, c'est exactement ce genre de retranscription que je n'arrivais pas à mettre en place...

Je vais adapter ça (j'ai de multiples champs checkbox avec le même fonctionnement en un formulaire) et le tester dès que possible, merci beaucoup

[rawsrc]

Salut,

tu sais que tu peux aussi personnaliser les clés du tableau :
name="customerIsoHeatType[25]" ou name="customerIsoHeatType['abc']", cela te permettra d'utiliser directement isset($_POST['customerIsoHeatType']['abc'])

[Demirdiouff]

J'ai tenté d'adapter pour une version qui pourrait coller avec le formulaire que j'ai, cependant, lors de l'insertion en BDD, lorsque je sélectionne 2 checkbox, j'ai 2 lignes en BDD qui se rajoutent au lieu d'une.

Faut-il apporter une modification sur le modèle lors de la requête d'insertion pour ne pas avoir ces deux lignes ? J'ai cru comprendre que de toute manière, lorsque un champ checkbox n'était pas coché, il n'était simplement pas enregistré dans la BDD.

Egalement, j'ai un second soucis dans mon formulaire qui est le fait que j'ai plusieurs champs checkbox au sein du formulaire, qui sont a choix multiples.
Dans ce contexte la, comment faut-il procéder ? Effectuer des foreach dans foreach pour chaque $_POST ?

J'ai l'impression que le soucis est relativement simple à résoudre mais je n'arrive malheureusement pas à visualiser la solution...

[andry.aime]

Bonjour,

Comment est structuré ta/tes table(s)? Il y a une colonne pour chaque valeur? Un customerIsoHeatTypes ne devrait avoir qu'un seul type (dans ce cas des radio bouton est le mieux adapté que des checkbox)? où tu as deux tables avec une relation un à plusieurs?

A+.

[Demirdiouff]

Bonjour,

J'ai qu'une seule table "customersiso" dans lequel il y a toutes les informations à fournir, et il y a une colonne pour chaque valeur.

Dans le cas de "customerIsoHeatType", par exemple, il peut y avoir plusieurs champs à sélectionner d'un coup, d'où le "checkbox" à choix multiple avec le même "name", ce champ peut également être vide parfois.
Dans d'autres cas, il peut y avoir qu'un seul bouton à cocher, donc dans ce cas effectivement, je pourrais fonctionner avec les "radioButton", et la le problème est plutôt simple je pense.

En gros mon formulaire est structuré de cette manière :

type text
type text
...
type checkbox[] (choix multiples ou champ vide)
type checkbox[] (choix multiples ou champ vide)
type radio (choix simple ou champ vide)
type checkbox[] (choix multiples ou champ vide)
...
type text
type text
...
submit

[andry.aime]

J'ai qu'une seule table "customersiso" dans lequel il y a toutes les informations à fournir, et il y a une colonne pour chaque valeur.

Alors pourquoi tu avais ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
new CustomerIso([
                'heat_type' => $_POST['customerIsoHeatType']
            ])

Ça donne l'impression qu'il n'y a qu'une seule colonne pour le customerIsoHeatType.
On peut voir la structure de la table concernée?

A+.

[Demirdiouff]

Ce bout de code, c'était le fonctionnement que j'utilise d'habitude lorsqu'il n'y a qu'une seule entrée à faire, sous un "input type text" par exemple.

Voici la structure de ma table actuelle :

Code SQL :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
CREATE TABLE `customersiso` (
  `customeriso_id` int(11) NOT NULL,
  `vendor` varchar(50) DEFAULT NULL,
  `lastname` varchar(50) DEFAULT NULL,
  `firstname` varchar(50) DEFAULT NULL,
  `address` varchar(100) DEFAULT NULL,
  `locality` text DEFAULT NULL,
  `city` varchar(70) DEFAULT NULL,
  `postal_code` char(5) DEFAULT NULL,
  `phone_number` char(10) DEFAULT NULL,
  `email` char(50) DEFAULT NULL,
  `situation` varchar(30) DEFAULT NULL,
  `area_attic` char(4) DEFAULT NULL,
  `area_wall` char(4) DEFAULT NULL,
  `area_flooring` char(4) DEFAULT NULL,
  `iso_type` varchar(15) DEFAULT NULL,
  `house_age` varchar(15) DEFAULT NULL,
  `attic_status` varchar(30) DEFAULT NULL,
  `window` varchar(30) DEFAULT NULL,
  `attic_access` varchar(30) DEFAULT NULL,
  `heat_type` varchar(20) DEFAULT NULL,
  `financing` varchar(30) DEFAULT NULL,
  `population` char(2) DEFAULT NULL,
  `documents` varchar(200) DEFAULT NULL,
  `date` date DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

[andry.aime]

Si j'ai bien compris, la valeur sélectionnée devrait être dans la colonne `heat_type`, dans ce cas pourquoi ne pas utilisé des radio bouton au lieu des checkbox puisque tu ne peux avoir qu'une seule valeur?

A+.

[Demirdiouff]

J'aurais des deux, il y aura des champs tel que "window", "financing" qui vont être de type radio, car qu'un seul choix possible: à cocher ou non. Donc le bouton radio pourrait convenir dans ces champs la.

Cependant, pour les champs tel que "attic_status", "heat_type", "attic_access", il y aura plusieurs choix possibles, d'où ma nécessité d'utiliser les "checkbox" et de pouvoir stocker plusieurs valeurs de "checkbox" en un.. comme le cas d'un morceau du formulaire ci-dessous :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
		<label for="inputIsoType">Type de chauffage</label>
				<div class="row">
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeGaz">
							</span><label for="inputHeatTypeGaz" class="form-control">Gaz</label>
						</div>
					</div>
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeFioul">
							</span><label for="inputHeatTypeFioul" class="form-control">Fioul</label>
						</div>
					</div>
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeElectricite">
							</span><label for="inputHeatTypeElectricite" class="form-control">Electricité</label>
						</div>
					</div>
					<div class="col-lg-6">
						<div class="input-group">
							<span class="input-group-addon"> <input type="checkbox"
								aria-label="HeatType" name="customerIsoHeatType[]" id="inputHeatTypeBois">
							</span><label for="inputHeatTypeBois" class="form-control">Bois</label>
						</div>
					</div>
				</div>

J'ai l'impression que c'est difficile, voir impossible de stocker plusieurs valeurs d'un checkbox par exemple dans un seul champ en BDD, à moins d'effectuer quelques bidouillages avant ?
Beaucoup parlent de créer une autre table, et de les relier entre eux, mais dans ce cas la, je ne sais pas du tout comment procéder...

Merci

[darkstar123456]

Effectivement, on créé souvent une table à part pour ce genre de choses.
C'est souvent plus propre et plus maniable.
Par exemple, si demain tu ajoutes une nouvelle valeur possible, tu n'as -théoriquement- rien à changer.

Pour bien faire, je ferais 3 tables :
- ta table customers que tu as déjà
- une table qui contient toutes les valeurs possibles
- une table qui lie les valeurs aux customers

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
list_heat_types
===============
heat_type_id (INT)
heat_type_value (VARCHAR)
 
 
customeriso
===========
customeriso_id (INT)
etc...
 
 
customeriso_heat_types
======================
heat_type_id (INT)
customeriso_id (INT)

[Demirdiouff]

Au tout début de mon projet j'étais parti dans cette optique la, avec 5 tables différentes pour tout ce que je devais stocker en BDD.
Cependant, j'ai vite changé de voie car je me suis rendu compte que lorsque j'utilisais plusieurs tables, il était compliqué avec une seule entité d’insérer dans plusieurs tables en même temps lors d'un seul "submit" formulaire (car je dispose d'un seul et unique formulaire assez long qui gère tout).

Comment est-ce que je peux procéder dans ce cas la ?

Je dois avouer que c'est la première fois que je me lance dans un gros chantier comme ça, et que j'essaye de le faire proprement, (en MVC, en POO...) pour essayer de voir tous les aspects...

[andry.aime]

Effectivement, on créé souvent une table à part pour ce genre de choses.
C'est souvent plus propre et plus maniable.

Oui et non, ici par exemple il n'y a que 4 types possible et je pense qu'il n'y aura pas d'autre, un ENUM semble plus approprié.

A+.

[Demirdiouff]

Bonjour,

Oui et non, ici par exemple il n'y a que 4 types possible et je pense qu'il n'y aura pas d'autre, un ENUM semble plus approprié.

Pourrais-tu me donner un peu plus d'informations sur le ENUM ?

[andry.aime]

Bonjour

Cependant, pour les champs tel que "attic_status", "heat_type", "attic_access", il y aura plusieurs choix possibles, d'où ma nécessité d'utiliser les "checkbox" et de pouvoir stocker plusieurs valeurs de "checkbox" en un.. comme le cas d'un morceau du formulaire ci-dessous :

Si tu peux avoir plusieurs heat_type pour un customersiso, tu dois procéder comme ce que l'avais suggérer darkstar123456, tu fais d'abord l'insertion dans la table customersiso, ensuite tu insères son id dans la table de liaison dans une boucle.

Pourrais-tu me donner un peu plus d'informations sur le ENUM ?

Si tu ne peux qu'avoir un seul heat_type pour un customersiso, tu mets la colonne heat_type en ENUM et remplace les checkbox par des boutons radio.

A+.