Erreur : Invalid argument supplied for foreach() lors d'ajouts en BD

Version imprimable

Bonsoir à tous :D !
Voilà mon soucis est que je n'arrive pas à ajouter correctement dans ma base de donnée des informations d'un fichier csv
j'ai le message "invalid argument supplied for foreach()" pour ajouter les informations dans ma base de donnée

Voilà mon code :

Code:

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
 
public function putMeteoDataBase()
	{
		require('./config.php');
 
                //connexion base de donnée
		$mysqli = new mysqli('localhost', 'root', '','previsions_meteo');
		$mysqli->set_charset("utf8"); //encodage utf8
 
                // Vérification de la connexion
		if($mysqli->connect_errno)
		{
 
			printf("Echec de la connexion %s\n", $mysqli->connect_error);
			exit();
		}
 
		$file = fopen($file, "r");
		while (!feof($file))
		{
			// On recupere toute la ligne
			$line = addslashes(fgets($file));
			//On met dans un tableau les differentes valeurs trouvés (ici séparées par un ';')
			$array_values = explode(';', $line);
 
 
			foreach($array_values as $key => $array)
			{
 
				foreach($array as $values)
				{
 
			              // On crée la requete pour inserer les donner (ici il y a 8 champs donc de [0] a [7])
			             $result=$mysqli->query('INSERT INTO 
                                     meteo("date","ville","periode","resume","identifiant_de_resume","temperature_minimale","temperature_maximale","commentaire") VALUES 
                                     ("'.$values[0].'","'.$values[1].'", "'.$values[2].'", "'.$values[3].'", "'.$values[4].'", "'.$values[5].'", "'.$values[6].'", "'.$values[7].'"');
 
		 	       }
		      }
 
		}
		//vérification et envoi d'une réponse à l'utilisateur
		if ($result)
		{
			echo "<h2>Ajout dans la base de donnée effectué avec succès</h2>";
		}
		else
		{
			echo "Echec de l'ajout dans la base de donnée";
		}
 
	}

Merci de me donner un coup de main !:zoubi:

25/01/2021, 23h50
CosmoKnacki
Pour trouver l'erreur, relis ton code en te demandant à chaque nouvelle variable ce qu'elle contient:
Code:

1 2 //On met dans un tableau les différentes valeurs trouvées (ici séparées par un ; ) $array_values = explode(';', $line);
Jusque là, ça va à peu près.

Ensuite quel est l'intérêt de faire des foreach ? Toutes les valeurs dont tu as besoin pour construire ta requête INSERT sont déjà présentes et directement accessibles via $array_values et tu n'as aucun besoin de les faire défiler avec une boucle (et encore moins deux).

À part ça, pour lire un fichier csv, il y a fgetcsv() (bien plus pratique que d'utiliser fgets() puis explode()).

Je ne pense pas que le addslashes() soit bien utile et que la construction de ta requête par concaténation des valeurs venues de ton csv bien raisonnable du point de vue pratique et du point de vue sécurité: tu dois utiliser une requête préparée. (Je te conseille aussi d'aller sur ce site qui est certes en anglais mais qui est limpide et right to the point.)

Tu es parti pour lire le fichier csv puis pour construire la requête d'insertion et l'éxécuter avec PHP. C'est une manière de faire. Mais sache qu'il est aussi possible d'importer un fichier csv directement avec MySQL.
26/01/2021, 00h03
houd.ini

Rien à ajouter.
A part que la parenthèse fermante de ton INSERT est manquante (tu l'aurais sans doute aperçu bien vite).
26/01/2021, 16h03
XAVIERV

Merci je vais changer tout ça et je reviens vers vous si je n'y arrive toujours pas encore merci de répondre:zoubi:

remerciment

Merci de m'avoir aider du coup j'ai résolue mon problème en utilisant du coup fgetcsv voilà mon code :

Code:

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
53
54
 
 
                $mysqli = new mysqli('localhost', 'root', '','previsions_meteo');
		$mysqli->set_charset("utf8"); //encodage utf8
 
		if($mysqli->connect_errno)
		{
 
			printf("Echec de la connexion %s\n", $mysqli->connect_error);
			exit();
		}
 
		if (($handle = fopen("./csv/meteo.csv", "r")) !== FALSE) 
		{
			$result=$mysqli->query('SELECT count(*) FROM meteo');
 
   			$row = $result->fetch_array();
 
    		while (($data = fgetcsv($handle, 1000, ",")) !== FALSE) 
    		{
 
        		$num = count($data);
 
        		for ($c=0; $c < $num; $c++)
        		{
        			$values = explode(";", $data[$c]);
 
                    if($row[0] == 0)
           	 		{
           	 			$result=$mysqli->query('INSERT INTO meteo(date,ville,periode,resume,identifiant_de_resume,temperature_minimale,temperature_maximale,commentaire) VALUES ("'.$values[0].'","'.$values[1].'", "'.$values[2].'", "'.$values[3].'", "'.$values[4].'", "'.$values[5].'", "'.$values[6].'", "'.$values[7].'")');
 
 
						if(!$result)
						{
							$message = 'Echec de l\'ajout dans la base de données';
						}
						else
						{
							$message = '<h2>Ajout dans la base de données effectué avec succès</h2>';
						}
 
           	 		}
           	 		else
					{
						$message ='Les informations du fichier csv éxiste déjà dans la table';
           	 		}
 
 
 
           		}
           	}
        }
 
        fclose($handle);

Bonne soirée :zoubi:

C'est pas encore ça, tu as encore collé un explode et une boucle inutiles. fgetcsv() se charge tout seul de découper la ligne du fichier csv en champs, encore faut-il lui donner le bon délimiteur de champs (qui visiblement est ;).
Tu continues à construire ta requête en concaténant les valeurs issues de ton csv, ce qu'il ne faut pas faire pour des raisons de sécurité et d'éventuels problèmes d'échappements. Il faut à la place utiliser une requête préparée.

À partir de ton code précédent, ça donnerait quelque chose dans ce goût là:

Code:

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
53
54
55
56
57
$mysqli = new mysqli('localhost', 'root', '', 'previsions_meteo');
$mysqli->set_charset("utf8");
 
if ( $mysqli->connect_errno ) {
    printf("Echec de la connexion %s\n", $mysqli->connect_error);
    exit();
}
 
if ( false !== $handle = fopen("./csv/meteo.csv", "r") ) {
    $result = $mysqli->query('SELECT count(*) FROM meteo'); 
 
    $row = $result->fetch_array();
 
    // on écrit la requête avec des placeholders ?
    $query = 'INSERT INTO meteo (date,
                                 ville,
                                 periode,
                                 resume,
                                 identifiant_de_resume,
                                 temperature_minimale,
                                 temperature_maximale,
                                 commentaire)
              VALUES (?, ?, ?, ?, ?, ?, ?, ?)';
 
    // on prépare la requête
    $stmt = $mysqli->prepare($query);
 
    // on lie les variables à la requête préparée
    $stmt->bind_param('ssssssss', $date, $ville, $periode, $resume, $id_resume, $t_min, $t_max, $commentaire);
 
    while ( false !== $values = fgetcsv($handle, 1000, ";") ) {
        if ( $row[0] == 0 ) {                      //   ^-- on choisit le bon délimiteur de champs
                                                   //       pas besoin de faire un explode par la 
                                                   //       suite, ni de boucle
 
            // on met à jour les variables liées avec les valeurs des champs de la ligne du csv                                        
            list($date, $ville, $periode, $resume, $id_resume, $t_min, $t_max, $commentaire) = $values;
 
            // on exécute la requête préparée
            $result = $stmt->execute();
 
            if ( $result ) {
                $message = '<h2>Ajout dans la base de données effectué avec succès</h2>';
            } else {
                $message = 'Echec de l\'ajout dans la base de données';
            }
        } else {
            $message = 'Les informations du fichier csv éxistent déjà dans la table';
        }
 
    }
 
    // on ferme la requête
    $stmt->close();
}
 
fclose($handle);

Je ne vois pas trop où tu veux en venir avec ta requête SELECT count(*)... ni pourquoi tu testes son résultat if ( $row[0] == 0 ) à cet endroit du code, c'est pourquoi je l'ai laissé tel quel.