Requête calcul moyenne sans les valeurs NULL

**Delba146** · 14/12/2023, 19h22

Envoyé par Séb.

Non, tu manipules des numériques il faut donc un type SQL DECIMAL (ou INTEGER si tu as seulement des entiers), le VARCHAR sert uniquement pour du texte.
Les difficultés autour de cela viennent d'une méconnaissance et pas du type de données utilisé.

Insérer les données correctement. Quand la valeur est absente tu utilises le marqueur SQL NULL => La fonction SQL NULLIF() sera utile

NULLIF(param1, param2) -- Renvoie NULL si param1 est égal à param2, sinon renvoie param1

https://dev.mysql.com/doc/refman/8.0...unction_nullif

Le script PHP est une vraie horreur.

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
function insertReading($sensor, $location, $value1, $value2, $value3, $value4, $pm25, $pm10) {
    global $servername, $username, $password, $dbname;
 
    // Create connection
    $conn = new mysqli($servername, $username, $password, $dbname);
    // Check connection
    if ($conn->connect_error) {
      die("Connection failed: " . $conn->connect_error);
    }
 
    $sql = "INSERT INTO SensorData (sensor, location, value1, value2, value3, value4, pm25, pm10)
 
       VALUES ('" . $sensor . "', '" . $location . "', '" . $value1 . "', '" . $value2 . "', '" . $value3 . "','" . $value4 . "', '" . $pm25 . "', '" . $pm10 ."')";
 
   if ($conn->query($sql) === TRUE) {
      return "New record created successfully";
 
    }
}

