Er dettte login system ok?

Det ser lidt kompliseret ud, men eller ser det godt ud!

Det nemmeste tror jeg at finde en lidt mere nem kode for den der bliver lidt svær at lave videre på.

Prøv at kigge på det loginsystem på nickwyke.dk

Mit take på den kode:

Pæn velstrukturet kode skrevet af en programmør, som ved hvad det handler om. Endda objekt orienteret.

Dog lidt forældet efterhånden. Der er sket en del med PHP siden 2004 hvor scriptet er fra:

Understøtter kun to niveauer af brugerroller: bruger og admin (og gæst).

Bruger addslashes(). Jeg ville nok foretrække mysql_real_escape_string() i stedet.

Bruger mysql_numrows() i stedet for mysql_num_rows(). mysql_numrows() er deprecated - måske kan den slet ikke bruges på dit webhotel.

Bruger de gamle ereg-funktioner for regulære udtryk. Disse er meget snart på vej ud af PHP. Man bør bruge preg-funktionerne i stedet.

I øvrigt har han nogle lidt spøjse regulære udtryk ... som om at han ikke helt har styr på lige den del af koden.

Forresten burde nogle af hans møstre have være placeret sammen med de andre define's.

Det ser ud til at han har holdt sig pænt til at bruge $_POST i stedet for de mere hurtige og grimme alternativer. Smukt!

Han har ellers defineret en pæn Session-klasse. Alligevel er der meget session gymnastik som ikke går via den. Lidt uforståligt...

Når han redirecter med header() går han ud fra at filerne hedder noget specifikt. Du bliver nok nødt til at justere din nuværende struktur hvis du skal bruge det. Hoved filen skal f.eks. hedde "main.php".

.oOo.

Absolut pæn kode som det er værd at lade sig inspirere af. Det bærer dog præg af at være skrevet for 3 år siden, og der er ingen garanti for at det nu vil køre på det webhotel du har. Desuden ville det under alle omstændigheder snart skulle igennem en kraftig opdatering.

Ikke helt enig med superdreng om at det er svært at lave om på ... men man skal være godt inde i PHP programmering og OOP tankegangen for at kunne gøre det. Men så er det til gengæld også ret nemt at finde ud af hvor der skal rettes og hvordan det skal gøres.

Tak for en fantastisk gennemgang.

nielle -> Lig et svar!

Svar :^)

Så mangler du bare at acceptere mit svar :^)

??

Beklager Vente tiden!

Takker for point :^)