REquete SQL + PHP [ Impossible de sortir un second echo en cas de valeur vide ]

**david77185** · 03/11/2022, 22h37

Bonjour a tous

j'aurai besoin d'aide pour ma condition svp

J'ai creé une petite bdd sous phpmyadmin pour pouvoir gerer des numeros de serie

La partie enregistrement , c'est bon. Mes numeros s'inserent en base avec requete SQL et formulaire php

Dorenavant, je souhaite tester si un numero de serie est present en base ou pas

1°) Lorsque le numero de serie est present en base , ma page php affiche bien mon numero de serie, attaché avec sa date préalablement enregistré (ok pour cette partie)

mais....

2°) Lorsque le numero de serie n'est pas present en base, je n'ai pas d'erreur, , mais je n'arrive pas à sortir mon echo php " numero non present en base "

la structure de mon php est correcte, je l'ai testé ici
https://www.phptools.online/php-checker

De plus, etant sous hostinger, avec leur editeur de fichier, nous pouvons voir si le php est tronqué ou pas. Ce n'est pas le cas. donc je deviens dingue

je vous mets mon code

J'ai testé avec isset et empty
et je vous avoue que les conditions IF et IF ELSE me donnent le tourni

Code :

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

code deleted pour une meilleur comprehension

**Toufik83** · 04/11/2022, 03h01

Bonjour,

