Requête calcul moyenne sans les valeurs NULL

Version imprimable

Voir 40 message(s) de cette discussion en une page

Citation:

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:

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:

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.

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:

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:

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:

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.

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

Je corrige :

Code:

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;

15/12/2023, 13h40
Delba146

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:

$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:

$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.
17/12/2023, 12h32
unanonyme

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.

Voir 40 message(s) de cette discussion en une page