[php][MySQL] Funkcje - gdzie zrobilem blad ? bo niewiem =/ Opcje

[dawhol]

A wiec tak jest juz to moja trzecia funkcja jaka napisałem hehe moze bylo ich nei wiele ale z pierwszymi dwoma nie mialem problemu natomiast z ta mam niewielkie.

Jest to plik akcja.php z mojej ksiegi gosci (panelu admina)
1) jezeli w adresie dam akcja.php?gbook=edit&id=1 to bede mógł edytowac 1 wpis
2) jezeli w adresie dam akcja.php?gbook=del&id=1 to będę mógł usunac wpis o numerze 1
3 jezeli w adresie dam akcja.php?gbook=save&id=1 to bede mógł zapisac wpis 1 po edycji.

No i problem jest taki ze jezeli robie edytujk to on mi najpierw usuwa ;/ wpis a pozniej chce go edytowac mimo ze ma funkcje warunkowo w której usunac moze tylko wtedy gdy $gbook jest równa del ;/

Prosze o pomoc w znalezieniu błedu bo szukam juz od prawie godziny =/

[PHP] pobierz, plaintext 
<?php
$gbook = $_GET['gbook'];
 
 
function gbook () 
{ $id = $_GET['id'];
 
  if ($gbook = 'del' OR $gbook = 'edit' OR $gbook = 'save')
  {
	if ($gbook = 'del') 
	{
	  $zapytanie = "DELETE FROM cms_gbook WHERE id=".$id."";
	  $wykonaj = mysql_query($zapytanie);
	 
	  if ($wykonaj) 
	  {
		echo ('Dane zostały skasowane');
	  };
	};
 
	if ($gbook = 'edit')
	{
	  $zapytanie2 = "SELECT * FROM cms_gbook WHERE id=".$id."";
	  $wykonaj2 = mysql_query($zapytanie2);
	  $dane = mysql_fetch_array($wykonaj2);
 
	  echo ('<form action="akcja.php?gbook=save&id='.$id.'" method="post"><br/><br/>');
	  echo ('<textarea rows="30" cols="40" name="wpis">'.$dane['wpis'].'</textarea>');
	  echo ('<input type="submit" name="send" value="Zapisz"></form>'); 
 
	};
 
	if ($gbook = 'save')
	{
	  $zapytanie3 = "UPDATE cms_gbook SET wpis=".$wpis." WHERE id=".$id."";
	  $wykonaj3 = mysql_query($zapytanie3);
 
	  if ($wykonaj3) 
	  {
		echo ('Dane zostały zapisane');
	  };
	};
  }
  else echo mysql_error();
}
 
 
if (isset($gbook)) gbook();
?>
[PHP] pobierz, plaintext 

PS. Skrypt jest prototypem dlatego nie zabezpieczyłem go jescze przed atakami SQL ale zrobie to jak rozwiarze wpierw ten problem co mam .

[Cysiaczek]

Znasz może różnice pomiędzy operatorem przypisania, a operatorem porównania?
Może jak poznasz, to poprawisz skrypt i będzie działał

Pozdrawiam

[-Wieviór-]

[PHP] pobierz, plaintext 
<?php
if ($gbook = 'del') 
	{	};
php?>
[PHP] pobierz, plaintext 

Tutaj robisz if'a z warunkiem przypisania $gbook wartości del.

Poprawny kod:

[PHP] pobierz, plaintext 
<?php
if ($gbook == 'del') 
	{	};
php?>
[PHP] pobierz, plaintext 

Ten post edytował Wieviór 8.08.2006, 16:43:22

[dawhol]

mialem z dwoma == i mi tez nie działało ;/

tzn jak mam dwa to nic sie nie dzieje ani nie wyswietla.

Ten post edytował dawhol 8.08.2006, 16:52:09

[Cysiaczek]

A wszędzie dajesz == ? Bo poza tym, to kod wygląda normalnie (powinien działać). Może czegoś brakuje? Jakiejś zmiennej? Sprawdzas, czy istnieje $id ?

[dawhol]

wszedzie a id istnieje wpis w bazie jest ;/ naprawde sam sie zastanawiam przez godzine prawie szukalem bledu i zadnego nie znalazłem i jak mowie próbowałem z == i z = bo i nic ;/ jak nie znajde bl;edu to szkoda napisze jescze raz ale juz troche inaczej bo mam nawet pomysł jak, choc wolał bym jak by mi ktos wskazał ten bład by bylo mniej pracy

EDIT:

dobra jeden problem naprawilem ale pojawi lsie drugi

problem mialem poniewaz zmienna $gbook = $_GET['gbook']; byla poza fukncja musiałem ja tez tam wlozyc i prawie wszystko zadziałało oprócz zapisu ale jescze sie z nim pobawie moze znajde swój bład

Ten post edytował dawhol 8.08.2006, 17:10:36

[-Wieviór-]

[PHP] pobierz, plaintext 
<?php
	  $zapytanie3 = "UPDATE cms_gbook SET wpis=".$wpis." WHERE id=".$id."";
	  $wykonaj3 = mysql_query($zapytanie3);
?>
[PHP] pobierz, plaintext 

Zmień ten kod na:

[PHP] pobierz, plaintext 
<?php
	  $zapytanie3 = "UPDATE cms_gbook SET wpis=".$wpis." WHERE id=".$id."";
	  mysql_query($zapytanie3);
?>
[PHP] pobierz, plaintext 

[dawhol]

~Wieviór
Co do tego co kazałes zmienic nic to nie da i nie dalo bo sprawdzałem jedyne co siezmieni to w razie wykonania nie zostane poinformowany o tym ze wpis został zmieniony,

[-Wieviór-]

To może daj jescze or mysql error, żeby pokazał jaki jest błąd

[dawhol]

dalem ale zapomnialem o tym napisac bo z lekka sie dzisiaj zle czuje o to odpowiedz::

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE id=1' at line 1

niby cos z zapytaniem sql ale zastanawia mnie co ;/

[Cysiaczek]

Wyprintuj sobie to zapytanie,to zabaczysz dlaczego jest blad

Pozdrawiam

[-Wieviór-]

[PHP] pobierz, plaintext 
<?php
$zapytanie3 = "UPDATE cms_gbook SET wpis='".$wpis."' WHERE id=$id";
mysql_query($zapytanie3);
?>
[PHP] pobierz, plaintext 

Po prostu kwestia cudzysłów itp. itd. Zobacz tak

[dawhol]

działa kurde zapomnialem o glupich cudzysłowiach ;/

ale tylko teoretycznie ;/ bo do bazy danych zapisuje puste pole ;/

[Kicok]

No a co ma zapisywać jak nie puste pole? Przypisałeś gdziekolwiek w tym kodzie, który podałeś u góry jakąkolwiek wartość do zmiennej $wpis?

[dawhol]

tak jezeli wcisnie sie zapisz to jescze raz zostanie przeladowana strona i z pola textarea name=wpis zostanie wyciagnieta zmienna poniewaz mam register globals on ale pozniej to poprawie ale to nie o to chodzi bo ani tak ani tak nie aktualizuje pola tylko je czysci ;/

[Kicok]

Nawet jeśli masz register_globals = On, to ta zmienna $wpis ci sam do wnętrza funkcji nie wlezie. Jak już tak chcesz to zapisać, to dodaj na początku swojej funkcji:

[PHP] pobierz, plaintext 
<?php
global $wpis;
?>
[PHP] pobierz, plaintext 

[dawhol]

Dzięki teraz wszystko pięknie działa mój bład ale wkoncu kazdy sie nia nich uczy myslałem ze on mi ja tam wciagnie . Wielkie dzieki