Il existe des fonctions mysqli comme mysqli_num_rows (exemple 2 c'est le style procedural) qui permet de récupérer le nombre de lignes retournés par une requête, donc si cette fonction retourne 0 ceci dit qu'il n y a pas de correspondance dans la table.

Code php :

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

1
2
3
4
$nbr=mysqli_num_rows($result);
if($nbr==0){
   echo "pas de correspondance...";
}

Attention, votre requête n'est pas sécurisée, vous devriez au moins utiliser mysqli_real_escape_string ou bien mieux, une requête préparée.

**escartefigue** · 04/11/2022, 08h43

Envoyé par Toufik83

Il existe des fonctions mysqli comme mysqli_num_rows (exemple 2 c'est le style procedural) qui permet de récupérer le nombre de lignes retournés par une requête, donc si cette fonction retourne 0 ceci dit qu'il n y a pas de correspondance dans la table.

Pour vérifier l'existence d'une valeur dans une table

soit on fait une requête SELECT en filtrant sur une valeur de colonne ayant une contrainte unique, idéalement la PK
soit il n'existe pas de contrainte unique, auquel cas on limite la recherche avec la clause normative FETCH FIRST n ROWS ONLY (Clause LIMIT pour MYSQL)
soit encore on utilise la clause EXISTS dans une requête imbriquée.

Compter le nombre de lignes est contre performant et inutile quand on a seulement besoin de vérifier l'existence : s'il y a un million de lignes pour le critère, on va toutes les parcourir, ce qui prend du temps et ne sert à rien.

**david77185** · 04/11/2022, 13h37

Envoyé par Toufik83

Bonjour,

Il existe des fonctions mysqli comme mysqli_num_rows (exemple 2 c'est le style procedural) qui permet de récupérer le nombre de lignes retournés par une requête, donc si cette fonction retourne 0 ceci dit qu'il n y a pas de correspondance dans la table.

Code php :

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

1
2
3
4
$nbr=mysqli_num_rows($result);
if($nbr==0){
   echo "pas de correspondance...";
}

Attention, votre requête n'est pas sécurisée, vous devriez au moins utiliser mysqli_real_escape_string ou bien mieux, une requête préparée.

Bonjour Toufik

merci pour ton aide

Tu me proposes de modifier ce bout de code :

Code :

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

$rows = mysqli_fetch_all($result, MYSQLI_ASSOC);

par le tiens ? nous sommes d'accord ?

**david77185** · 04/11/2022, 13h42

Envoyé par escartefigue

Pour vérifier l'existence d'une valeur dans une table

soit on fait une requête SELECT en filtrant sur une valeur de colonne ayant une contrainte unique, idéalement la PK
soit il n'existe pas de contrainte unique, auquel cas on limite la recherche avec la clause normative FETCH FIRST n ROWS ONLY (Clause LIMIT pour MYSQL)
soit encore on utilise la clause EXISTS dans une requête imbriquée.

Compter le nombre de lignes est contre performant et inutile quand on a seulement besoin de vérifier l'existence : s'il y a un million de lignes pour le critère, on va toutes les parcourir, ce qui prend du temps et ne sert à rien.

Bonjour Escartefigue

J'ai pris soin effectivement de creer une colonne "id" pour gerer la PK

Nom : Sans titre.png
Affichages : 198
Taille : 17,3 Ko

Mais comment dois je modifier ma requete du coup ?
Car sur le forum SQL, on m'a dit que c'etait un probleme php( plus lié au code) , donc j'ai fermé mon post sur le forum SQL en pensant que ma requete SQL etait bonne puisqu'elle me sortait bien ma valeur si et seulement si elle etait presente

ma requete actuelle

Code :

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

$result = mysqli_query($base,"select date from ma_table WHERE serial ='".$_GET['serie']."'");

**Toufik83** · 04/11/2022, 14h17

Envoyé par david77185

Tu me proposes de modifier ce bout de code par le tiens ? nous sommes d'accord ?

Pas tout à fait d'accord, essaies comme ça :

Code php :

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
<?php
    mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
    $host = "xxxx";
    $userName = "xxx";
    $password = "xxx";
    $dbName = "xxxx";
    $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion"); 
    $serial = mysqli_real_escape_string($base,$_GET['serie']??0);
    $result = mysqli_query($base,"select date from ma_table WHERE serial ='".$serial."'");
    $numRows=mysqli_num_rows($result);
    if($numRows>0){
        $rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
        foreach (...){...}
    }
    else{
        echo "aucune ligne n'a été retournée";
    }
?>

Après il faudrait aussi Prendre en considération les remarques de escartefigue.

**david77185** · 04/11/2022, 14h28

Merci Toufik,

pour info, j'ai supprimé mon code d'origine car trop compliqué
J'ai pu le simplifier entre temps pour arriver au meme point, j'y vois deja plus clair

concernant ton code , j'ai une erreur

Parse error: syntax error, unexpected '...' (T_ELLIPSIS)

Cela vient de ton while

Code :

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

  while(...){...}

Dois je remplir le while de mon coté ?

voici mon code simplifié au cas ou

Code html :

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
<!DOCTYPE html>
<html>
<head>
<!-- Latest compiled and minified CSS -->
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" integrity="sha384-BVYiiSIFeK1dGmJRAkycuHAHRg32OmUcww7on3RYdg4Va+PmSTsz/K68vbdEjh4u" crossorigin="anonymous">
<style>
 
#loading-img{
display:none;
}
 
.response_msg{
margin-top:10px;
font-size:13px;
background:#E5D669;
color:#ffffff;
width:250px;
padding:3px;
display:none;
}
 
</style>
</head>
<body>
 
<div class="container">
<div class="row">
 
 
<h1><img src="logo-web-transparent.png" width="250px"></h1>
 
<div class="col-md-8">
 
</div>
<br>
<br>
<h4>Su busqueda :  <?php echo $_GET['serie'] ?></h4>
   <?php
 
    $serial = $_GET['serie'];
    
        $host = "xxx";
    $userName = "xxx";
    $password = "xxx";
      $dbName = "xxx";
    
    mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
    $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion"); 
 
 
                  
//////////  MES REQUETES SQL /////////////
 
    $result = mysqli_query($base,"select date from contact_form_info WHERE serial ='".$_GET['serie']."'");
//  $req=mysqli_query($base,"select serial from contact_form_info WHERE serial ='".$_GET['serie']."'");  // OK
  
    $rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
    foreach ($rows as $row) {
     
   if(isset($serial))
{  
        echo "<pre>";  /// Mise en forme 
        
    printf("%s (%s)\n ", $row["date"], $serial = $_GET['serie']);
    echo isset($serial['']) ?
    :'El numero de serie existe en base de datos desde la fecha indicada !.';
    
    echo "</pre>";  /// Mise en forme fin
}
}
?> 
<br>
<br>
<br>
<br>
 
 
<form action="xxxx">
    <input type="submit" value="Grabar su producto aqui SI NO EXISTE" />
</form>
 
 
 
    </body>
    </html>

**Toufik83** · 04/11/2022, 14h34

Bien sûr que vous devriez remplir la ligne de foreach, un copier/coller me parait simple non ?

**david77185** · 04/11/2022, 14h38

Bonjour Toufik

Donc avec ton code, plus rien ne marche

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
   <?php
    mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
 
    $serial = mysqli_real_escape_string($base,$_GET['serie']??0);
 
    $host = "xx";
    $userName = "xx";
    $password = "xx";
    $dbName = "xx";
 
 
    $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion"); 
    $result = mysqli_query($base,"select date from ma_table WHERE serial ='".$serial."'");
    $numRows=mysqli_num_rows($result);
    if($numRows>0){
        $rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
        foreach (...){...}
    }else{
        echo "aucune ligne n'a été retournée".
    }
?>

Toujours avec le while ou foreach

Parse error: syntax error, unexpected '...' (T_ELLIPSIS)

**Toufik83** · 04/11/2022, 14h42

Remplacez foreach(...){...} par

Code php :

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

1
2
3
4
5
foreach ($rows as $row) {
    echo "<pre>";  /// Mise en forme 
    echo "date :".$row["date"].", serial :".$row["serial"];
    echo "</pre>";
}

Voilà.

**david77185** · 04/11/2022, 15h02

Envoyé par Toufik83

Bien sûr que vous devriez remplir la ligne de foreach, un copier/coller me parait simple non ?

desole

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
   <?php
     mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
     $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion"); 
     $serial = mysqli_real_escape_string($base,$_GET['serie']??0);
     $host = "";
    $userName = "";
    $password = "";
      $dbName = "";
 
 
 
 
 
 
    $result = mysqli_query($base,"select date from ma_table WHERE serial ='".$serial."'");
    $numRows=mysqli_num_rows($result);
    if($numRows>0){
    	$rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
 
      foreach ($rows as $row) {
 
   if(isset($serial))
{  
  	echo "<pre>";  /// Mise en forme 
 
    printf("%s (%s)\n ", $row["date"], $serial = $_GET['serie']);
    echo isset($serial['']) ?
    :'El numero de serie existe en base de datos desde la fecha indicada !.';
 
    echo "</pre>";  /// Mise en forme fin
}
}
    }else{
        echo "aucune ligne n'a été retournée".
    }
?>

J'ai une erreur

Code :

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

Parse error: syntax error, unexpected '}'

cela vient du else il me semble
je suis perdu avec les { }

**Toufik83** · 04/11/2022, 15h25

Avant d'essayer de récupérer les données d'une table mysql, il faudrait plutôt maitriser les bases de PHP.

Votre code devrait ressembler à ça :

Code php :

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
<?php
     mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
     $host = "";
     $userName = "";
     $password = "";
     $dbName = "";
     $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion"); 
     $serial = mysqli_real_escape_string($base,$_GET['serie']??0);
     $result = mysqli_query($base,"select date from ma_table WHERE serial ='".$serial."'");
     $numRows=mysqli_num_rows($result);
     if($numRows>0){
    	$rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
        foreach ($rows as $row) {  
  	   echo "<pre>";  /// Mise en forme 
           echo "date :".$row["date"].", serial :".$serial;
           echo "</pre>";  /// Mise en forme fin
        }//fin foreach
     }//fin if($numRows>0)
     else{
        echo "aucune ligne n'a été retournée";
    }
?>

**david77185** · 04/11/2022, 16h06

Bonjour Toufik

nos messages se croisent, je viens de raccrocher le wagon

donc,

Envoyé par Toufik83

Remplacez foreach(...){...} par

Code php :

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

1
2
3
4
5
foreach ($rows as $row) {
    echo "<pre>";  /// Mise en forme 
    echo "date :".$row["date"].", serial :".$row["serial"];
    echo "</pre>";
}

Voilà.

mon code :

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
   <?php
     mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
 
     $serial = mysqli_real_escape_string($base,$_GET['serie']??0);
     $host = "xxx";
    $userName = "xxx";
    $password = "xxxx";
      $dbName = "xxx";
 
 
 
 
 
        $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion");              
     $result = mysqli_query($base,"select date from ma_table WHERE serial ='".$serial."'");
    $numRows=mysqli_num_rows($result);
   if($numRows>0){
        $rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
       foreach ($rows as $row) {
    echo "<pre>";  /// Mise en forme 
    echo "date :".$row["date"].", serial :".$row["serial"];
    echo "</pre>";
}
    }else{
        echo "aucune ligne n'a été retournée".
    }
?>

J'obtiens un message d'erreur

Parse error: syntax error, unexpected '}'

Nom : Sans titre.png
Affichages : 181
Taille : 14,9 Ko

J'ai 2 accolades ouvertes et donc 2 accolades fermées , je ne vois pas ce qui cloche, je ne vois pas quoi rajouter

**david77185** · 04/11/2022, 16h19

réponse à 15H19

merci toukif

j'ai le resultat attendu avec ton code

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
   <?php
     mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
 
 
    $host = "xxx";
    $userName = "xxx";
    $password = "xxx";
    $dbName = "xxx";
 
     $base = mysqli_connect($host, $userName, $password, $dbName)or die("Erreur de connexion");  
     $serial = mysqli_real_escape_string($base,$_GET['serie']??0);            
     $result = mysqli_query($base,"select date FROM contact_form_info WHERE serial ='".$serial."'");
     $numRows=mysqli_num_rows($result);
   if($numRows>0){
     $rows = mysqli_fetch_all($result, MYSQLI_ASSOC);
        foreach ($rows as $row) {  
  	   echo "<pre>";  /// Mise en forme 
           echo "date :".$row["date"].", serial :".$serial;
           echo "</pre>";  /// Mise en forme fin
        }//fin foreach
     }//fin if($numRows>0)
     else{
        echo "aucune ligne n'a été retournée";
    }
?>

**Toufik83** · 04/11/2022, 16h22

1-Il manque un point virgule ";" après la fin de l'echo du bloc else :

Code php :

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

echo "aucune ligne n'a été retournée".

Supprimez le dernier point et remplacez le par un point virgule :

Code php :

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

echo "aucune ligne n'a été retournée";

2-Remplacez $row["serial"] par $serial dans

Code php :

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

echo "date :".$row["date"].", serial :".$row["serial"];

sinon vous aurez une erreur undefined index serial puisque la colonne "serial" n'est pas ajouté dans la requête.

**david77185** · 06/11/2022, 10h40

Bonjour Touffik

Encore milles merci pour ton aide precieuse

J'etais bloqué depuis une semaine

En revanche, tu m'as inquieté avec le fait que ma requete n'etait pas securisée

je ne voudrais pas etre hacké .

je vais ouvrir un nouveau post ici

et passer en resolu ( une seconde fois) ce post

David

REquete SQL + PHP [ Impossible de sortir un second echo en cas de valeur vide ]

PHP & Base de données

Discussions similaires

Partager

Partager