Discussion :

[nkordiko]

Bonjour,

A partir d'un fichier csv je souhaite mettre à jour le champ d'une de mes tables.
Je récupère bien les infos souhaités dans mon csv.
D'après les sorties d'ecrans, tout se passe bien au niveau de ma requête mais il y a aucune modification effectuée dans ma base de données.

Quand je copie-colle directement la requête générée depuis mon fichier dans ma base de données les modifications se font bien.

Voici ma requête en question

Code :

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

1
2
3
UPDATE ps_attribute_lang al, ps_product_attribute pa, ps_product_attribute_combination pac 
SET al.redoute='0WRXDL62' WHERE al.name='50 ' 
and pa.id_product='2' and pa.id_product_attribute=pac.id_product_attribute and pac.id_attribute=al.id_attribute

Le problème pourrait venir de la jointure des tables dans le Where?
Je n'ai vraiment aucune idée.

Merci d'avance

[nkordiko]

Ci dessous mon php

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
 
<?php
//Le chemin d'acces a ton fichier sur le serveur
$fichier = fopen("test2.csv", "r");
 
//tant qu'on est pas a la fin du fichier :
while (!feof($fichier))
{
// On recupere toute la ligne
$uneLigne = fgets($fichier, 1024);
//On met dans un tableau les differentes valeurs trouvés (ici séparées par un ';')
$tableauValeurs = explode(';', $uneLigne);
 
if(is_array($tableauValeurs) && isset($tableauValeurs[2])) {
$sql = "UPDATE ps_attribute_lang al, ps_product_attribute pa, 
ps_product_attribute_combination   pac 
SET al.redoute='".$tableauValeurs[2]."' 
WHERE al.name='".$tableauValeurs[5]."' 
and pa.id_product='".$tableauValeurs[3]."' 
and pa.id_product_attribute=pac.id_product_attribute 
and pac.id_attribute=al.id_attribute";
 
echo $sql."<br />";
 
$req=mysql_query($sql)or die (mysql_error());
 
 
// la ligne est finie donc on passe a la ligne suivante (boucle)
	}
}
//vérification et envoi d'une réponse à l'utilisateur
if ($req)
{
 
echo"Ajout dans la base de données effectué avec succès <br />";
echo mysql_errno($liendb) . ": " . mysql_error($liendb) . "<br />";
echo "Nombre de Lignes Affectées: " .mysql_affected_rows()."<br />";
}
else
{
echo"Echec dans l'ajout dans la base de données <br />";
}
 
echo "traitement terminé";
 
?>

Merci d'avance

[ericd69]

salut,

un truc simple remplace ton update par un "select count(*)" avec le même where et regarde combien de lignes sont concernées

[nkordiko]

Salut Ericd69,

J'ai 3 lignes concernées quand je fais la requête dans mysql mais que je l'appelle depuis mon script

Il me sort -1

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
 
if(is_array($tableauValeurs) && isset($tableauValeurs[2])) {
 
$sql = "SELECT COUNT(*) FROM ps_attribute_lang al, ps_product_attribute pa, ps_product_attribute_combination pac WHERE pa.id_product_attribute=pac.id_product_attribute 
and pac.id_attribute=al.id_attribute 
and al.name='".$tableauValeurs[5]."' 
and pa.id_product='".$tableauValeurs[3]."'";
 
echo $sql."<br />";
echo "Nombre de Lignes Affectées: " .mysql_affected_rows()."<br />";
$req=mysql_query($sql)or die (mysql_error());
 
 
 
	}

Y'a bien un problème depuis mon script que je n'arrive pas à comprendre.

[ericd69]

déjà pour lire un csv c'est fgetcsv
qui te sort le tableau correspondant à une ligne de ton csv...

ça va simplifier ton code... essaye déjà avec ça sans faire la requête

[nkordiko]

Ericd69, j'ai jugé inutile de passer par fgetcsv parce que je récupère parfaitement les données souhaitées.

Le soucis que j'avais c'est que j'avais un blanc après mes valeurs. J'ai corrigé ce problème.

Un problème en amenant un autre maintenant mon update ne se fait pas dans son intégralité pourtant les requêtes executées sont correctes et fonctionne en direct dans Phpmyadmin.

En isolant les requêtes posant problème et en les mettant dans un nouveau csv ça fonctionne mais dès 40 lignes dans mon csv l'update ne se fait plus dans son intégralité.

Ci-joint mon fichier csv d'origine.

Merci d'avance pour votre compréhension

[Benjamin Delespierre]

Ericd69, j'ai jugé inutile de passer par fgetcsv parce que je récupère parfaitement les données souhaitées.

