Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP

login fout

ottorocket - 13/08/2006 20:55

Onbekend

Ik probeer nu een login te maken voor mijn site, hij is nog niet af, maar ik zit al met een probleem

ik gebruik als naam: W0utah, ik heb geen paswoord ingevuld om te testen of hij wel de melding geeft dat het paswoord niet correct is.

deze error krijg ik altijd:

Citaat:

Er is iets fout in de gebruikte syntax bij '== 'W0utah'' in regel 1

php code - Bekijk de code zonder highlighting - Klap code in

<?php
if(isset($_POST['Login'])) {
	$info = mysql_query("SELECT * FROM leden WHERE id >= '1'") or die(mysql_error());
	$leden = mysql_fetch_assoc($info);
	if($_POST['naam'] == $leden['naam']){ //de naam voorkomt in de database
		$paswoord = mysql_query("SELECT paswoord FROM leden WHERE naam == '".$_POST['naam']."'") or die(mysql_error());
		if($_POST['paswoord'] != $paswoord['paswoord']) {
			echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
		};
	} else { //de naam komt niet voor in de database
		echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
	};
}
?>

<?php
if(isset($_POST['Login'])) {
	$info = mysql_query("SELECT * FROM leden WHERE id >= '1'") or die(mysql_error());
	$leden = mysql_fetch_assoc($info);
	if($_POST['naam'] == $leden['naam']){ //de naam voorkomt in de database
		$paswoord = mysql_query("SELECT paswoord FROM leden WHERE naam == '".$_POST['naam']."'") or die(mysql_error());
		if($_POST['paswoord'] != $paswoord['paswoord']) {
			echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
		};
	} else { //de naam komt niet voor in de database
		echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
	};
}
?>

als iemand de fout ziet zou het leuk zijn

17 antwoorden

Gesponsorde links

Simon - 13/08/2006 21:06
PHP expert		sql gebruikt = en niet == WHERE naam = '".$_POST['naam']."'" en niet WHERE naam == '".$_POST['naam']."'"

Wim - 13/08/2006 21:21
Crew algemeen		inderdaad Simon. nog een opmerking: een IF/ELSE constructie moet niet met een ; worden afgesloten. maw verwijder de ; op regel 9 en 12

ottorocket - 13/08/2006 21:35 (laatste wijziging 13/08/2006 21:51)

Onbekend

nu werkt het wel

2de vraag

nu is het paswoord nooit correct, ik weet niet of ik het paswoord dat in ingetypt heb moet doen met sha1 en md5 of het paswoord van in mijn database, ik denk dat ik het zo goed doe, ziet iemand de fout?

php code - Bekijk de code zonder highlighting - Klap code in

		$info = mysql_query("SELECT * FROM leden WHERE id >= '1'") or die(mysql_error());
		$leden = mysql_fetch_assoc($info);
		if($_POST['naam'] == $leden['naam']){ //de naam voorkomt in de database
			$paswoord = mysql_query("SELECT paswoord FROM leden WHERE naam = '".$_POST['naam']."'") or die(mysql_error());
			$pw_input = sha1(md5($_POST['paswoord']));
			if($pw_input != $paswoord['paswoord']) {
				echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
			} else {
				echo "<tr><td>Het paswoord is correct.</td></tr>";
			}

$info = mysql_query("SELECT * FROM leden WHERE id >= '1'") or die(mysql_error());
		$leden = mysql_fetch_assoc($info);
		if($_POST['naam'] == $leden['naam']){ //de naam voorkomt in de database
			$paswoord = mysql_query("SELECT paswoord FROM leden WHERE naam = '".$_POST['naam']."'") or die(mysql_error());
			$pw_input = sha1(md5($_POST['paswoord']));
			if($pw_input != $paswoord['paswoord']) {
				echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
			} else {
				echo "<tr><td>Het paswoord is correct.</td></tr>";
			}

roelh - 13/08/2006 21:53
PHP gevorderde		Ligt eraan. Heb je hem alleen MD5 in de DB gezet? Dan alleen met MD5 controleren ;)

ottorocket - 13/08/2006 21:55
Onbekend		ik heb hem in mijn db gezet als sha1(md5()) maar ik zet mijn ingetypt paswoord ook om, maar het is nooit correct

roelh - 13/08/2006 21:58
PHP gevorderde		Waar haal je pw_input vandaan? Snap weinig van je script zo op het eerste gezicht. Post hem helemaal eens op http://plaatscode.be

Simon - 13/08/2006 22:02 (laatste wijziging 13/08/2006 22:02)

PHP expert

otto, je hebt vergeten een mysql_fetch_* van je query vergeten

php code - Bekijk de code zonder highlighting - Klap code in

$paswoord = mysql_query("SELECT paswoord FROM leden WHERE naam = '".$_POST['naam']."'") or die(mysql_error());
      $pw_input = sha1(md5($_POST['paswoord']));

$paswoord = mysql_query("SELECT paswoord FROM leden WHERE naam = '".$_POST['naam']."'") or die(mysql_error());
      $pw_input = sha1(md5($_POST['paswoord']));

moet je veranderen naar:

php code - Bekijk de code zonder highlighting - Klap code in

$qPaswoord = mysql_query("SELECT paswoord FROM leden WHERE naam = '".$_POST['naam']."'") or die(mysql_error());
$paswoord = mysql_fetch_assoc($qPaswoord);
      $pw_input = sha1(md5($_POST['paswoord']));

$qPaswoord = mysql_query("SELECT paswoord FROM leden WHERE naam = '".$_POST['naam']."'") or die(mysql_error());
$paswoord = mysql_fetch_assoc($qPaswoord);
      $pw_input = sha1(md5($_POST['paswoord']));

ottorocket - 13/08/2006 22:05
Onbekend		@Simon, bedankt daar zat de fout nu werkt hij, weeral zo een domme fout

Simon - 13/08/2006 22:11

PHP expert

toch zou ik een iets andere scriptstijl aannemen

php code - Bekijk de code zonder highlighting - Klap code in

<?php
$qLid = mysql_query("SELECT COUNT(id), paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
if(mysql_result($qLid,0) > 0){ //de naam voorkomt in de database
	$aLid = mysql_fetch_assoc($qLid);
	if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
		echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
	} else {
		echo "<tr><td>Het paswoord is correct.</td></tr>";
	}
}
?>

<?php
$qLid = mysql_query("SELECT COUNT(id), paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
if(mysql_result($qLid,0) > 0){ //de naam voorkomt in de database
	$aLid = mysql_fetch_assoc($qLid);
	if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
		echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
	} else {
		echo "<tr><td>Het paswoord is correct.</td></tr>";
	}
}
?>

*niet getest*

ottorocket - 13/08/2006 23:01
Onbekend		maar waarom zou je tellen hoeveel er voorkomen, als er toch maar 1tje is, volgens mij blijft het hetzelfde als het op mijn manier is

Simon - 14/08/2006 09:15 (laatste wijziging 14/08/2006 09:16)
PHP expert		Ten eerste, mysql_result is sneller dan mysql_num_rows of in jouw geval assoc (:s). Ten tweede, zo selecteer je alleen de nodige rijen, wat ook weer sneller gaat. Ten derde, jij hebt 2 assoc's waar slecht 1 nodig is, allemaal tijdsverlies. Ten vierde, jouw script zal niet werken, want je fetcht slechts 1 rij uit je database, dus als je 10 leden hebt, kan er maar 1 inloggen, id 2 lijkt mij. Je zou dus moeten een while rond je eerste assoc zetten, wat ook alweer voor vertraging zorgt... redenen genoeg?

ottorocket - 14/08/2006 11:52 (laatste wijziging 14/08/2006 12:19)

Onbekend

jah, nu wel

EDIT: net getest en het werkt ook

EDIT2: na beter testen was er toch een fout
ik heb het nu zo, als een een naam intypt dat niet bestaat gaat alles goed, maar als je een bestaande naam gebruik loopt het mis

Citaat:

FOUT bij qLid :Het mixen van GROUP kolommen (MIN(),MAX(),COUNT()...) met no-GROUP kolommen is foutief indien er geen GROUP BY clausule is

php code - Bekijk de code zonder highlighting - Klap code in

<?php
if(isset($_POST['Login'])) {
	if(strlen($_POST['naam']) < 5) {
		echo "<tr><td>Je coachnaam moet minstens 5 letters lang zijn.</td></tr>";
	} elseif(strlen($_POST['paswoord']) < 5) {
		echo "<tr><td>Je paswoord moet minstens 5 letters lang zijn.</td></tr>";
	} else {
		$qLid = mysql_query("SELECT COUNT(id), paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
		if(mysql_result($qLid,0) > 0){ //de naam voorkomt in de database
  		$aLid = mysql_fetch_assoc($qLid);
  			if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
    				echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
  			} else {
    			echo "<tr><td>Het paswoord is correct.</td></tr>";
  			}
		} else { //de naam komt niet voor in de database
			echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
		}
	}
}
?>

<?php
if(isset($_POST['Login'])) {
	if(strlen($_POST['naam']) < 5) {
		echo "<tr><td>Je coachnaam moet minstens 5 letters lang zijn.</td></tr>";
	} elseif(strlen($_POST['paswoord']) < 5) {
		echo "<tr><td>Je paswoord moet minstens 5 letters lang zijn.</td></tr>";
	} else {
		$qLid = mysql_query("SELECT COUNT(id), paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
		if(mysql_result($qLid,0) > 0){ //de naam voorkomt in de database
  		$aLid = mysql_fetch_assoc($qLid);
  			if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
    				echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
  			} else {
    			echo "<tr><td>Het paswoord is correct.</td></tr>";
  			}
		} else { //de naam komt niet voor in de database
			echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
		}
	}
}
?>

Thomas - 14/08/2006 12:22
Moderator		Waarom haal je eerst alle ledengegevens op? Het is overigens niet gegarandeerd dat het eerste lid wat je ophaalt het lid is wat je zoekt... Waar is deze code eigenlijk voor bedoeld? Is het om iemand in te schrijven, van iemand de gegevens te updaten of om iemand in te loggen?! Haal gewoon de gegevens op van het lid waar je gegevens van wilt, en let op MySQL-injections...

ottorocket - 14/08/2006 13:29
Onbekend		fangorn, het scripts is bedoeld om in te loggen, ik haal toch enkel het paswoord op van het lid dat is ingetypt.

Rens - 14/08/2006 13:33
Crew algemeen		En waarom zou je dat doen? Wat dacht je er van om het ID op te halen van het lid? Dus dat je zo'n soort query krijgt: php code - Bekijk de code zonder highlighting - Klap code in "SELECT id FROM leden WHERE username='".MySQL_Escape_String($_POST['username'])."' AND password='".md5($_POST['password'])."'"; "SELECT id FROM leden WHERE username='".MySQL_Escape_String($_POST['username'])."' AND password='".md5($_POST['password'])."'"; Als er dan geen id word opgehaald bestaat het lid niet met die gebruikersnaam/wachtwoord combinatie...

Simon - 14/08/2006 16:29 (laatste wijziging 14/08/2006 16:29)

PHP expert

rensjuh, volgens mij wil hij verschillende foutmeldingen kunnen geven, niet tegelijk

php code - Bekijk de code zonder highlighting - Klap code in

<?php
if(isset($_POST['Login'])) {
  if(strlen($_POST['naam']) < 5) {
    echo "<tr><td>Je coachnaam moet minstens 5 letters lang zijn.</td></tr>";
  } elseif(strlen($_POST['paswoord']) < 5) {
    echo "<tr><td>Je paswoord moet minstens 5 letters lang zijn.</td></tr>";
  } else {
    $qLid = mysql_query("SELECT id, paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
    if(mysql_num_rows($qLid) > 0){ //de naam voorkomt in de database
      $aLid = mysql_fetch_assoc($qLid);
        if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
            echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
        } else {
          echo "<tr><td>Het paswoord is correct.</td></tr>";
        }
    } else { //de naam komt niet voor in de database
      echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
    }
  }
}
?>

<?php
if(isset($_POST['Login'])) {
  if(strlen($_POST['naam']) < 5) {
    echo "<tr><td>Je coachnaam moet minstens 5 letters lang zijn.</td></tr>";
  } elseif(strlen($_POST['paswoord']) < 5) {
    echo "<tr><td>Je paswoord moet minstens 5 letters lang zijn.</td></tr>";
  } else {
    $qLid = mysql_query("SELECT id, paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
    if(mysql_num_rows($qLid) > 0){ //de naam voorkomt in de database
      $aLid = mysql_fetch_assoc($qLid);
        if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
            echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
        } else {
          echo "<tr><td>Het paswoord is correct.</td></tr>";
        }
    } else { //de naam komt niet voor in de database
      echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
    }
  }
}
?>

ik heb je code aangepast via een num_rows ipv result. Ik weet niet hoe je het kan aanpassendat de result ook werkt, maar ik denk zoiets

php code - Bekijk de code zonder highlighting - Klap code in

<?php
if(isset($_POST['Login'])) {
  if(strlen($_POST['naam']) < 5) {
    echo "<tr><td>Je coachnaam moet minstens 5 letters lang zijn.</td></tr>";
  } elseif(strlen($_POST['paswoord']) < 5) {
    echo "<tr><td>Je paswoord moet minstens 5 letters lang zijn.</td></tr>";
  } else {
    $qLid = mysql_query("SELECT COUNT(id) as c, paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
    if(mysql_result($qLid,"c") > 0){ //de naam voorkomt in de database
      $aLid = mysql_fetch_assoc($qLid);
        if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
            echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
        } else {
          echo "<tr><td>Het paswoord is correct.</td></tr>";
        }
    } else { //de naam komt niet voor in de database
      echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
    }
  }
}
?>

<?php
if(isset($_POST['Login'])) {
  if(strlen($_POST['naam']) < 5) {
    echo "<tr><td>Je coachnaam moet minstens 5 letters lang zijn.</td></tr>";
  } elseif(strlen($_POST['paswoord']) < 5) {
    echo "<tr><td>Je paswoord moet minstens 5 letters lang zijn.</td></tr>";
  } else {
    $qLid = mysql_query("SELECT COUNT(id) as c, paswoord FROM leden WHERE naam = '" .$_POST['naam']. "'") or die("FOUT bij qLid :" .mysql_error());
    if(mysql_result($qLid,"c") > 0){ //de naam voorkomt in de database
      $aLid = mysql_fetch_assoc($qLid);
        if(sha1(md5($_POST['paswoord'])) != $aLid['paswoord']) {
            echo "<tr><td>Het paswoord komt niet overeen met de coachnaam.</td></tr>";
        } else {
          echo "<tr><td>Het paswoord is correct.</td></tr>";
        }
    } else { //de naam komt niet voor in de database
      echo "<tr><td>De ingevoerde coachnaam is niet bekend in onze database.</td></tr>";
    }
  }
}
?>

ottorocket - 14/08/2006 16:33
Onbekend		bedankt, maar ik had al een login van bij scripts gebruikt en deze wat verandert en die werkt

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten