Pagina ombouwen naar PHP 7. MySQLI_query mist $query parameter

zie http://php.net/mysqli-query

mysqli_query heeft per se minimaal 2 parameters nodig.

mysql_query() had ook de mogelijkheid om er 2 mee te geven, maar had de 2e optioneel.
nu zijn nr 1 en 2 van plek gewisseld EN verplicht

[size=xsmall]Toevoeging op 29/01/2019 10:05:51:[/size]

en op regel 22 maak je de betreffende var aan: $msql;

Staat in de handleiding.

Procedural style :

http://php.net/manual/en/mysqli.query.php

Vergelijk o.a. regel 30 met de voorbeelden.

Bedankt voor jullie antwoorden!

Als ik het goed begrijp moet ik er dus dit van maken? de $msql parameter meenemen om de connectie te specificeren.

$sql_ip = mysqli_query($msql,"SELECT * FROM animoso_ipblock WHERE ip = '$uip'");

Klopt! Al is het beter om je velden te benoemen in plaats van * te kiezen.

als je een lijst wilt maken van opmerkingen over dit script: neem meer dan 1 blaadje papier mee....

Er zitten echt juweeltjes tussen.
<?php
$vandaag = Date("d-m-Y");
$datearr2 = explode("-",$vandaag);
$d_vandaag = mktime(0,0,0,$datearr2[1],$datearr2[0],$datearr2[2]);
$datum_vandaag = strftime("%d-%m-%Y",$d_vandaag);
?>

Waarom niet niet
<?php
$vandaag = Date("d-m-Y");
$datum_vandaag = $vandaag;
?>


of
<?php
$d = mysqli_query("SELECT * FROM stats");
while ($record = mysqli_fetch_object($d))
{
$dd = $record->dag;
}

// Als een nieuwe dag aanbreekt, voer dan de volgende acties uit:
if ($dd <> "$nb_dag") {
}
?>
Je haalt dus alle records op uit een tabel. Volgorde willekeurig.
je loopt door de resultaten, en kijkt tenslotte daarna, (nadat je misschien inmiddels 1000 keer zinloos een rondje hebt gedaan, of de inhoud van het laatste record toevallig het nummer bevat van de datum van de huidige dag.
(zonder relatie met welk record dat was, en of er misschien ook nog een maand en een jaar van belang waren.

Datum, waar je tenminste nog ooit iets zinnigs mee wilt, sla je op in een kolom van het type DATE (of DATETIME)

Dus dan zou de query zijn:

<?php
$d = mysqli_query($msql, "SELECT COUNT(1) vandaaggedaan FROM stats WHERE datum >= CURRENT_DATE;");
$dd = $record->vandaaggedaan ;

// Als een nieuwe dag aanbreekt, voer dan de volgende acties uit:
if ($dd == 0) {
}
?>


En zulke grappen met datums doe je bij bijna elke query. 4 of 5 regels om allerlei onderdelen van datums te goochelen tot je een datum in een formaat hebt.
Terwijl de database gewoon met NOW() of CURRENT_DATE kan bepalen wat de datum van dit moment is.


Verder:
INSERT query's.
Als je daarin ook de kolommen noemt, hoef je a) niet een auto-increment kolom met een lege string te vullen en b) loop je niet het risico dat de volgorde ooit wijzigt, of als je een kolom toevoegt, je alle query's moet aanpassen. Dus ook de query's die eigenlijk niets met die kolom gaan doen.

Zeker ook, omdat veel variabele nogal korte namen hebben: $dd, $d of $tijd_nb.