fgetcsv = fgets + explode, la portabilité en plus.

Et sinon, t'as pensé à changer le reporting d'erreur et à regarder les messages de warning de MySQL ?

[nkordiko]

Et sinon, t'as pensé à changer le reporting d'erreur et à regarder les messages de warning de MySQL ?

J'ai juste le mysql_error() qui ne m'indique aucune erreur.
Que me proposes tu d'autres?

[Benjamin Delespierre]

1. Utilise mysql_info pour savoir s'il y a eu des warnings
2. mets la directive error_reporting à -1
3. mets la directive display_errors à 1
4. mysql_query(...) or die(...) en dev si tu veux mais jamais en production
5. vérifie systématiquement l'exécution de tes requêtes
6. trouve des noms un peu plus parlants que tableauValeur, sinon tu va galérer quand tu vas devoir te relire

[nkordiko]

1. Utilise mysql_info pour savoir s'il y a eu des warnings
2. mets la directive error_reporting à -1
3. mets la directive display_errors à 1
4. mysql_query(...) or die(...) en dev si tu veux mais jamais en production
5. vérifie systématiquement l'exécution de tes requêtes
6. trouve des noms un peu plus parlants que tableauValeur, sinon tu va galérer quand tu vas devoir te relire

Bonjour Benjamin,

Toutes mes excuses pour le retard mais je travaillais sur autre chose.
Merci pour tes conseils pour tracker mes requêtes SQL.
Malheureusement, le problème reste le même.
Je n'ai aucun warning et certains Update sont validées par mon php mais je ne retrouve pas les données dans la table.

J'en perds la tête.

[ericd69]

tu es face à un problème assez commun, les updates sur jointure c'est toujours risqué...

comment un select count(*) peut te sortir -1?

écrit déja une jointure normalisée avec la syntaxe de maintenant, pas celle antérieure à 1992:

Code php :

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

1
2
3
4
5
6
7
"SELECT COUNT(*)
FROM ps_attribute_lang al
inner join ps_product_attribute_combination pac
	on pac.id_attribute=al.id_attribute
inner join ps_product_attribute pa
	on pa.id_product_attribute=pac.id_product_attribute and  pa.id_product='{$tableauValeurs[3]}'
WHERE al.name='{$tableauValeurs[5]}'"

voilà en respectant que la table principal est celle qui subira l'update...

ce qui devrait de donner en version update:

Code php :

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

1
2
3
4
5
6
7
"update ps_attribute_lang al
inner join ps_product_attribute_combination pac
	on pac.id_attribute=al.id_attribute
inner join ps_product_attribute pa
	on pa.id_product_attribute=pac.id_product_attribute and  pa.id_product='{$tableauValeurs[3]}'
SET al.redoute='{$tableauValeurs[2]}'
WHERE al.name='{$tableauValeurs[5]}'"

utiliser {} dans "" te permet de passer des tableaux directement sans concaténation... mieux non?

teste ça

si le résultat est pas bon c'est que tu fait pas les bonne conditions de jointure

rappel: inner join implique que tu aies une correspondance dans la table à droite ET à gauche...

tu serais pas en innodb sans commit de fait par hasard, vu que certains sont pris et pas d'autres...?

[nkordiko]

Bonjour Eric,

J'ai testé ton select Count(*) dans phpMyAdmin avec tes jointures normalisées et y'a pas de problème.
J'ai le bon résultat.

Par contre j'ai un soucis avec mon update phpMyAdmin me retourne une erreur que voici

#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'inner join ps_product_attribute_combination pac on pac.id_attribute=al.id_attrib' at line 1

Voici ma requête

Code :

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

1
2
3
4
5
6
7
 
UPDATE ps_attribute_lang al SET al.cdiscount='MP00010483-0001', al.ean_cdiscount='3606505375253' 
inner join ps_product_attribute_combination pac 
         on pac.id_attribute=al.id_attribute 
inner join ps_product_attribute pa 
         on pa.id_product_attribute=pac.id_product_attribute and pa.id_product='29' 
WHERE al.name='50'

Ma base de données est en myisam

[ericd69]

autan pour moi

il faut que tu mettes les jointure avant le set...

Code sql :

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

1
2
3
4
5
6
7
UPDATE ps_attribute_lang al 
inner join ps_product_attribute_combination pac 
         on pac.id_attribute=al.id_attribute 
inner join ps_product_attribute pa 
         on pa.id_product_attribute=pac.id_product_attribute and pa.id_product='29'
SET al.cdiscount='MP00010483-0001', al.ean_cdiscount='3606505375253'
WHERE al.name='50'