[php][mysql] problem z kodem potwierdzajacym Opcje

[hondek]

mam taki kod:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['code_zam'])) {
	polacz();
	$cod = $_GET['code_zam'];
	$zapytanie = "SELECT * FROM zamowienia WHERE code = '".$cod."' ";
	$dzialaj = mysql_query($zapytanie);
	$row = mysql_num_rows($dzialaj);
	if($row) {
		$pole = mysql_fetch_assoc($dzialaj);
		$zapytanie = "UPDATE zamowienia SET code = '0' WHERE code = '".$cod."' ";
		mysql_query($zapytanie);
		$code_zam = "Aktywowane";
	} else {
		$code_zam = "Błędny kod";
	}
	rozlacz();
}
?>
[PHP] pobierz, plaintext 

I teraz, jeżeli kod znajduje się w bazie zmienia go na '0' i zmienna $code_zam przyjmuje "Błędny kod", gdzie jest problem?

[vtuner]

Znajdź:

[PHP] pobierz, plaintext 
<?php
if($row) {
?>
[PHP] pobierz, plaintext 

zamień na:

[PHP] pobierz, plaintext 
<?php
if($row > 0) {
?>
[PHP] pobierz, plaintext 

[zakręcony]

co robi mysql_num_rows();? zwraca ilość wierszy, tak? 0 też jest ilością, tak? więc if($row) jest zawsze spełnione talk?

rozwiązanie: if($row>0)

edit:
ktos mnie uprzedził, ale nie skasuję, bys wiedział dlaczego nie działało, tak na przyszłośc (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował zakręcony 27.03.2008, 20:14:20

[l0ud]

Ym... Na oko nie jest możliwe to co piszesz, czyli

jeżeli kod znajduje się w bazie zmienia go na '0' i zmienna $code_zam przyjmuje "Błędny kod"

za jednym wywołaniem. Sprawdź to dokładniej. Rada vtuner'a jest stosowna ale w tym przypadku raczej nic nie da...

btw.

[PHP] pobierz, plaintext 
<?php
$pole = mysql_fetch_assoc($dzialaj);
?>
[PHP] pobierz, plaintext 

a po co to?

[hondek]

Rozwiązałem to inaczej..

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['code_zam'])) {
	polacz();
	$cod = $_GET['code_zam'];
	$zapytanie = "SELECT * FROM zamowienia WHERE code = '".$cod."' ";
	$dzialaj = mysql_query($zapytanie);
	$row = mysql_num_rows($dzialaj);
	if($row) {
		$zapytanie = "UPDATE zamowienia SET code = '0' WHERE code = '".$cod."' ";
		mysql_query($zapytanie);
		rozlacz();
		header("Location: index.php?zamowienie_c=1");
	} else {
		rozlacz();
		header("Location: index.php?zamowienie_c=0");
	}
}
?>
[PHP] pobierz, plaintext 

To działa poprawnie, w tamtym kodzie nie warunek jest problemem... jakimś cudem kod wykonuje sie 2 razy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Bo UPDATE w bazie sie wykonuje (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) a zmienna $code_zam przyjmowała "Błędny kod" dziwne.....

l0ud -> ta linijka nie jest potrzbna (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) fakt (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował hondek 27.03.2008, 20:17:49

[l0ud]

Jeżeli wykonuje się dwukrotnie, to powiedzmy szczerze... nie jest dobrze. Pewnie i header wysyła się teraz 2 razy.
Proponuję szukać przyczyny takiego stanu rzeczy, może daj więcej kodu (który dołącza ten)?

[hondek]

to plik code_zam.php:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['code_zam'])) {
	polacz();
	$cod = $_GET['code_zam'];
	$zapytanie = "SELECT * FROM zamowienia WHERE code = '".$cod."' ";
	$dzialaj = mysql_query($zapytanie);
	$row = mysql_num_rows($dzialaj);
	if($row) {
		$pole = mysql_fetch_assoc($dzialaj);
		$zapytanie = "UPDATE zamowienia SET code = '0' WHERE code = '".$cod."' ";
		mysql_query($zapytanie);
		rozlacz();
		header("Location: index.php?zamowienie_c=1");
	} else {
		rozlacz();
		header("Location: index.php?zamowienie_c=0");
	}
}
?>
[PHP] pobierz, plaintext 

