Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> 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
mat-bi
post
Post #2





Grupa: Zarejestrowani
Postów: 690
Pomógł: 92
Dołączył: 6.02.2011

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

nie ma to jak stare, dobre SQL Injection, jeszcze do tego najprosztsze, jakie może być (IMG:style_emoticons/default/wink.gif)

A tak serio:
1. wywal echo przed header
2. popraw skrypt n odpornośc przed SQL Injection
3. użyłbym mysql_num_rows zamiast mysql_fetch_assoc
4. może nawet w ogóle bym wywalić zwykłego selecta, zastępując go COUNT(), gdyż i tak zapisujesz do sesji tylko te dane, które dostajesz z formularza
Go to the top of the page
+Quote Post
danielntk
post
Post #3





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

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

ale co to znaczników ? , Bo są dwa razy użyte <?php ?> (IMG:style_emoticons/default/smile.gif) Jest dobrze zrobione
Go to the top of the page
+Quote Post
mat-bi
post
Post #4





Grupa: Zarejestrowani
Postów: 690
Pomógł: 92
Dołączył: 6.02.2011

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

Mozesz otworzyć tyle razy te znaczniki, ile chcesz (oczywiście, w granicach poprawności i normy (IMG:style_emoticons/default/wink.gif) )
Go to the top of the page
+Quote Post
markonix
post
Post #5





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
« 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: 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 Aktualny czas: 25.08.2025 - 01:55
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn