Discussion :

[jimmo]

Bonsoir à tous,
Je viens de trouver un script qui répond parfaitement à mes attentes mais qui est en mysqli et mon site en PDO.
Et je vous avoue nager completement la.
Si une âme charitable passe par là

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
<?php
$link = mysqli_connect("localhost","root","","addrow") or die("Error " . mysqli_error($link));
 
// store in the DB 
if(!empty($_POST['ok'])) {	
	// first delete the records marked for deletion. Why? Because we don't want to process them in the code below
	if( !empty($_POST['delete_ids']) and is_array($_POST['delete_ids'])) {
		// you can optimize below into a single query, but let's keep it simple and clear for now:
		foreach($_POST['delete_ids'] as $id) {
			$sql = "DELETE FROM products WHERE id=$id";
			$link->query($sql);
		}
	}
 
	// now, to edit the existing data, we have to select all the records in a variable.
	$sql="SELECT * FROM products ORDER BY id";
	$result = $link->query($sql);
 
	// now edit them
	while($product = mysqli_fetch_array($result)) {
		// remember how we constructed the field names above? This was with the idea to access the values easy now
		$sql = "UPDATE products SET qty='".$_POST['qty'.$product['id']]."', name='".$_POST['name'.$product['id']]."'
		WHERE id='$product[id]'";		
		$link->query($sql);
	}
	// (feel free to optimize this so query is executed only when a product is actually changed)
 
	// adding new products
	if(!empty($_POST['qty'])) {
		foreach($_POST['qty'] as $cnt => $qty) {
			$sql = "INSERT INTO products (qty, name) VALUES ('$qty', '".$_POST['name'][$cnt]."');";
			$link->query($sql);
		}
	}	
}
 
// select existing products here
$sql="SELECT * FROM products ORDER BY id";
$result = $link->query($sql);
 
 
while($product = mysqli_fetch_array($result)): ?>
		<p id="oldRow<?=$product['id']?>">
		Item quantity: <input type="text" name="qty<?=$product['id']?>" size="4" value="<?=$product['qty']?>" /> 
		Item name: <input type="text" name="name<?=$product['id']?>" value="<?=$product['name']?>" /> 
		<input type="checkbox" name="delete_ids[]" value="<?=$product['id']?>"> Mark to delete<
	<?php endwhile;?>

[jimmo]

J'avance à petits pas, mais je tombe sur 2 erreurs que je ne comprend pas

Warning: Invalid argument supplied for foreach() in.......

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
foreach($_POST['id'] as $id) {
    $req = $bdd->prepare("UPDATE products SET qty=:qty, name=:name WHERE id=".$id."");  
    $req->bindValue(':id', $id);
    $req->execute();
}
 
 
 
 
if(!empty($_POST['qty'])) {
        foreach($_POST['qty'] as $cnt => $qty) {
            $req = $bdd->prepare("INSERT INTO products (qty, name) VALUES (".$qty.", '".$_POST['name'][$cnt]."');");
            $req->execute();
            $result = $req->fetch(PDO::FETCH_OBJ);
        }
    }

[van-bom]

fais un var_dump

Code :

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

1
2
 
var_dump($_POST['id']) ;

ça donne quoi ?

[jimmo]

Le var_dump m'affiche : string '52' (length=2)

[van-bom]

voilà donc c'est une chaine de caractère pas un tableau .

foreach sert à parcourir un tableau ou un objet ( les attributs ).

doc officielle :

Code :

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

1
2
3
4
 
La structure de langage foreach fournit une façon simple de parcourir des tableaux.
 foreach ne fonctionne que pour les tableaux et les objets, 
et émettra une erreur si vous tentez de l'utiliser sur une variable de type différent ou une variable non initialisée.

[jimmo]

heuu ok, et tu me conseilles quoi?

[van-bom]

ça déponds de ta logique , pourquoi tu veux parcourir le superglobale $_POST['id'] et n'est pas un tableau ou objet ?

[jimmo]

Pour tout te dire je suis un peu perdu la. Je suis en train d'essayer de passer en PDO le code en Mysqli ci dessus. J'ai donc repris la logique utilisée pour mon site qui est en objet.
Mais ça ne me réussi pas trop.
je te met le code modifié si c'est plus parlant.