En le remaniant un minimum :

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
function insertReading($sensor, $location, $value1, $value2, $value3, $value4, $pm25, $pm10)
{
    global $servername, $username, $password, $dbname;
 
    // La connexion devrait être un paramètre de la fonction
    $conn = new mysqli($servername, $username, $password, $dbname);
    if ($conn->connect_error) {
      die("Connection failed: " . $conn->connect_error);
    }
 
    $sensor = $conn->real_escape_string($sensor);
    $location = $conn->real_escape_string($location);
    $value1 = $conn->real_escape_string($value1);
    $value2 = $conn->real_escape_string($value2);
    $value3 = $conn->real_escape_string($value3);
    $value4 = $conn->real_escape_string($value4);
    $pm25 = $conn->real_escape_string($pm25);
    $pm10 = $conn->real_escape_string($pm10);
 
    // Syntaxe INSERT/SET, propriétaire MySQL mais tellement plus lisible...
    // Je te mets du NULL partout si chaîne de valeur vide, à adapter
    $sql = <<<SQL
        INSERT INTO SensorData
        SET
            sensor = '{$sensor}',
            location '{$location}',
            value1 = NULLIF('{$value1}, ''),
            value2 = NULLIF('{$value2}, ''),
            value3 = NULLIF('{$value3}, ''),
            value4 = NuLLIF('{$value4}, ''),
            pm25 = NULLIF('{$pm25}', ''),
            pm10 = NULLIF('{$pm10}', '')
         SQL;
 
    if ($conn->query($sql) !== TRUE) {
        return "INSERT failed";
    }
 
    return "New record created successfully";
}

Bonjour,
J'ai saisi les quelques lignes de code que tu as proposé mais j'ai l'erreur suivante :

Parse error: Invalid body indentation level (expecting an indentation level of at least 10) in /storage/ssd4/987/21478987/public_html/esp-database.php on line 47

La ligne 47 correspondant à : INSERT INTO SensorData je pense qu'il y a une erreur de syntaxe avec la ligne précédente j'ai essayé de trouver un exemple similaire pour corriger mais sans succés.

**unanonyme** · 14/12/2023, 20h16

Bonjour,

c'est la syntaxe HEREDOC qui est malformée.

https://www.php.net/manual/fr/langua...syntax.heredoc

Il faudrait, au choix, décaler la requête vers la droite,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
    $sql = <<<SQL
		INSERT INTO SensorData
		SET
		    sensor = '{$sensor}',
		    location '{$location}',
		    value1 = NULLIF('{$value1}, ''),
		    value2 = NULLIF('{$value2}, ''),
		    value3 = NULLIF('{$value3}, ''),
		    value4 = NuLLIF('{$value4}, ''),
		    pm25 = NULLIF('{$pm25}', ''),
		    pm10 = NULLIF('{$pm10}', '')
         SQL;

Ou, décaler les identifiants HEREDOC vers la gauche

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
$sql = <<<SQL
	INSERT INTO SensorData
	SET
		sensor = '{$sensor}',
		location '{$location}',
		value1 = NULLIF('{$value1}, ''),
		value2 = NULLIF('{$value2}, ''),
		value3 = NULLIF('{$value3}, ''),
		value4 = NuLLIF('{$value4}, ''),
		pm25 = NULLIF('{$pm25}', ''),
		pm10 = NULLIF('{$pm10}', '')
SQL;

Il me semble qu'on peut tout aussi bien écrire ceci avec des guillemets double, à vérifier.

Code :

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

1
2
3
4
5
6
7
8
9
10
$toto = "INSERT INTO SensorData
	SET
		sensor = '{$sensor}',
		location '{$location}',
		value1 = NULLIF('{$value1}, ''),
		value2 = NULLIF('{$value2}, ''),
		value3 = NULLIF('{$value3}, ''),
		value4 = NuLLIF('{$value4}, ''),
		pm25 = NULLIF('{$pm25}', ''),
		pm10 = NULLIF('{$pm10}', '')";

Bonne journée.

**Séb.** · 14/12/2023, 21h53

En effet, mauvaise indentation du terminateur (1 espace de trop).

Je corrige :

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
 
    // Syntaxe INSERT/SET, propriétaire MySQL mais tellement plus lisible...
    // Je te mets du NULL partout si chaîne de valeur vide, à adapter
    $sql = <<<SQL
        INSERT INTO SensorData
        SET
            sensor = '{$sensor}',
            location '{$location}',
            value1 = NULLIF('{$value1}, ''),
            value2 = NULLIF('{$value2}, ''),
            value3 = NULLIF('{$value3}, ''),
            value4 = NuLLIF('{$value4}, ''),
            pm25 = NULLIF('{$pm25}', ''),
            pm10 = NULLIF('{$pm10}', '')
        SQL;

**Delba146** · 15/12/2023, 13h40

Bon j'ai bien avancé.
J'ai standardisé ma BDD avec désormais uniquement des données en DECIMAL j'ai effectué les modifications pour insérer les valeurs comme proposé précédemment et ça fonctionne,

Vous m'aviez proposé cette ligne lors de l'insertion des données dans la BDD, est ce encore utile ? :

Code :

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

$pm10 = $conn->real_escape_string($pm10);

Je vais ce genre de requête pour le calcul des moyennes Auparavant avec mon bazar de valeurs en varchar j'avais contourné le pb avec cela : et je me demande si cela est encore utile le CAST

Code :

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

$sql = "SELECT AVG(CAST(" . $value . " AS SIGNED)) AS avg_amount FROM SensorData WHERE reading_time BETWEEN NOW() - INTERVAL ". $nbdays ." DAY AND NOW()";

Je vais encore continuer à bosser mais un grand merci.

**unanonyme** · 17/12/2023, 12h32

Bonjour,

à mon avis plutôt que d'utiliser mysqli_real_escape_string et la concaténation de string,
comme dans ("SELECT AVG(CAST(" . $value . " AS SIGNED))..."),
vous devriez vous coller à utiliser des requêtes préparées avec des paramètres nommés.

https://www.php.net/manual/fr/mysqli...statements.php

mysqli_real_escape_string nécessite un tout petit peu de paramétrage dont je suis à peu près certain
que l'on ne peut pas s'appuyer dessus de manière sereine.
https://www.php.net/manual/fr/mysqli...ape-string.php

Bonne journée.

Requête calcul moyenne sans les valeurs NULL [MySQL]

PHP & Base de données

Discussions similaires

Partager

Partager