Forum PHP.pl > [php][MySQL] Funkcje - gdzie zrobilem blad ? bo niewiem =/

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php][MySQL] Funkcje - gdzie zrobilem blad ? bo niewiem =/

dawhol

8.08.2006, 16:26:08

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

8.08.2006, 16:42:30

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

8.08.2006, 16:42:50

Cytat(dawhol @ 8.08.2006, 17:26 )

[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

dawhol

8.08.2006, 16:49:10

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

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

Cysiaczek

8.08.2006, 16:55:02

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

8.08.2006, 17:10:13

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

Wieviór

8.08.2006, 17:39:55

Cytat(dawhol @ 8.08.2006, 17:26 )

[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

8.08.2006, 18:13:28

~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

8.08.2006, 18:20:29

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

dawhol

8.08.2006, 18:24:02

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

8.08.2006, 18:29:11

Wyprintuj sobie to zapytanie,to zabaczysz dlaczego jest blad

Pozdrawiam

Wieviór

8.08.2006, 18:30:53

[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

8.08.2006, 18:45:58

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

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

Kicok

8.08.2006, 20:10:10

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

8.08.2006, 20:20:51

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

8.08.2006, 20:35:49

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

8.08.2006, 20:49:38

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

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.