PHP/MySQL: Krijg variabelen niet in MySQL querywerkend.

[toxict]

Ik probeer een query te bouwen, zodat ik vanuit PHP een mySQL database kan query-en.
De bedoeling is dus dat de "resultaten" van de 2 variabelen hieronder in de query daaronder terecht komen.

$date_yesterday_last_row = date('Y-m-d H:i:s', strtotime('today'));
$date_yesterday_first_row = date('Y-m-d H:i:s', strtotime('yesterday'));

$sql_001 = "SELECT MAX(p1_meterreading_in_1) FROM p1_log WHERE p1_timestamp BETWEEN $date_yesterday_first_row AND $date_yesterday_last_row;";

Op een of andere manier komen de resultaten niet in de query terecht.
Als ik de query direct uitvoer met 2 data in de TIMESTAMP format, dan krijg ik direct en goed resultaat....mijn conclusie...de query is goed...alleen de variabelen er goed in krijgen niet.

Wie helpt mij?

[Onoffon]

Probeer eens aanhalingstekens rond de variabelen te zetten, dus:

$sql_001 = "SELECT MAX(p1_meterreading_in_1) FROM p1_log WHERE p1_timestamp BETWEEN '$date_yesterday_first_row' AND '$date_yesterday_last_row';";

[ThinkPad]

Ik deed het zo:

PHP:

    $SQL = "INSERT INTO p1.readings (id, time, laag_tarief, hoog_tarief, huidig_verbruik, gas) 
VALUES (NULL, NULL, '".$mEVLT."', '".$mEVHT."', '".$mEAV."', '".$mG."')";     
mysql_query($SQL);

Misschien heb je wat aan mijn code: ThinkPad's TweakBlog: Uitlezen van de slimme meter P1-poort met een Arduino en waarden opslaan in MySQL-database (gebruik het nu niet meer overigens, vervangen door een Raspberry met Domoticz).

P.s. gebruik [code=php]<hier_code> [ /code] om je regels code leesbaarder te maken

[ Voor 6% gewijzigd door ThinkPad op 02-02-2015 21:47 ]

[Aionicus]

Stap 1 zou zijn om even te kijken naar wat je krijgt als je $sql_001 echo'd

laat hij daar in de query al wel de waardes goed zien ? zo ja dan is het gewoon even simpel escapen en dan zou het moeten werken. als hij daar al de waardes niet goed neer zet dan heb je een quotje verkeerd staan.

[toxict]

ThinkPadd schreef op maandag 02 februari 2015 @ 21:46:
Ik deed het zo:

PHP:

    $SQL = "INSERT INTO p1.readings (id, time, laag_tarief, hoog_tarief, huidig_verbruik, gas) 
VALUES (NULL, NULL, '".$mEVLT."', '".$mEVHT."', '".$mEAV."', '".$mG."')";     
mysql_query($SQL);

Misschien heb je wat aan mijn code: ThinkPad's TweakBlog: Uitlezen van de slimme meter P1-poort met een Arduino en waarden opslaan in MySQL-database (gebruik het nu niet meer overigens, vervangen door een Raspberry met Domoticz).

P.s. gebruik [code=php]<hier_code> [ /code] om je regels code leesbaarder te maken

Ik krijg bij het lezen van " INSERT INTO" het idee dat ik waardes in een tabel ga plaatsen.
Mijn bedoeling is om middels variabelen een query te bouwen om juist iets uit te lezen...
Zie ik dit goed of heb ik het mis?

[pedorus]

Ik zou dat voorbeeld vermijden, mysql_* is deprecated, en een sql statement als deze hoort parameterized queries te gebruiken. Zie iets als http://php.net/manual/en/pdo.prepared-statements.php

[ThinkPad]

toxict schreef op maandag 02 februari 2015 @ 21:53:
[...]


Ik krijg bij het lezen van " INSERT INTO" het idee dat ik waardes in een tabel ga plaatsen.
Mijn bedoeling is om middels variabelen een query te bouwen om juist iets uit te lezen...
Zie ik dit goed of heb ik het mis?

Het was ook niet bedoeling om de hele query 1:1 over te nemen..... Het was meer als voorbeeld van hoe ik de PHP-variabelen in de query gebruikte.