sachant tout de même que la partie foreach=>INSERT m'affiche une erreur mais enregistre les données.
la partie foreach=>UPDATE ne fait rien d'autre que m'afficher l'erreur.

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
<?php
if(!empty($_POST['ok'])) {	
	// first delete the records marked for deletion. Why? Because we don't want to process them in the code below
	if( !empty($_POST['delete_ids']) and is_array($_POST['delete_ids'])) {
		// you can optimize below into a single query, but let's keep it simple and clear for now:
		foreach($_POST['delete_ids'] as $id) {
			$req = $bdd->prepare('DELETE FROM products WHERE id='.$id.'');
			$req->execute();
			$result = $req->fetch(PDO::FETCH_ASSOC);
		}
	}
 
	$req = $bdd->prepare('SELECT * FROM products ORDER BY id');
	$req->execute();
	$result = $req->fetch(PDO::FETCH_OBJ);
 
 
	foreach($_POST['id'] as $id) {
		$req = $bdd->prepare("UPDATE products SET qty=:qty, name=:name WHERE id=".$id."");	
		$req->bindValue(':id', $id);
		$req->execute();
	}
	var_dump($_POST['id']) ;
 
 
	if(!empty($_POST['qty'])) {
		foreach($_POST['qty'] as $cnt => $qty) {
			$req = $bdd->prepare("INSERT INTO products (qty, name) VALUES (".$qty.", '".$_POST['name'][$cnt]."');");
			$req->execute();
			$result = $req->fetch(PDO::FETCH_OBJ);
		}
	}
}
 
 
$req = $bdd->prepare('SELECT * FROM products ORDER BY id');
$req->execute();
$result = $req->fetch(PDO::FETCH_OBJ);
?>
<?php
	// let's assume you have the product data from the DB in variable called $products
	while ($result = $req->fetch(PDO::FETCH_OBJ)){ ?>
		<p id="oldRow">
		id : <input type="text" name="id" value="<?php echo $result->id;?>" /> 
		Item quantity: <input type="text" name="qty" size="4" value="<?php echo $result->qty;?>" /> 
		Item name: <input type="text" name="name" value="<?php echo $result->name;?>" /> 
		<input type="checkbox" name="delete_ids[]" value="<?php echo $result->id;?>"> Mark to delete<
	<?php }?>
	</div>
	<p><input type="submit" name="ok" value="Save Changes"></p>
	</form>

[Spartacusply]

Dans ta logique de code,$_POST['id'] devrait être un tableau, or c'est une chaîne de caractère, d'où ton erreur..

[jimmo]

Ok ok, mais concretement ca pourrait donner quoi?

[sabotage]

Dans ton code de départ il n'y a pas de $_POST['id'] ; qu'est ce que tu as changé dans ton formulaire ?

Également, on ne fait pas un "prepare" dans une boucle.

[jimmo]

Salut sabotage,
J'ai effectivement remplacé le prepare par exec.
Ensuite, dans le formulaire , je n'ai aucune idée de ce qu'est <input type="text" name="qty<?=$product['id']?>" size="4" value="<?=$product['qty']?>" /> j'en ai conclu que ça permettais de récupérer l'id, donc j'ai rajouté ceci id : <input type="hidden" name="id" value="<?php echo $result->id;?>" />
d'ou par la suite mon $_POST['id']

[sabotage]

Si ton but est de modifier ton code mysqli vers PDO, il n'y a rien a changer dans le formulaire ni dans les boucles de traitement, ni dans les variables en jeu, ni dans les requêtes : seulement les éléments mysqli par PDO.
Donc dans ton cas la fonction mysqli_fetch_array() est remplacé par le méthode PDO fetch()
et la méthode mysqli query() est remplacé par exec() ou query() selon que tu faire une lecture ou une ecriture.

[Celira]

sur Sabotage. Et si tu veux changer le fonctionnement du formulaire, il faut aussi changer le formulaire lui-même.

Au passage, l'idéal serait d'utiliser des requêtes préparées, pasque bon sinon on perd un peu de l'intérêt de PDO (mais comme le script d'origine n'utilise pas non plus les requêtes préparées avec mysqli... )

[jimmo]

Merci de votre aide.....
je vais voir tout ca. bonne soirée