Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [php][mysql]panel administracyjny, usuwanie nie tego zawodnika co bym chciał
Oscar_83
post 20.05.2007, 23:06:44
Post #1





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

Mam problem ponieważ gdy klikam na jakiegoś zawodnika to usuwa mi z najniższym id, wiem że coś musiałem namieszać z przekazywaniem jakie id ma być skasowane, lecz nie mogę dojśc co jest nie tak, prosiłbym o pomoc:
index.php
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. $_SESSION['username'] = $_POST['user'];
  4. $_SESSION['userpass'] = $_POST['pass'];
  5. $_SESSION['authuser'] = 0;
  6. //sprawdź użytkownika i hasło
  7. if (($_SESSION['username'] == '***') and
  8.    ($_SESSION['userpass'] == '***')) {
  9.    $_SESSION['authuser'] = 1;
  10.    } else {
  11.    echo "Przepraszam, ale nie masz uprawień do tej strony!";
  12.    exit();
  13.    }
  14.  
  15. $polaczenie = mysql_connect("localhost", "root", "krasnal") or die("Sprawdź połączenie z serwerem");
  16. 			  mysql_select_db('tabele',$polaczenie) or die (mysql_error());
  17. mysql_query('SET NAMES latin2');
  18. ?>
  19. <html>
  20. <head>
  21. <title>Baza zawodników</title>
  22. <style type="text/css">
  23. TD{color:#353535;font-family:verdana"}
  24. TH{color:#FFFFFF;font-family:verdana;background-color:#336699}
  25. </style>
  26. </head>
  27. <body>
  28. <table border="0" width="600" cellspacing="1" cellpadding="3" bgcolor="#353535" align="center">
  29. 	   <tr>
  30. 		   <td bgcolor="#FFFFFF" colspan="2" align="center">
  31. 			   Zawodnicy <a href="zawodnicy2.php?action=add&id=">[DODAJ]</a>
  32. 		   </td>
  33. 	   </tr>
  34. <?php
  35. //pobieranie zawodników do edycji
  36. 	 $zawodsql = "SELECT * FROM zawodnicy";
  37. 	 $result = mysql_query($zawodsql)
  38. 	 or die ("Błędne zapytanie: " . mysql_error ());
  39. 	 while ($row = mysql_fetch_array($result)){
  40. ?>
  41. 	   <tr>
  42. 		   <td bgcolor="#FFFFFF" width="50%">
  43. 			   <?php echo $row['imie'];
  44. 					 echo "&nbsp";
  45. 					 echo $row['nazwisko']; ?>
  46. 		   </td>
  47. 		   <td bgcolor="#FFFFFF" width="50%" align="right">
  48. 			   <a href="zawodnicy2.php?action=edit&id=<?php
  49. 				  echo $row['id'];?>">[EDYTUJ]</a>
  50. 			   <a href="delete.php?type=zawodnicy&id=<?php
  51. 				  echo $row['id'];?>">[USUŃ]</a>
  52. 		   </td>
  53. 	   </tr>
  54. 	   <?php
  55. 	   }
  56. 	   ?>
  57. </table>
  58. </body>
  59. </html>
[PHP] pobierz, plaintext

i plik delete.php
[PHP] pobierz, plaintext 
  1. <?php
  2. $polaczenie = mysql_connect("localhost", "root", "krasnal")
  3. or die ("Nie udało się połączyć: " . mysql_error());
  4. mysql_select_db('tabele',$polaczenie)
  5. or die (mysql_error());
  6. //zmiana nazw type na odpowiednie teksty
  7. $convert['zawodnicy']= 'zawodnika';
  8. //skrypt usuwania
  9. 		 if(!isset($_GET['do']) || $_GET['do'] !=1) {
  10. ?>
  11. 		 <p align="center" style="color:#FF0000">
  12. 			Czy jesteś pewien, że chcesz usunąć <?php
  13. 			echo $convert[$_GET['type']]; ?>?<br>
  14. 			<a href="<?php echo $_SERVER['REQUEST_URI']; ?>&do=1">tak</a>
  15. 			lub <a href="index.php">lista</a>
  16. 		 </p>
  17. <?php
  18. } else
  19. {
  20. 	  //wygenerowanie polecenia SQL
  21. 	   $sql = "DELETE FROM " . $_GET['type'] . "
  22. 			  WHERE 'id' = 'id'
  23. 			  LIMIT 1";
  24. 	  $result = mysql_query($sql)
  25. 	  or die ("Niepoprawne zapytanie: " . mysql_error());
  26. ?>
  27. 		 <p align="center" style="color:#FF0000">
  28. 		 Właśnie usunąłeś <?php echo $convert[$_GET['type']]; ?>
  29. 		 <a href="index.php">Lista</a>
  30. 		 </p>
  31. <?php
  32. }
  33. ?>
[PHP] pobierz, plaintext


Czy ktoś może wie co ja tu namieszałem questionmark.gif
I jeszcze jedno, jak zapamiętać to moje hasło żeby po kliknięciu na końcu na "Lista" przeniosło mnie do stronki index.php (bo w chwili obecnej tylko wrzuca że nie mam dostępu (czyli nie pamięta nazwy użytkownika i hasła)

PS: dodam jeszcze że moja tabela z zawodnikami nazywa się "zawodnicy" i kolumna pierwsza "id" primary key

wydaje mi się że błąd był tu:
[PHP] pobierz, plaintext 
  1. <?php
  2. //wygenerowanie polecenia SQL
  3. 	   $sql = "DELETE FROM " . $_GET['type'] . "
  4. 			  WHERE 'id' = 'id'
  5. 			  LIMIT 1";
  6. 	  $result = mysql_query($sql)
  7. 	  or die ("Niepoprawne zapytanie: " . mysql_error());
  8. ?>
[PHP] pobierz, plaintext

troche zmieniłem ale dalej nie działa poprawnie :/
teraz mam coś takiego:
[PHP] pobierz, plaintext 
  1. <?php
  2. $sql = "DELETE FROM " . $_GET['type'] . "
  3. 			  WHERE " . $_GET['type'] . "= '" . $_GET['id']."'
  4. 			  LIMIT 1";
  5. ?>
[PHP] pobierz, plaintext

wyskakuje błąd:
Niepoprawne zapytanie: Nieznana kolumna 'zawodnicy' w where clause czyli zapytanie niedobre - faktycznie bo nie mam kolumny zawodnicy tylko tabela nazywa się zawodnicy a kolumna jest id, ale jak to zmienić żeby podawało takie id jak ja klikam??

Ten post edytował Oscar_83 20.05.2007, 23:22:56
Go to the top of the page
+Quote Post
barat
post 20.05.2007, 23:22:12
Post #2





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
-----

Chyba będzie tak smile.gif

[PHP] pobierz, plaintext 
  1. <?php
  2. $sql = "DELETE FROM " . $_GET['type'] . "
  3. 			  WHERE id =".$_GET['id']."
  4. 			  LIMIT 1";
  5. ?>
[PHP] pobierz, plaintext


Ten post edytował barat 20.05.2007, 23:23:13
Go to the top of the page
+Quote Post
Oscar_83
post 20.05.2007, 23:27:49
Post #3





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

hehe no i super smile.gif gra i buczy smile.gif)) dzięki bardzo!!! smile.gif a jakie oczywiste się wydaje smile.gif
teraz jest super smile.gif jeszcze tylko:
jak zapamiętać to moje hasło żeby po kliknięciu na końcu na "Lista" przeniosło mnie do stronki index.php (bo w chwili obecnej tylko wrzuca że nie mam dostępu (czyli nie pamięta nazwy użytkownika i hasła)

Ten post edytował Oscar_83 20.05.2007, 23:30:53
Go to the top of the page
+Quote Post
barat
post 20.05.2007, 23:47:05
Post #4





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
-----

W każdym pliku który będzie korzystał z Sesji, lub je tylko potem przekazywał dalej musisz na górze wkleić

session_start ();

Więcej - w pliku Delete powinieneś też sprawdzać czy użytkownik jest zalogowany, bo w obecnej chwili każdy może sobie wejść w plik delete.php i kasować co mu się podoba smile.gif
Wystarczy sprawdzać warunek, czy jest ustawiony użytkownik smile.gif

Ten post edytował barat 21.05.2007, 08:46:41
Go to the top of the page
+Quote Post
Oscar_83
post 21.05.2007, 16:09:04
Post #5





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

Ustawiłem tak jak mi podpowiadałeś wszędzie session_start (); ale i tak potem jak klikam żeby powrócił do listy to i tak klika że nie mam dostępu.
PS: jakim warunkiem sprawdza się czy jest ktoś zalogowany? bo faktycznie można wejść bez logowania np do pliku delete.

Ten post edytował Oscar_83 21.05.2007, 16:12:05
Go to the top of the page
+Quote Post
barat
post 21.05.2007, 17:03:38
Post #6





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
-----

Obecnie masz spory bład ...

Najpierw ustawiasz username i userpass zgodne z $_POST i authuser na 0

Więc jak wracasz ze swojego skryptu delete, ustawiasz znowu sesje username i userpass, a ponieważ nie ma już danych $_Post to te sesje ustawiasz na puste więc nie ma sily by Cie zalogowało smile.gif

Musisz to przerobić ... najpierw sprawdzasz czy są dane $_POST i jeśli są to przypisujesz im dane username i userpas (pierwsza instrukcja if)
Potem sprawdzasz czy SESSION userpass i username istnieją (isset($_SESSION['userpass/username'])) i odpowiadają wzorcom i ustawiasz Authuser na 1 jeśli tak, a Authuser 0 jeśli nie (instrukcja if - else)
Na końcu sprawdzasz czy istnieje sesja Authuser i jeśli wynosi 1 to nic nie robisz:

if (isset($_SESSION['authuser']) && $_SESSION['authuser'] == 1) {
// nic nie rób
} else {

Przekierowujesz na stronę błędu lub wywalasz komunikat o błędzie i kończysz skrypt)

W pozostałych stronach sprawdzasz warunek ten co wyżej , możesz go wkleić do każdej strony (sprawdza czy jest authuser )

Ten post edytował barat 21.05.2007, 19:48:15
Go to the top of the page
+Quote Post
Oscar_83
post 21.05.2007, 20:54:05
Post #7





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

Hmm pisałem to wszystko na podstawie książki - robiłem to 1 raz,
nie wiem jak się sprawdza czy są dane $_POST, ale faktycznie tak jak mówisz pierw ustawiam tu u siebie autuhser na 0, czyli faktycznie nie może mi to działać.

Do sprawdzania czy użytkownik się zalogował znalazłem w książce kod:
[PHP] pobierz, plaintext 
  1. <?php
  2. if ($_SESSION['authuser'] !=1 {
  3.    echo "Przepraszamy ale nie masz uprawnień do zobaczenia tej strony!";
  4.    exit ();
  5. }
  6. ?>
[PHP] pobierz, plaintext

ten kod chyba podobnie działa co i ten co Ty mi podałeś, powrzucam go.

na moje to ten pierwszy if to powinno być coś takiego:
[PHP] pobierz, plaintext 
  1. <?php
  2. if (($_POST['user'] !=0) and
  3.  ($_POST['pass'] !=0)) 
  4. {$_SESSION['username'] == $_POST['user']
  5. $_SESSION['userpass'] == $_POST['pass']
  6. }
  7. else {
  8. ?>
[PHP] pobierz, plaintext

czy to coś takiego będzie?? dobrze kombinuje?? przerobione troszke
i tak samo do drugiego ifa
oki próbuje jak to będzie działało

Ten post edytował Oscar_83 21.05.2007, 21:14:21
Go to the top of the page
+Quote Post
barat
post 21.05.2007, 21:35:24
Post #8





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
-----

Wystarczy

!$_POST['user'] , !$_POST['pass'] czyli wykrzykniki przed ... nie masz sprawdzać czy coś jest różne od zera tylko czy te zmienne w ogóle istnieją smile.gif


Dobrze kombinujesz smile.gif
Go to the top of the page
+Quote Post
Oscar_83
post 21.05.2007, 21:37:19
Post #9





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

teraz to już wogóle namieszałem tak że szok, jak zacząłem kombinować to na moje normalne hasło nie chce wchodzić, zakręciłem się mocno w tych ifach, co ja natworzyłem tu:
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. if (!$_POST['user'] and !$_POST['pass'])
  4. {$_SESSION['username'] == $_POST['user'];
  5. $_SESSION['userpass'] == $_POST['pass'];
  6. }
  7. if (isset($_SESSION['userpass/username'])) {
  8.    $_SESSION['authuser'] = 1;
  9. }
  10.  else {
  11. 	 $_SESSION['authuser'] = 0;
  12. 	 }
  13. 	 
  14.  if (($_SESSION['username'] == '***') and
  15.    ($_SESSION['userpass'] == '***')) {
  16.    $_SESSION['authuser'] = 1;
  17.    } else {
  18.    echo "Przepraszam, ale nie masz uprawień do tej strony!";
  19.    exit();
  20.    }
  21.  if (isset($_SESSION['authuser']) && $_SESSION['authuser'] == 1) {
  22. // nic nie rób
  23. } else {
  24. 	echo "Przepraszam, ale nie masz uprawień do tej strony!";
  25.    exit();
  26.    }
  27. ?>
[PHP] pobierz, plaintext

nie wiem czy chociaż kawałek tego będzie dobry :/

Ten post edytował Oscar_83 21.05.2007, 21:41:20
Go to the top of the page
+Quote Post
barat
post 21.05.2007, 21:46:28
Post #10





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
----- 

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. if ($_POST['user']) and
  4.  ($_POST['pass'])
  5. {$_SESSION['username'] == $_POST['user'];
  6. $_SESSION['userpass'] == $_POST['pass'];
  7. }
  8. if (isset($_SESSION['username'] && $_SESSION['userpass'])) {
  9. if($_SESSION['username'] == "zuku" && $_SESSION['userpass'] == "27k11") {
  10.    $_SESSION['authuser'] = 1;} else {$_SESSION['authuser'] = 0;}
  11. }
  12.  else {
  13. 	 $_SESSION['authuser'] = 0;
  14. 	 }
  15. 	 
  16.  
  17.  if (isset($_SESSION['authuser']) && $_SESSION['authuser'] == 1) {
  18. // nic nie rób
  19. } else {
  20. 	echo "Przepraszam, ale nie masz uprawień do tej strony!";
  21.    exit();
  22.    }
  23. ?>
[PHP] pobierz, plaintext


Mogą być literówki smile.gif

Ten post edytował barat 21.05.2007, 21:48:37
Go to the top of the page
+Quote Post
Oscar_83
post 21.05.2007, 21:51:37
Post #11





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

rozumiem że w 3 i 4 linii przed $_POST powinny być wykrzykniki tak?
wielkie dzięki za przeróbkę smile.gif
Go to the top of the page
+Quote Post
barat
post 21.05.2007, 21:54:23
Post #12





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
-----

Nie powinno być wykrzykników smile.gif

Tu sprawdzamy, czy $_POST zawierają dane (czyli bez wykrzykników) i jeśli tak , ustawiamy je w sesji
Go to the top of the page
+Quote Post
Oscar_83
post 21.05.2007, 22:16:49
Post #13





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

troszeczke zmieniłem bo brakowało w 2 miejscach nawiasów no i teraz jest coś takiego:
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. if (($_POST['user']) and ($_POST['pass']))
  4. {$_SESSION['username'] == $_POST['user'];
  5. $_SESSION['userpass'] == $_POST['pass'];
  6. }
  7. if (isset($_SESSION['username']) && ($_SESSION['userpass'])) {
  8. if($_SESSION['username'] == "zuku" && $_SESSION['userpass'] == "27k11") {
  9.    $_SESSION['authuser'] = 1;} else {$_SESSION['authuser'] = 0;}
  10. }
  11.  else {
  12. 	 $_SESSION['authuser'] = 0;
  13. 	 }
  14.  
  15.  
  16.  if (isset($_SESSION['authuser']) && $_SESSION['authuser'] == 1) {
  17. // nic nie rób
  18. } else {
  19. 	echo "Przepraszam, ale nie masz uprawień do tej strony!";
  20.    exit();
  21.    }
  22. ?>
[PHP] pobierz, plaintext

i niestety dalej nie działa, tzn nie mogę się zalogować wpisując prawidłowo hasło i login :/

Gdy skasowałem wszystko od 16 linii to myślałem już że jest wszystko oki bo zalogowało mnie a potem po delete wskoczyło normalnie znowu, ale okazało się że poprostu co by nie wpisał to wchodziło :/ więc dalej kombinuje
Go to the top of the page
+Quote Post
barat
post 21.05.2007, 22:16:59
Post #14





Grupa: Zarejestrowani
Postów: 183
Pomógł: 0
Dołączył: 19.05.2007

Ostrzeżenie: (0%)
-----

sprawdź w 3 linii w w AND czy muszą być podwójne nawiasy - chyba nie smile.gif (to są te co dodałeś, a ja usunąłem tongue.gif)

Na razie piszę dla siebie trochę pokomplikowane cuś (też jestem początkujący) więc nie za bardzo mam czas sie wgłębić i pisze z głowy smile.gif

Na bank gdzieś mam przerobione przykłady z logowaniem na sesjach ... znajde to dam , ale pewnie szybciej sam na to wpadniesz smile.gif

Pisz od początku testowy skrypt ... po kolei,, kroczkami,
Najpierw niech wyłapie $_POST, ustawi password i username na to co było w post i wyświetli (echo) ... potem powoli dodawaj nowe ... smile.gif

Ja po prostu jak piszę, to właśnie co kilka kroczków na chwile wstukuje echo ostatniej lub jakiejś ważnej zmiennej i sprawdzam co zwraca smile.gif

Ten post edytował barat 21.05.2007, 22:20:51
Go to the top of the page
+Quote Post
Oscar_83
post 21.05.2007, 22:44:16
Post #15





Grupa: Zarejestrowani
Postów: 80
Pomógł: 0
Dołączył: 10.05.2007

Ostrzeżenie: (0%)
-----

dodałem bo wywalało mi błąd ale teraz usunąłem i działa, lecz ciągle pisze że nie mam dostępu wpisując prawidłowe hasło, zmienie jeszcze ten drugi nawias który dodałem, może też niepotrzebnie.
Gdy usunąłem te drugie nawiasy to już była lipa - błędy - oki będę kombinował.

Wydaje mi się że ta linijka:
[PHP] pobierz, plaintext 
  1. <?php
  2. if($_SESSION['username'] == "zuku" && $_SESSION['userpass'] == "27k11")
  3. ?>
[PHP] pobierz, plaintext

nie powinna być w { } tyle że samo zlikwidowanie {} nie pomoże trzeba by było to jakoś przekształcić.

Ten post edytował Oscar_83 21.05.2007, 22:22:05
Go to the top of the page
+Quote Post
Kicok
post 22.05.2007, 08:02:11
Post #16





Grupa: Zarejestrowani
Postów: 1 033
Pomógł: 125
Dołączył: 17.09.2005
Skąd: Żywiec

Ostrzeżenie: (0%)
-----

A nie prościej zrobić to w ten sposób:


1. Startujesz sesję
2. Sprawdzasz, czy $_SESSION['authuser'] jest ustawiona (isset" title="Zobacz w manualu PHP" target="_manual) i czy jest równa 1
> 3. Jeśli tak, to wyświetlasz treść strony dla zalogowanych

4. Jeśli nie, to sprawdzasz, czy wysłany został formularz:(empty" title="Zobacz w manualu PHP" target="_manual)
> 5. Jeśli nie, to przerzucasz użytkownika do strony z formularzem logowania (header" title="Zobacz w manualu PHP" target="_manual)
> 6. Jeśli tak, to sprawdzasz czy podane login i hasło się zgadzają:
>> 7. Jeśli nie, to wyświetlasz informacje, że podano złe dane podczas logowania
>> 8. Jeśli tak, to ustawiasz $_SESSION['authuser'] na 1 i przeładowujesz stronę, żeby cały kod - łącznie ze sprawdzaniem $_SESSION['authuser'] - wykonał się ponownie (header" title="Zobacz w manualu PHP" target="_manual)

Do tego jeszcze jakiś pliczek logout.php, który usunie zmienną sesyjną $_SESSION['authuser'] (unset" title="Zobacz w manualu PHP" target="_manual) i przekieruje na stronę główną, albo do formularza logowania (header" title="Zobacz w manualu PHP" target="_manual)


--------------------
"Sumienie mam czyste, bo nieużywane."
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 18.08.2025 - 02:50
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn