Lege pagina?
Prospere Gaming op 25/02/2015 15:33:59:
de reden dat ik niet met mysqli werk is dat ik me er nog niet in verdiept heb dus dat tot nog toe alleen errors geeft ipv werkt.
Dus je gaat alles 2x doen? ;)
Zo moeilijk is het omzetten van mysql naar mysqli niet.
Ten aanzien van je code:
Waarom maak je variabelen als $ip en $username aan (regel 11/ 16 / 48)?
Wat is het nut van regel 16 en 48? Gebeurt hetzelfde?!
Ik mis foutafhandeling bij je query (o.a. regel 20 / 51).
In de head plaats je al een header (regel 34 e.v.). Volgens mij hoort dat daar niet (maar in de body (en die mist dan weer)).
Inline opmaak (/css) is niet meer echt van deze tijd (o.a. regel 35).
Waarom een while-lus (regel 53)? Als het goed is levert de query maar 1 resultaat op. En ook het aanmaken van de variabelen op de regels 54-60 is niet nodig.
Waarom is er verschil tussen hoe je programmeert op regel 50+51 en 62?
Denk overigens dat ik de query op regel 62 (en de volgende regels) naar boven zou halen (op regel 11 maak je al een variabele $ip aan).
Op regel 98 plaats je tekst via echo, een paar regels later sluit je PHP af om HTML output te genereren. Mij lijkt uniformiteit overzichtelijker.
Begrijp niet goed waarom je op regel 98 "' hebt staan.
Gewijzigd op 25/02/2015 22:22:46 door Obelix Idefix
mysql = Oud gebruik dit niet maak er mysqli van het overzetten kost hooguit een half uurtje.
Je gebruik verder direct input in een database.
Stel ik kan ergens iets invullen dat gebruik maakt van je database dan hoef ik alleen dit te doen.
Code (php)
1
2
3
4
2
3
4
<?php
$sql = "UPDATE Accounts SET ip='$ip' WHERE naam='$username'";
mysql_query($sql);
?>
$sql = "UPDATE Accounts SET ip='$ip' WHERE naam='$username'";
mysql_query($sql);
?>
Enigste dat ik moet doen is.
in de input invullen :
ik hoef het niet af te sluiten ); want dat doet jou script al.
als ik dit invul dan verwijder ik gelijk al je gebruikers.
Gewijzigd op 25/02/2015 23:01:32 door Eeyk Vd noot