Øhhhh det kunne så ikke lige lade sig gøre at paste PHP ind i en tråd her :-(
He he nyt problem :-) men jeg kan sende på en mail hvis der skulle være en venlig sjæl som vil kigge mit script igennem, det er kun ca 40 linier
--
/ Rolle
Pastebin det nu bare som en hver anden normal person ville gøre...
Ingen gider til at fedte med mail frem og tilbage, hvis du må have hjælp, må du altså også selv gøre lidt.
--
Gæstebruger, opret dit eget login og få din egen signatur. Prøver lige sådan her i stedet.
http://gupl.dk[...]
Her er et billede af min kode.
#2 Ja det er rigtigt, tror også det er ved at være lidt sent, er måske ikke helt så frisk længere :-)
--
/ Rolle ; Skal ikke på som afslutning.
--
Du kan ikke afslutte en block med semikolon, som #4 skriver.
--
Desuden så kan din kode injectes så let som ingen ting, brug i det mindste mysql escape string på dit password.
--
#6 Det kan den sådan set ikke.. Han bruger ikke noget brugerinput i sit SQL statement.
--
Er det det ; i lilje 37 i mener der er for meget, for det giver nøjagtigt det samme resultat :-(
Ideen er at pass kommer via $_get fra en form i min index fil, så jeg får vel bruger input..... eller ?
Tænker bare at jeg i første omgang skal have det til at fungere, så må jeg se om jeg kan finde ud af at lave det lidt mere sikkert bagefter :-) det er bare for at prøve at lege lidt med det, man skal jo starte et sted.
--
/ Rolle
#8 Alle steder du sætter et semikolon efter en block. Også linje 23.
--
#7 Ja sad jeg også lige og tænkte, men jeg regner forhåbentlig med at han tager skridtet videre og rent faktisk sender det hele videre til databasen. Samt husker id.
Du skal også huske at hashe dine passwords.
--
Gæstebruger, opret dit eget login og få din egen signatur. Har også fjernet det efter linje 23, lige lidt hjælper det :-(
Men det undrer mig også lidt at jeg ikke får at vide om det er forkert pass eller rigtigt, det er ligesom den aldrig afvikler de linjer, eller er det bare mig som er noob
#11
Sådan som jeg regner med at det fungerer er at jeg får det password som kommer via min index side, og bliver til $password, det tager jeg så og sammenligner med et jeg har liggende i min DB, men det kan man vel ikke uden videre bare sende en request på, som udefra kommende... eller hva ?
--
/ Rolle
#12 Et godt råd: lær lidt om det du arbejder med, inden du begynder at copy-paste tilfældig kode. Det er ikke for at være et bedrevidende fjols, men det hjælper altså en del.
Du mangler at afslutte while løkkens block. Indsæt et "}" på linje 22.
--
Din require_once er også forkert.
Den skal være require_once("mysql_connect.php"); i stedet :)
--
#14 Det er ikke nødvendigt.
--
#15.. Nå.. Sådan plejer jeg normalt at gøre, så den så bare noget mærkeligt ud :P
--
#12
Jamen jeg syntes nu også jeg har prøvet at sætte mig ind i hvad jeg laver, det er bare svært hvis man ikke får lidt praktisk erfaring.
Jeg har set timevis af toturials på youtbe, og læst "Php og mysql" og lavet en masse af de øvelser der er der i.
Det som jeg sidder med her, har jeg klippet sammen fra flere toturials på youtube, hvor hver linje er blevet grundigt forklaret, så jeg er måske ikke skrap til det, men jeg har da en ide om hvad jeg har klippet sammen :-)
Nu har jeg flytter den "}" som stod på linje 23 til linje 22, ikke at det har gjort nogen forskel.
--
/ Rolle
#17 Du skal tilføje en til, ikke flytte den. Du har ikke afsluttet virkefeltet for din while løkke.
--
Det lader til at det virkede, nu kan også godt se at den ekstra "}" skal være der :-)
Nu får jeg en ny fejl i line 15 :-)
Catchable fatal error: Object of class mysqli could not be converted to string in C:\xampp\htdocs\websites\xxxxxx\Scripts\login.php on line 15
Så har jeg lidt at arbejde videre med, jeg vender tilbage hvis jeg løber tør for ideer igen, men mange tak for hjælpen, og god jul til jer alle :-)
--
/ Rolle
#19 > Den korrekte syntax er mysql_query(query, connection) :)
Dvs. mysql_query($myConnection, $sqlcommand) - prøv det.
--
Fuldtids nørd - med et strejf af kaffe!
Webudvikler og designer - www.MARTINC.nu[...]
En mening er vel tilladt? - www.MARTINCARLSEN.dk[...] Vend dig til at bruge indentering, så ser du hurtigt hvis der mangler brackets.
Brug eventuelt en editor der gør det automatisk.
Og brug pastebin.com til din kode, så kan folk se koden korrekt og lave rettelser du kan se.
--
Core i7-920 D0 @ 2.66 GHz -|- Gigabyte GA-X58-UD4 -|- Kingston 6x2 DDR3 (1333 MHz) -|- GTX285 1GB DDR3 -|- Windows 7 Ultimate 64bit