pedorus schreef op maandag 02 februari 2015 @ 21:55:
Ik zou dat voorbeeld vermijden, mysql_* is deprecated, en een sql statement als deze hoort parameterized queries te gebruiken. Zie iets als http://php.net/manual/en/pdo.prepared-statements.php

Klopt, dit was gewoon voor de hobby thuis, voor in productie was ik niet van plan om deprecated commando's te gaan gebruiken.

[toxict]

Een ECHO van $SQL_001 geeft als resultaat: SELECT MAX(p1_meterreading_in_1) FROM p1_log WHERE p1_timestamp BETWEEN '2015-02-01 00:00:00' AND '2015-02-02 00:00:00';

Als ik het resultaat in mysql plak, dan krijg ik de waarde terug die ik ook zoek.
Ik begrijp daarom dan niet wat er in het PHP gedeelte fout gaat.

PHP:

<html>
<?php
$servername = "localhost";
$username = "username";
$password = "password";
$dbname = "p1";

// Create connection

$conn = new mysqli($servername, $username, $password, $dbname);

// Check connection

if ($conn->connect_error)
    {
    die("Connection failed: " . $conn->connect_error);
    }

// ------------------------------------------------------------------------------------------------------------------------------------------------- \\

$date_yesterday_last_row = date("Y-m-d H:i:s", strtotime('today'));
$date_yesterday_first_row = date("Y-m-d H:i:s", strtotime('yesterday'));

$sql_001 = "SELECT MAX(p1_meterreading_in_1) FROM p1_log WHERE p1_timestamp BETWEEN '$date_yesterday_first_row' AND '$date_yesterday_last_row';";
$result = $conn->query($sql_001);

<Hier moet wat komen om het resultaat weer te geven.>

// ------------------------------------------------------------------------------------------------------------------------------------------------- \\

$conn->close();
?>  

</body>
</html>

[Onoffon]

Doet een simpele select het wel? En heb je al geprobeerd om in je PHP de timestamps hard-coded neer te zetten?

[Aionicus]

Zou inderdaad mezelf even afvragen wat je terug krijgt vanuit de database, pakt hij een andere select of een simpele grab from id wel ?

[Gomez12]

@TS : Hoe weet je dat je geen resultaat krijgt?

In je stukje voorbeeldcode staat : "<Hier moet wat komen om het resultaat weer te geven.>" maar zolang daar niets kloppends staat kan je query goed zijn, maar je gaat gewoon niets getoond krijgen.

Persoonlijk vermoed ik simpelweg dat je de verkeerde kolom probeert op te vragen in je resultaat... Ik zou je daarom adviseren om je max() een alias mee te geven zodat je zelf 100% duidelijk is wat de kolomnaam is die je hebt gecreeerd.

Maar opzich valt er weinig over te zeggen zonder meer info...

[Merethil]

Volgens mij open je ook nergens je body van je HTML. Maar daarnaast: Hoe probeer je je resultset uit te printen op het scherm? Probeer eens een var_dump van de $result, kijk wat erin zit.

[toxict]

Goedemorgen....thanks voor alle hulp....heb niet zoveel kennis van PHP, maar jullie antwoorden hebben mij wat inzichten gegeven en ik ben verder gekomen.

ECHO van de SQL query gaf het juiste resultaat..Dit resultaat kon ik direct in MySQL plakken en dat werkte uiteindelijk.

Kwam erop neer dat ik de variabelen om moest draaien.

(PS: Mijn code hier op de site was niet complete, wel op mijn system...excuses)

-slotje-

[Verwijderd]

Hoewel het misschien niet helemaal "on topic" is zou ik in dit soort scenario's kiezen voor geparametriseerde stored procedures. Dat scheelt je een hoop gekl**t met dubbele aanhallingstekens etc. en je kunt je queries makkelijker testen/debuggen en aanpassen etc.

[cariolive23]

Je kunt "gisteren" ook berekenen met SQL: CURRENT_DATE - INTERVAL 1 DAY. Heb je ook geen quotes e.d. meer nodig, gewoon plain SQL. Met SQL kun je zoveel meer dan alleen maar een recht-toe-recht-aan SELECT....

Voor de liefhebber, Modern SQL: http://www.slideshare.net/MarkusWinand/modern-sql