Új hozzászólás Aktív témák

• #2383 Sk8erPeter nagyúr SektorFlop #2379

  Új Válasz 2012-03-07 18:01:44 #2383

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Sk8erPeter

  nagyúr

  válasz SektorFlop #2379 üzenetére

  Ez a kód már csak azért is fájó, mert olyan könnyen támadható az oldalad SQL Injectionnel, mint az állat. Nem ártana escape-elni a felhasználótól érkező tartalmat - soha ne bízz meg abban az adatban, ami a felhasználótól közvetlenül érkezik...

  Másrészt ha kódot illesztesz be, akkor jelöld ki a kódodat itt a szövegmezőben, majd kattints a "Programkód" gombra, pont arra való, hogy valami normálisan átlátható formában legyenek a kódok.

  Harmadrészt: a mailcsere.php fájlban abszolúte semmi értelme JavaScripttel átirányítani a felhasználót, arra ott a PHP header() függvénye is - felesleges tehát kiíratni bármit is.
  Ahogy a példában van:
  header('Location: http://www.example.com/');
  de ezelőtt ne legyen SEMMILYEN kiíratás.
  DE (!!!) ezt NE TEDD BELE!! Egyáltalán ne rakj bele semmilyen átirányítást, mert most AJAX-os kommunikáció zajlik, lásd később a magyarázatot.

  Aztán menjünk tovább.
  Ez van most a kódodban:
  /* Send the data using post and put the results in a div */
  $.post( 'login/mailcsere.php', { email: term },
  function( data ) {
  var content = $( data ).find( '#content' );
  $( "#result" ).empty().append( content );
  }
  );

  Ez most azt csinálja, hogy a login/mailcsere.php fájlnak elküldi a term-ben található adatot, az emailbe objektumba begyűjtve, ez majd PHP-oldalon is megjelenik.
  A sikeres (!) adatküldés esetén (pl. létezik a fájl, elérhető, nem dobott vissza hibát, stb.) a content változóba úgy pakolja be az adatot, hogy a visszakapott adatokból kikeresi a "content" id-val rendelkező DOM-elemet (!), aztán a content változó tartalmát bepakolja a "result" id-val rendelkező HTML-elembe.

  Egyrészt: amit Te visszaadsz a PHP-fájllal, abba nem raksz bele semmi olyat, amiben "content" id-val rendelkező elem van, másrészt egyáltalán nincs "result" id-vel rendelkező elemed az oldalon.
  Sőt, a PHP-fájlban csak azt a JavaScriptes visszairányítós részt íratod ki, ami AJAX-os kommunikáció esetén eleve felesleges.

  Először próbáld csak úgy, hogy kiíratod echo-val, hogy "siker" vagy "hiba", aztán a $.post részt egyelőre (!) így módosítod:
  var content = $( data ).find( '#content' );
  $( "#result" ).empty().append( content );
  HELYETT
  var content = $( data );
  $( "#result" ).empty().append( content );

  Ezenkívül még tegyél be a mobil.php fájlba egy ilyet valahova a végére mondjuk:
  <div id="result"></div>

  Ide fogja pakolni az AJAX-kommunikáció eredményét.

  Sk8erPeter

Új hozzászólás Aktív témák