Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Pytanie dotyczące znaczników, Pytanie dotyczące znaczników
danielntk
post
Post #1





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 4.08.2010

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

Witam:) To chyba mój pierwszy post.

Czy dokument php jest poprawnie napisany ? , chodzi o zastosowanie znaczników php a w środku jest html. ?


[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. include('mysql.php');
  4.  
  5. ?>
  6.  
  7. <html>
  8. <head>
  9. <body>
  10. <form action="zaloguj.php" method="POST">
  11. <table>
  12. <tr>
  13. <td><label for="1">Gracz:</label></td><td><input type="text" name="uzytkownik" id="1"></td>
  14. </tr>
  15.  
  16. <tr>
  17. <td><label for="2">Hasło:</label></td><td><input type="password" name="haslo"id="2"></td>
  18. <td><input type="submit" value="Zaloguj" </td>
  19. </tr>
  20. </table>
  21. </form>
  22. </body>
  23. </head>
  24. </html>
  25.  
  26. <?php
  27.  
  28. if(isset($_POST['uzytkownik']) && isset($_POST['haslo'])) {
  29.  
  30. $uzytkownik = $_POST['uzytkownik'];
  31. $haslo = $_POST['haslo'];
  32.  
  33.  
  34.  
  35. $haslo = sha1(md5($haslo));
  36.  
  37. $zaloguj = "SELECT * FROM `gracz` WHERE `uzytkownik`='$uzytkownik' AND `haslo`='$haslo'";
  38. $rezultat = mysql_query($zaloguj);
  39. $pokaz = mysql_fetch_assoc($rezultat);
  40.  
  41. }
  42.  
  43. if(isset($pokaz['uzytkownik']) && isset($pokaz['haslo'])) {
  44.  
  45.     $_SESSION['uzytkownik'] = $uzytkownik;
  46.     $_SESSION['haslo'] = $haslo;
  47.  
  48.    echo header("Location: index.php");
  49.  
  50. }
  51. ?>
  52.  
  53.  
[PHP] pobierz, plaintext


Ten post edytował danielntk 19.06.2011, 16:47:55
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
markonix
post
Post #2





Grupa: Zarejestrowani
Postów: 2 707
Pomógł: 290
Dołączył: 16.12.2008
Skąd: Śląsk

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

Przecież jest to totalnie niepoprawne.
header nigdy się nie wykona gdy coś prześlesz do przeglądarki, a cała strona jest przesyłana.
Jeśli umieszczasz kod PHP w pliku z HTMLem to staraj się go dawać na samym początku, a w samym kodzie PHP stosuj tylko funkcje typu echo.

Np. teraz warto by dać informacje o niepoprawnym loginie, a chyba nie wyświetlisz tego w stopce strony?
Poza tym po co zmienne $zaloguj i $rezultat gdy można wszystko zapisać tak:
[PHP] pobierz, plaintext 
  1. $row =  mysql_fetch_assoc(mysql_query("SELECT * FROM `gracz` WHERE `uzytkownik`='$uzytkownik' AND `haslo`='$haslo'"));
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post

Posty w temacie
- danielntk   Pytanie dotyczące znaczników   19.06.2011, 16:37:45
- - mat-bi   nie ma to jak stare, dobre SQL Injection, jeszcze ...   19.06.2011, 17:40:00
- - danielntk   ale co to znaczników ? , Bo są dwa razy użyte ...   19.06.2011, 17:41:44
- - mat-bi   Mozesz otworzyć tyle razy te znaczniki, ile chcesz...   19.06.2011, 17:44:26
- - markonix   Przecież jest to totalnie niepoprawne. header nigd...   19.06.2011, 18:10:13

« Następny starszy · PHP · Następny nowszy »
 

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

 

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

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

RSS Aktualny czas: 13.10.2025 - 13:27
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn