Vanheftur fyrir 13 árum, 5 mánuðum Fjöldi lestra: 424

if (mysql_num_rows($result) == 0)

Ég fær þessa villu hér á seinni if-inu

Warning: mysql_num_rows() expects parameter 1 to be resource, boolean given in

	if($_POST['username'] == "" || $_POST['password'] == "")
	{
	echo 'Verður að slá inn bæði notendanafn og lykilorð';
	}
	else 
	{
		$query = ('SELECT userid FROM users WHERE username='.$_POST['username'].' AND password='.$_POST['password'].'');
		$result = mysql_query($query);
		if (mysql_num_rows($result) == 0)
		{
		 echo "Innskraing tokst ekki ";
		}
	}
}

wolfy fyrir 13 árum, 5 mánuðum

Fyrirspurn þín í grunninn er vitlaus. Þ.e. þú ert ekki að fá neitt resource til baka.

Prufaðu að prenta út fyrirspurnina og keyra hana sjálfur og athugaðu hvort þú fáir einhverjar villur.

Good luck!

Kv, wolfy.

Fyrra álit

wolfy fyrir 13 árum, 5 mánuðum

UPDATE: Getur ekki verið að það vanti single quote utanum strenginn?

$query = sprintf("SELECT userid FROM users WHERE username='%s' AND password='%s'",
    mysql_real_escape_string($_POST['username']),
    mysql_real_escape_string($_POST['password']));

Kv, wolfy.

Fyrra álit

Vanheftur fyrir 13 árum, 5 mánuðum

Þegar ég bætir or die við result þá kemur tekur hann við fyrirspurninni eða þá leitar í gagnagrunninum og segir til hvort hann finni eða ekki.

$result = mysql_query($query) or die ($query);

Já, Ég var búinn bæta þessum kóða hér.

		$vpassword = $_POST['password'];
		$vusername = $_POST['username'];
		$vusername = stripslashes($vusername);
		$vpassword = stripslashes($vpassword);
		$vusername = mysql_real_escape_string($vusername);
		$vpassword = mysql_real_escape_string($vpassword);

Allur kóðinn eins og hann lítur út núna.

<?php 
require('../inc/global.php');
if ( $_GET['action'] == 'logincheck' ) 
{
		$vpassword = $_POST['password'];
		$vusername = $_POST['username'];
		$vusername = stripslashes($vusername);
		$vpassword = stripslashes($vpassword);
		$vusername = mysql_real_escape_string($vusername);
		$vpassword = mysql_real_escape_string($vpassword);
	if($vusername == "" || $vpassword  == "")
	{
	echo 'Verður að slá inn bæði notendanafn og lykilorð';
	
	}
	else 
	{
		
		$query = 'SELECT userid FROM users WHERE username='.$vusername.' AND password='.$vpassword.'';
		$result = mysql_query($query) ;
		$count = mysql_num_rows($result);
		if ($count == 0)
		{
		 echo "Innskraing tokst ekki ";
		}
		else
		{
		echo "skradur inn?";
		}
	}
}
print '
<form action="?action=logincheck" method="post" name="loginform">
Username
<input name="username" type="text">
Password
<input name="password" type="password">
<input name="sendlogin" type="submit" value="Login">
</form>
';

Og svo inc skjalið.

$server = 'localhost';
$username = 'xxx';
$password = 'xxx';
$database = 'xxx';
$connect = mysql_connect($server, $username, $password) or die ('Connection to mysql Error'. mysql_error);
mysql_select_db ($database, $connect) or die ('Can not connect database'. mysql_error);

Fyrra álit

wolfy fyrir 13 árum, 5 mánuðum

Þú hefur ekki enn lagað það sem ég benti á.

Ef þú sendir þessa fyrirspurn:

SELECT userid FROM users WHERE username=test AND password=test

þá færðu villu.

Þú verður að hafa single quote utanum strenginn í fyrirspurninni svo hún sé lögleg.

Dæmi:

SELECT userid FROM users WHERE username='test' AND password='test'

Það vantar hjá þér.

Kv, wolfy.

Fyrra álit

Vanheftur fyrir 13 árum, 5 mánuðum

Þræl virkar núna.

Áttaði mig ekki strax á mysql_real_escape_string setninguna.

Þakka ;)

Fyrra álit

egth fyrir 13 árum, 5 mánuðum

$dbh = new mysqli('localhost', 'root', 'toor', 'database');
if(!empty($_POST['username']) || !empty($_POST['password']))
{
    #ættir að vera að nota prepared statements :<<<
    #:<
    $username = $dbh->real_escape_string($_POST['username']);
    $password = $dbh->real_escape_string($_POST['password']);
    
    $query = "SELECT userid AS count FROM users WHERE username = '$username' AND password = '$password'";
    $result = $dbh->query($query);
    #Myndi reyndar hafa fail partinn inní <else>
    if($result->num_rows == 0)
    {
        #tókst ekki að gera stuffies
    }

Annars til að spara geðheilsuna þá myndi ég aldrei vera með gæsalappa stúss inní query. Ef þú endilega vilt það vertu með það í skipulögðu formi.

Fyrra álit

egth fyrir 13 árum, 5 mánuðum

$query ætti að vera: $query = “SELECT userid FROM users WHERE username = ‘$username’ AND password = ‘$password’”;

“userid AS count” Var farinn að hugsa um að nota count(*) as count þarna, en fattaði svo að ég var kominn vel út fyrir kassann :p

Fyrra álit

Vefsíðugerð

Vefsíðugerð

Ofurhugar

Stjórnendur

Hverjir eru inni

Korkar - Almennt

if (mysql_num_rows($result) == 0)

Dýr

Dægurmál

Fræði

Leikir

Lífið

Lífsstíll

Margmiðlun

Menning

Skjár

Sport

Tónlist

Tækni

Vélar

Hugi