Else returnere ikke ?

Når du laver en while i din kode, lægger det op til at din SQL skal/kan returnere flere resultater. Men i praksis vil der højst være ét resultat.

Desuden har du også noget redudant kode, det er ikke noget problem, men gør måske koden lidt sværre at gennemskue. Du laver en SQL hvor brugernavn og password skal passe og bagefter checker du det igen i din PHP kode.

Istedet for et while loop, kunne du lave et check på hvor mange resultater der bliver returneret. Koden er ikke testet.

$query = mysql_query("SELECT * FROM brugere WHERE brugernavn='$bruger' AND password='$hash'") or die(mysql_error());

if (mysql_num_rows($query)>0)
{
$row = mysql_fetch_assoc($query);
	//if ($row['brugernavn'] == $bruger && $row['password'] == $hash)
	//{

		mysql_query("INSERT INTO tagwall (navn, besked) VALUES ('$bruger', '$besked')") or die(mysql_error());
	
			echo "<br><br><center>";
			echo "Hej $bruger. Din besked er registreret<br>";
			echo "<br><br></center>";
	
	//}
}
else	{
	echo "Du skal enten have en bruger, eller skrive de korrekte informationer for at benytte tagwall´en";
	
}

$query = mysql_query("SELECT * FROM brugere WHERE brugernavn='$bruger' AND password='$hash'") or die(mysql_error());

while($row = mysql_fetch_assoc($query))

Din sql giver kun et resultat, hvis der findes en bruger med det brugernavn og password. Hvis der ikke er noget resultat bliver din while ikke udført.

Så ved forkert login vil koden aldrig nå der til, hvor den skal udføre din IF sætning.

Hej Stefan.

Tak for dit svar. Har du også en løsning på det?
Jeg er ret ny i faget, og kan sku ikke lige gennemskue løsningen :-)

Det må jeg kigge på når jeg kommer hjem.

Jeg er ikke helt inde i de forskellige løkker osv endnu

Tak for svar! Jeg håber det virker, og ellers har det hvert fald givet mig forståelse for hvorfor det ikke virker.