i ten plik includuje na samym początku w index.php :/

a tak btw. zakrecony :
Wydaje mi sie, ze nie masz racji:) if($zmienna) sprawdzy czy $zmienna jest true czy false, wartosci inne od 0 to true a 0 to false... wiec twoja teoria chyba nie jest poprawna.

Ten post edytował hondek 27.03.2008, 20:24:41

[l0ud]

Zakomentuj te headery i przed pierwszym warunkiem dodaj echo '1';
Jeżeli w wyniku otrzymasz 11, pokaż index.php...

[zakręcony]

@hondek, oczywiście masz rację, zakręciłem się (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[hondek]

@l0ud więc tak :

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['code_zam'])) {
	polacz();
	$cod = $_GET['code_zam'];
	$zapytanie = "SELECT * FROM zamowienia WHERE code = '".$cod."' ";
	$dzialaj = mysql_query($zapytanie);
	$row = mysql_num_rows($dzialaj);
	if($row) {
		$pole = mysql_fetch_assoc($dzialaj);
		$zapytanie = "UPDATE zamowienia SET code = '0' WHERE code = '".$cod."' ";
		mysql_query($zapytanie);
		rozlacz();
		echo "aktywowane";
				//$code_zam = "aktywowane";
		//header("Location: index.php?zamowienie_c=1");
	} else {
		rozlacz();
				echo "bledny ok";
		//$code_zam = "blędny kod";
		//header("Location: index.php?zamowienie_c=0");
	}
}
?>
[PHP] pobierz, plaintext 

to wykonuje się ok, natomiast jak odkomentuje $code_zam.... to jak daje echo $code_zam w niżej w index.php wyrzuca bledny kod (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[l0ud]

hondek, czyli co dokładnie zwrócił powyższy kod?

[hondek]

zauważylem pewną zależność:
jeżeli kod jest taki:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['code_zam'])) {
	polacz();
	$cod = $_GET['code_zam'];
	$zapytanie = "SELECT * FROM zamowienia WHERE code = '".$cod."' ";
	$dzialaj = mysql_query($zapytanie);
	$row = mysql_num_rows($dzialaj);
	if($row) {
		$pole = mysql_fetch_assoc($dzialaj);
		$zapytanie = "UPDATE zamowienia SET code = '0' WHERE code = '".$cod."' ";
		mysql_query($zapytanie);
		$code_zam = "Aktywowane";
	} else {
		$code_zam = "Błędny kod";
	}
	rozlacz();
}
?>
[PHP] pobierz, plaintext 

działa źle, ale wystarczy ze rozlacze sie z bazą w if i w else... i dziala ok..

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['code_zam'])) {
	polacz();
	$cod = $_GET['code_zam'];
	$zapytanie = "SELECT * FROM zamowienia WHERE code = '".$cod."' ";
	$dzialaj = mysql_query($zapytanie);
	$row = mysql_num_rows($dzialaj);
	if($row) {
		$pole = mysql_fetch_assoc($dzialaj);
		$zapytanie = "UPDATE zamowienia SET code = '0' WHERE code = '".$cod."' ";
		mysql_query($zapytanie);
		rozlacz();		
		$code_zam = "Aktywowane";
	} else {
		rozlacz();
		$code_zam = "Błędny kod";
	}
}
?>
[PHP] pobierz, plaintext 

i to działa ok.... ale dlaczego...?

[l0ud]

hondek, pokaż jak deklarujesz tą funkcję rozlacz();

[hondek]

rzeczywiscie... bardzo mozliwe ze w niej tkwi problem... oto te funkcje :

[PHP] pobierz, plaintext 
<?php
function polacz() {
	require_once "tajne/setup.inc";
	@mysql_connect(host,dblogin,dbhaslo) or die("Nie mozna polaczyc sie z bazz danych");	
	@mysql_select_db(db) or die("Nie mozna otworzyc bazy danych");	
}
function rozlacz() {
	@mysql_close();
}
?>
[PHP] pobierz, plaintext 

[zakręcony]

jak odnosisz się do code_zam.php, nie jest to w jakiejś pętli?

[hondek]

@zakrecony
Aż tak bym się w kulki nie zrobił (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[l0ud]

Ponawiam prośbę o test z echo '1'; na samym początku skryptu. Jaki wynik generuje?