[php] /[mysql] ilosc przetworzonych rekordow, w funkcji subskrypcji Opcje

[kossak]

Witajcie.

Nie mam juz sily z ta funkcja. Prosze o pomoc. Stworzylem funkcje, ktora ma za zadanie dodawac/usuwac adresy mail z bazy. Problem w tym, ze dodaje adrezsy poprawne, nie sprawdzajac czy jest takowy w bazie;/
Oto kod:

[PHP] pobierz, plaintext 
function subskrypcja()
{
	if(empty($_POST['email']))
	{
?>
	Jesli chcesz byc powiadamiany o nowosciach na stronie, wpisz sie na nasza liste 
subskrypcji.
	<form action=index.php method=POST>
	E-Mail:<input name=email type=text><br>
	Akcja:<select name=akcja>
<option>Zapisz</option>
<option>Wypisz</option>
</select>
<br>
<input type=submit value=Wyslij>
</form>
<?
	}
	else
	{
		$email=mysql_escape_string($_POST['email']);
		if(!sprawdz_mail($_POST['email']))
		{
		echo "Nieprawidlowy adres.";
		}
		else
		{
		$sql3="SELECT `email` FROM `sobskrypcja` WHERE `email`='".$email."'";
		$wynik3=mysql_query($sql3);
		$liczba=mysql_num_rows($wynik3);				 //linia 124
		if($liczba > '0')
		{
			echo "Adres wuidnieje juz w bazie.";
		}
		else
		{
			if($_POST['akcja']=="Zapisz")
			{
				$sql2="INSERT INTO `subskrypcja`(`id`,`email`) VALUES(NULL,'".$email."')";
				$wynik2=mysql_query($sql2);
				if(!$wynik2)
				{
					echo "Blad przy dodawaniu danych.";
				}
				else
				{
				echo "Adres zostal dodany.";
				}
			}
			if($_POST['akcja']=="Wypisz")
			{
				$sql2="DELETE FROM `subskrypcja` WHERE `email`='".$email."'";
				$wynik2=mysql_query($sql2);
				if(!$wynik2)
				{
				echo "Blad przy usuwaniu danych.";
				}
				else
				{
				echo "Pomyslnie usunieto adres.";
				}
			}
		}
 
		}
	}
 
 
}
function sprawdz_mail($adres)
{
	if(ereg('^[a-zA-Z0-9_.-]+@[a-zA-Z0-9-]+.[a-zA-Z0-9-.]+$',$adres))
	{
	return true;
	}
	else
	{
	return false;
	}
}
[PHP] pobierz, plaintext 

Wywala:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/sawak/public_html/test/funkcje.php on line 124

pomocy (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Ten post edytował kossak 9.02.2007, 15:19:19

[nospor]

[PHP] pobierz, plaintext 
<?php
$liczba=mysql_num_rows($wynik3);
?>
[PHP] pobierz, plaintext 

pobierasz liczbe nie sprawdzajac wogole czy ci sie zapytanie powiodlo...
.... a powiem ci ze sie nie powiodlo.
Miedzy innymi dlatego: `sobskrypcja` . widzisz literowke?

[kossak]

Faktycznie chodzilo o literowke:) Dzieki za pomoc.
O taki banal skrypt nie dzialal... heh