(waarbij tijd_nb dus in een onbruikbaar formaat wordt opgeslagen: NOW() in een datetime kolom was eenvoudiger geweest EN je zou er ook op kunnen sorteren.

@Ivo_p,

Bedankt voor op opmerkingen. Dit script zelf is alleen de pagina waarmee de hoofdconnecties worden gelegd. De website telt nog zo'n 60 andere pagina's met script. Ik moet dus ergens gaan beginnen.

We hebben als band in 2008 een eigen ledensite gebouwd volledig in PHP, deze werkte prima totdat de hosting partij overging naar PHP versie 7.0. Daarmee zijn we nu verplicht de webpagina compleet na te kijken. En heel eerlijk heb ik sinds de bouw van de website niet meer veel met PHP gedaan. Ik probeer er dus weer in te komen, maar heb verder zelf geen uitgebreide kennis.

Misschien een rare vraag, maar is het wel om te bouwen? We zijn er ook over aan het nadenken om de ledensite door een professional na te laten kijken, maar dat zal ook een aardige duit geld kosten.

Maar ik zal zeker de aanbevelingen mee nemen. Ik zag bijvoorbeeld ook al dat het niet meer mag starten met <?, maar nu echt <?php nodig heeft.

Er valt hierboven aan bijna elke regel iets te verbeteren.

Ik persoonlijk zou geheel opnieuw beginnen, en kijken of de oude data omgezet kan worden, zodat het niet verloren gaat.


Het is niet alleen een kwestie van verouderde functies (zoals mysql) maar ook onhandige dingen als "haal alle statistieken op (over 10 jaar dus kennelijk), doe er niets mee en kijk wat je als laatste binnen krijgt.
of in 4/5 regels met stukken datum schuiven tot je het formaat hebt dat je wilde.


PHP 7.0 is trouwens ook al verouderd.
Ik zou nu voor 7.2 of 7.3 gaan.

Of kijken of je site niet in een framework als Wordpress gehangen kan worden (of ander meer op community gericht zoals Elgg of zo (als dat nog bestaat)

Dat framework was ons eerste idee. Dan hoeven zelf niets meer naar PHP om te bouwen. Helaas is er niets passends te vinden in de plug-ins van Wordpress of Joomla. We kunnen nog eens kijken of we misschien zelf daar iets kunnen bouwen, maar dan ben je alsnog zelf aan het bouwen.

Ik maak gebruik van PHPStorm om de code te controleren. Ik denk dat die automatisch in de laatste versie van PHP controleert. Maargoed, ik zou verwachten dan ondanks dat we heel omslachtig de data binnenhalen en gebruiken het wel zou moeten werken als we het script goed aanpassen. Desondanks krijg ik alleen een databaseconnectie, maar krijg geen query's uitgevoerd. Daarop heb ik eens een simpel stukje code in een pagina gezet en ook deze geeft geen data terug.

Doe ik nu echt zelf iets fout of zit er ook iets met de database niet goed?

<?php
$servername = "server";
$username = "user";
$password = "pass";
$dbname = "db";

// Create connection
$conn = mysqli_connect($servername, $username, $password, $dbname);
// Check connection
if (!$conn) {
die("Connection failed: " . mysqli_connect_error());
}

$sql = "SELECT id, vnaam, naam FROM animoso_leden";
$result = mysqli_query($conn, $sql);

if (mysqli_num_rows($result) > 0) {
// output data of each row
while($row = mysqli_fetch_assoc($result)) {
echo "id: " . $row["id"]. " - Voornaam: " . $row["vnaam"]. "-Achternaam" . $row["naam"]. "<br>";
}
} else {
echo "0 results";
}

mysqli_close($conn);

zet eens bovenaan in het script
<?php
error_reporting(E_ALL);
ini_set('display_errors', true);
?>

Krijg je dan meldingen te zien?

[size=xsmall]Toevoeging op 30/01/2019 15:32:33:[/size]

ep op de huidige regel 16 zou kunnen helpen:

<?php
if(!$result) {
echo mysqli_error($conn);
}
?>

Ik krijg meldingen. Mijn verbinding naar de database werkt geeft het script aan. Maar hij kan de sqli scripts niet uitvoeren zover ik het begrijp.

Notice: Trying to get property of non-object in /home/public/sites/timo.animoso.nl/simple.php on line 21
0 resultsSuccess: A proper connection to MySQL was made! The my_db database is great.
Warning: mysqli_get_host_info(): Couldn't fetch mysqli in /home/public/sites/timo.animoso.nl/simple.php on line 39
Host information:
Warning: mysqli_close(): Couldn't fetch mysqli in /home/public/sites/timo.animoso.nl/simple.php on line 41