jeg tror det er nemmere for os andre og sikkert også for dig, hvis du formaterede coden korrekt med indrykninger.
hvis din editor ikke kan lave autoformatering kan jeg anbefale
https://netbeans.org/når du ligger code her på eksperten, så pak det ind i en blå box
(div)(pre)her skal din code stå(/pre)(/div)
nb: paranteserne skal være de kantede, så () skal []
læs guiden
http://www.eksperten.dk/guide/1325det gør indlægget nemmere at læse
står der det rigtige i denne udskrift ??
while ($stmt->fetch()) {
echo $ip;
echo $hits;
}
hvad står der hvis IP findes ??
hvad står der hvis IP
ikke findes ??
det du vil gøre kan skrives og tested kort som jeg skrev i
http://www.eksperten.dk/spm/1007095#reply_8200129det du gør er at vælte hele lortet sammen, og så ryger overblikket
du kan gøre din code nemmere at
læse / vedligeholde / genbruge / teste hvis du opbyggede det i functioner.
<?php
require_once("model.php"); // her ligger connection og alle sql funktioner
$privat_ip = $_GET['privat'];
$status = besogstaeller_IsIpCreated($hellestrik, $privat_ip);
if ($status == 0) { // ip addresen findes ikke
besogstaeller_Create($hellestrik, $privat_ip);
} else { // ip addresen findes, så skal den opdateres med et hit mere
besogstaeller_Update($hellestrik, $privat_ip);
}
?>
en bonus ved at lave det det som funktioner, er at du kan spare en del af din code, og ved at placere alle sql functioner i en fil gør du det nemt at skifte database API, feks over til PDO (kan kun anbefales)
besogstaeller_IsIpCreated er den vigtigste.
besogstaeller_Create er næsten lavet i
http://www.eksperten.dk/spm/1007095#reply_8200117 skal bare pakkes ind som en function
besogstaeller_Update har du selv et bud på, der skal pakkes ind som en function
men alle functioner kan laves og testes hver for sig, uden de andre er lavet først