Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP]Problem z formularzem kontaktowym
Forum PHP.pl > Forum > Przedszkole
deserteagle
3.04.2013, 21:07:46
Witam, mam niemały problem, dostałem w swoje ręce leciwą stronkę, muszę ja odswieżyć, poprawić i uporządkować. Mam formularz kontaktowy, który ponoć pewnego dnia sam z siebie się wysypał. Niestety php nie jest moją mocną stroną, próbowałem na wiele sposobów go naprawić, ale jak tylko jedno poprawiłem to zaraz pojawiał się nastepny błąd. Proszę o sprawdzenie co jest nie tak i małą podpowiedź. Dodam, że po kliknięciu na link prowadzący do formularza wyświetla się komunikat: Something is wrong in your syntax obok '' w linii 1

[PHP] pobierz, plaintext 
  1.  
  2. <?php
  3.  
  4. include ("polaczenie_z_baza.php");
  5.  
  6. $pobierz_art_sql = "SELECT Nr_kat, Fota, Nazwa_pl, Kategoria, Opis_pl, Atrybut_name FROM przedmioty_strona WHERE Id=".$id;
  7. 						$pobierz_art_rez = mysql_query($pobierz_art_sql) 
  8. 						or die(mysql_error($mysqli));
  9.  
  10. echo "
  11. 		<html>
  12. 		<head>
  13. 		<meta http-equiv=\"Content-Language\" content=\"pl\">
  14. 		<meta http-equiv=\"Content-type\" content=\"text/html; charset=iso-8859-2\" />
  15. 		<meta id=\"Robots\" name=\"Robots\" content=\"index\" />
  16. 		<title> Formularz kontaktowy przedmiotu" . $art_info['Nazwa_pl'] . " </title> 
  17.    <link rel=\"stylesheet\" href=\" styl.css \" type=\"text/css\" />
  18.    <!--[if lt IE 7.0]><link rel=\"stylesheet\" type=\"text/css\" href=\"ie.css\" /><![endif]-->
  19.    <!--[if IE 7.0]><link rel=\"stylesheet\" type=\"text/css\" href=\"ie7.css\" /><![endif]-->
  20.  
  21. 		</head>
  22. 		<body style=\"padding: 15px;\">";
  23.  
  24. while ($art_info = mysql_fetch_array($pobierz_art_rez)) {
  25.  
  26. if(! $_POST){
  27.     echo "<br />
  28. 		<img src=\"../pliki/male/".$art_info['Fota']."1.jpg\" class=\"kontaktowy2\" alt=\"Fotka przedmiotu\" />
  29. 		<br />
  30. 		<p class=\"kontaktowy3\">Formularz kontaktowy przedmiotu ".$art_info['Nazwa_pl']."</p><br/>
  31. 		<form method=\"post\" class=\"kontaktowy1\"><input type=\"hidden\" name=\"send\" value=\"yes\">
  32. 		<p class=\"kontaktowy4\">Twój adres e-mail:<br/>
  33. 		<input type=\"text\" name=\"sender\" maxlength=\"150\"/ class=\"kontaktowy\">
  34. 		</p><br />
  35. 		<p class=\"kontaktowy4\">Treść wiadomości:<br/>
  36. 		<textarea name=\"message\" wrap=\"virtual\" class=\"kontaktowyy\"></textarea></p><br />";
  37.  
  38.  
  39. // inicjujemy generator licz losowych  
  40. srand((float)microtime()*1000000);
  41.  
  42. // tablica zawierajaca tablice z obrazkiem i opisem do niego
  43. $txt[] = array('9sajdh.gif' => '125895');
  44. $txt[] = array('587km.gif' => '100983');
  45. $txt[] = array('609ks.gif' => '056890');
  46. $txt[] = array('904jgh.gif' => '720804');
  47. $txt[] = array('abcd78.gif' => '770096');
  48.  
  49. // wybieramy losowy element z tablicy
  50. $txt_rand = $txt[array_rand($txt)];
  51.  
  52. 		foreach($txt_rand as $key => $value) {
  53.      echo "<img src=\"http://artarmsantik.com/pliki/numerki/".$key."\"  class=\"kontaktowyobrazek\" />
  54. 		 <input type=\"hidden\" name=\"obrazek\" value=\"".$value."\" >
  55. 		 <p class=\"kontaktowy4\">Proszę przepisać cyfry z obrazka:</p>
  56. 		 <input type=\"text\" name=\"wartosc\" maxlength=\"50\"/ class=\"kontaktowykod\" >";
  57. } 
  58. 		echo "<p class=\"kontaktowy4\"><button type=\"submit\" name=\"submit\" value=\"send\" class=\"kontaktowy5\"/><img src=\"../pliki/wyslij.jpg\" /></button>
  59.  
  60. 		</form>";
  61. 		}
  62.  
  63. if($_POST){
  64. 	if(($_POST['sender']!='') && ($_POST['sender']!=' ') && ($_POST['obrazek']==$_POST['wartosc'])){
  65. 			$mailheaders = "Reply-To: ".$_POST['sender']."\n";
  66. 			$mailheaders .= "Content-Type: text/html; charset=ISO-8859-2\n";
  67. 	if ($send == "yes"){
  68. 			$message="".stripslashes($_POST['message']). "\r\n" . "\n <br/><span style=\"font-weight:bold; color:red\">email nadawcy:</span>\n".$_POST['sender'] ;
  69. 			$subject="Pytanie o: ".stripslashes($art_info['Nazwa_pl'])." nr kat:".$art_info['Nr_kat'];
  70. 			mail ("skrzynia@artarmsantik.com",$subject,$message,$mailheaders);}
  71.  
  72.  
  73. 		print "<br /><br /><br /><br /><br /><br /><p class=\"kontaktowy4\">Pytanie zostało wysłane!<br />Wkrótce odpowiemy</p><br /><br /><p class=\"kontaktowy4\" onclick=\"window.close()\">ZAKOŃCZ</p>";}
  74. 	else  {echo "<br /><p class=\"kontaktowy3\">Proszę poprawnie uzupełnić formularz!</p><br />
  75. 		<img src=\"../pliki/male/".$art_info['Fota']."1.jpg\" class=\"kontaktowy2\" alt=\"Fotka przedmiotu\" />
  76. 		<br />
  77. 		<p class=\"kontaktowy3\">Formularz kontaktowy przedmiotu ".$art_info['Nazwa_pl']."</p><br/>
  78. 		<form method=\"post\" class=\"kontaktowy1\"><input type=\"hidden\" name=\"send\" value=\"yes\">
  79. 		<p class=\"kontaktowy4\">Twój adres e-mail:<br/>
  80. 		<input type=\"text\" name=\"sender\" maxlength=\"150\"/ class=\"kontaktowy\">
  81. 		</p><br />
  82. 		<p class=\"kontaktowy4\">Treść wiadomości:<br/>
  83. 		<textarea name=\"message\" wrap=\"virtual\" class=\"kontaktowyy\"></textarea></p><br />";
  84.  
  85.  
  86. // inicjujemy generator licz losowych  
  87. srand((float)microtime()*1000000);
  88.  
  89. // tablica zawierajaca tablice z obrazkiem i opisem do niego
  90. $txt[] = array('9sajdk.gif' => '125895');
  91. $txt[] = array('587km.gif' => '100983');
  92. $txt[] = array('609ks.gif' => '056890');
  93. $txt[] = array('904jgh.gif' => '720804');
  94. $txt[] = array('abcd78.gif' => '770096');
  95.  
  96. // wybieramy losowy element z tablicy
  97. $txt_rand = $txt[array_rand($txt)];
  98.  
  99. 		foreach($txt_rand as $key => $value) {
  100.      echo "<img src=\"http://artarmsantik.com/pliki/numerki/".$key."\"  class=\"kontaktowyobrazek\"/>
  101. 		 <input type=\"hidden\" name=\"obrazek\" value=\"".$value."\" >
  102. 		 <p class=\"kontaktowy4\">Kod z obrazka:</p>
  103. 		 <input type=\"text\" name=\"wartosc\" maxlength=\"50\"/ class=\"kontaktowykod\" >";
  104. } 
  105. 		echo "<p class=\"kontaktowy4\"><button type=\"submit\" name=\"submit\" value=\"send\" class=\"kontaktowy5\"/><img src=\"../pliki/wyslij.jpg\" /></button>
  106. 		</form>";
  107. 		}
  108. 		}
  109.  
  110. echo"		
  111. 		</body>
  112. 		</html>";}
  113. ?>
  114.  
[PHP] pobierz, plaintext
Michael2318
3.04.2013, 22:06:01
przed <?php i po ?> nie ma prawa być żadnej spacji, kropki, odstępu - nic. A z tego co zaprezentowałeś w kodzie to takowe tam występują.
StrefaPi
3.04.2013, 22:35:22
puste linie nie mają znaczenia błąd jest w zapytaniu (linia 6)

[PHP] pobierz, plaintext 
  1. $pobierz_art_sql = "SELECT Nr_kat, Fota, Nazwa_pl, Kategoria, Opis_pl, Atrybut_name FROM przedmioty_strona WHERE Id=".$id;
[PHP] pobierz, plaintext

skąd pobierasz $id?

tzw. Undefined variable: id
deserteagle
4.04.2013, 11:30:57
Hm, ciężkie pytanie jak dla mnie, wiem, że nie odpowiada się pytaniem na pytanie ale nie mam wyjścia:a skąd powinienem pobierać id, myślę, że wszystkie dane pobierane są z bazy, stąd pierwsze polecenie
[PHP] pobierz, plaintext 
  1. include ("polaczenie_z_baza.php");
[PHP] pobierz, plaintext
, bynajmniej ja tak to widzę.
nospor
4.04.2013, 11:33:47
id idzie postem lub w linku. Ma być
$id = $_GET['id'];
lub
$id = $_POST['id'];


A kod nie przestał działać sam z siebie, tylko zmieniła się konfiguracja php i dlatego przestało ci działać.
deserteagle
4.04.2013, 12:58:32
Czyli przed 
[PHP] pobierz, plaintext 
  1. $pobierz_art_sql = "SELECT Nr_kat, Fota, Nazwa_pl, Kategoria, Opis_pl, Atrybut_name FROM przedmioty_strona WHERE Id=".$id;
[PHP] pobierz, plaintext

mam wstawić 
[PHP] pobierz, plaintext 
  1. $id = $_GET['id'];
[PHP] pobierz, plaintext
, pewnie nie bo to byłoby zbyt proste, nospor co masz na myśli "zmieniła się konfiguracja PHP", nie wiem czy ktoś wcześniej grzebał na tej stronie, właściciel twierdzi, że nie, on sam na pewno nie grzebałby w kodzie bo na sam wyraz komputer dostaje gęsiej skórki smile.gif
nospor
4.04.2013, 13:01:08
Cytat
, pewnie nie bo to byłoby zbyt proste
Naprawdę.... mogłeś to sprawdzić.....

Tak, masz to dać przed pierwszym użyciem $id - to jest dość logiczne.

A konfigurację to mógł zmienić hosting i pewnie tak było.
matiit
4.04.2013, 13:02:26
deserteagle, tak, daj taką linijkę jak powiedziałeś.
a jeszcze lepiej:


[PHP] pobierz, plaintext 
  1. if(isset($_GET['id']))
  2. {$id=$_GET['id'];
  3. //cala reszta kodu
  4. }
  5. else {
  6. echo 'Nie wyswietlasz zadnego posta';
  7. }
[PHP] pobierz, plaintext


coś takiego.

Nie jest to i tak bezpieczne (podatne na sql injection).
deserteagle
4.04.2013, 13:10:11
nospor spokojnie nie unoś się, nie ma sensu, wiem, że rozmowa z żółtodziobem może być irytująca, ale odrobinę wyrozumiałości, już na wstępie pisałem, że php to czarna magia dla mnie, a nie mogę tego tak szybko sprawdzić bo nie mam dostępu w tej chwili do źródła, nie robię tego za pieniądze i nie siedzę u człowieka w domu przez 8 godz. jutro tam będę to sprawdzę czy o to chodziło, wolę się upewnić gdzie mam coś wstawić, niż jechać prawie 20 km. tylko po to, żebym wkleił coś nie w tym miejscu. Dzięki za pomoc, dam znać czy w tym tkwił problem.

Dzięki matiit.
nospor
4.04.2013, 13:12:42
Ba.... jakbyś od razu napisał jak sprawa wygląda, to i ja bym się nie unosił smile.gif Ale 99% tego typu postów wygląda tak, że user ma php przed nosem, ale i tak zanim cokolwiek wciśnie to leci 15 razy na forum by się upewnić, jakby jego akcja miała wywołać 3 wojnę światową wink.gif

Pamiętaj tylko co napisałem: jak nie $_GET to $_POST - jedno z dwóch
deserteagle
4.04.2013, 13:24:53
Czyli całość powinna wyglądać tak:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. include ("polaczenie_z_baza.php");
  4.  
  5. if(isset($_GET['id']))
  6. {$id=$_GET['id'];
  7.  
  8. $pobierz_art_sql = "SELECT Nr_kat, Fota, Nazwa_pl, Kategoria, Opis_pl, Atrybut_name FROM przedmioty_strona WHERE Id=".$id;
  9. $pobierz_art_rez = mysql_query($pobierz_art_sql)
  10. or die(mysql_error($mysqli));
  11.  
  12. }
  13. else {
  14. echo 'Nie wyswietlasz zadnego posta';
  15. }
  16.  
  17. echo "
  18. <html>
  19. <head>
  20. <meta http-equiv=\"Content-Language\" content=\"pl\">
  21. <meta http-equiv=\"Content-type\" content=\"text/html; charset=iso-8859-2\" />
  22. <meta id=\"Robots\" name=\"Robots\" content=\"index\" />
  23. <title> Formularz kontaktowy przedmiotu" . $art_info['Nazwa_pl'] . " </title>
  24. <link rel=\"stylesheet\" href=\" styl.css \" type=\"text/css\" />
  25. <!--[if lt IE 7.0]><link rel=\"stylesheet\" type=\"text/css\" href=\"ie.css\" /><![endif]-->
  26. <!--[if IE 7.0]><link rel=\"stylesheet\" type=\"text/css\" href=\"ie7.css\" /><![endif]-->
  27.  
  28. </head>
  29. <body style=\"padding: 15px;\">";
  30.  
  31. while ($art_info = mysql_fetch_array($pobierz_art_rez)) {
  32.  
  33. if(! $_POST){
  34. echo "<br />
  35. <img src=\"../pliki/male/".$art_info['Fota']."1.jpg\" class=\"kontaktowy2\" alt=\"Fotka przedmiotu\" />
  36. <br />
  37. <p class=\"kontaktowy3\">Formularz kontaktowy przedmiotu ".$art_info['Nazwa_pl']."</p><br/>
  38. <form method=\"post\" class=\"kontaktowy1\"><input type=\"hidden\" name=\"send\" value=\"yes\">
  39. <p class=\"kontaktowy4\">Twój adres e-mail:<br/>
  40. <input type=\"text\" name=\"sender\" maxlength=\"150\"/ class=\"kontaktowy\">
  41. </p><br />
  42. <p class=\"kontaktowy4\">Treść wiadomości:<br/>
  43. <textarea name=\"message\" wrap=\"virtual\" class=\"kontaktowyy\"></textarea></p><br />";
  44.  
  45.  
  46. // inicjujemy generator licz losowych
  47. srand((float)microtime()*1000000);
  48.  
  49. // tablica zawierajaca tablice z obrazkiem i opisem do niego
  50. $txt[] = array('9sajdh.gif' => '125895');
  51. $txt[] = array('587km.gif' => '100983');
  52. $txt[] = array('609ks.gif' => '056890');
  53. $txt[] = array('904jgh.gif' => '720804');
  54. $txt[] = array('abcd78.gif' => '770096');
  55.  
  56. // wybieramy losowy element z tablicy
  57. $txt_rand = $txt[array_rand($txt)];
  58.  
  59. foreach($txt_rand as $key => $value) {
  60. echo "<img src=\"http://artarmsantik.com/pliki/numerki/".$key."\" class=\"kontaktowyobrazek\" />
  61. <input type=\"hidden\" name=\"obrazek\" value=\"".$value."\" >
  62. <p class=\"kontaktowy4\">Proszę przepisać cyfry z obrazka:</p>
  63. <input type=\"text\" name=\"wartosc\" maxlength=\"50\"/ class=\"kontaktowykod\" >";
  64. }
  65. echo "<p class=\"kontaktowy4\"><button type=\"submit\" name=\"submit\" value=\"send\" class=\"kontaktowy5\"/><img src=\"../pliki/wyslij.jpg\" /></button>
  66.  
  67. </form>";
  68. }
  69.  
  70. if($_POST){
  71. if(($_POST['sender']!='') && ($_POST['sender']!=' ') && ($_POST['obrazek']==$_POST['wartosc'])){
  72. $mailheaders = "Reply-To: ".$_POST['sender']."\n";
  73. $mailheaders .= "Content-Type: text/html; charset=ISO-8859-2\n";
  74. if ($send == "yes"){
  75. $message="".stripslashes($_POST['message']). "\r\n" . "\n <br/><span style=\"font-weight:bold; color:red\">email nadawcy:</span>\n".$_POST['sender'] ;
  76. $subject="Pytanie o: ".stripslashes($art_info['Nazwa_pl'])." nr kat:".$art_info['Nr_kat'];
  77. mail ("skrzynia@artarmsantik.com",$subject,$message,$mailheaders);}
  78.  
  79.  
  80. print "<br /><br /><br /><br /><br /><br /><p class=\"kontaktowy4\">Pytanie zostało wysłane!<br />Wkrótce odpowiemy</p><br /><br /><p class=\"kontaktowy4\" onclick=\"window.close()\">ZAKOŃCZ</p>";}
  81. else {echo "<br /><p class=\"kontaktowy3\">Proszę poprawnie uzupełnić formularz!</p><br />
  82. <img src=\"../pliki/male/".$art_info['Fota']."1.jpg\" class=\"kontaktowy2\" alt=\"Fotka przedmiotu\" />
  83. <br />
  84. <p class=\"kontaktowy3\">Formularz kontaktowy przedmiotu ".$art_info['Nazwa_pl']."</p><br/>
  85. <form method=\"post\" class=\"kontaktowy1\"><input type=\"hidden\" name=\"send\" value=\"yes\">
  86. <p class=\"kontaktowy4\">Twój adres e-mail:<br/>
  87. <input type=\"text\" name=\"sender\" maxlength=\"150\"/ class=\"kontaktowy\">
  88. </p><br />
  89. <p class=\"kontaktowy4\">Treść wiadomości:<br/>
  90. <textarea name=\"message\" wrap=\"virtual\" class=\"kontaktowyy\"></textarea></p><br />";
  91.  
  92.  
  93. // inicjujemy generator licz losowych
  94. srand((float)microtime()*1000000);
  95.  
  96. // tablica zawierajaca tablice z obrazkiem i opisem do niego
  97. $txt[] = array('9sajdk.gif' => '125895');
  98. $txt[] = array('587km.gif' => '100983');
  99. $txt[] = array('609ks.gif' => '056890');
  100. $txt[] = array('904jgh.gif' => '720804');
  101. $txt[] = array('abcd78.gif' => '770096');
  102.  
  103. // wybieramy losowy element z tablicy
  104. $txt_rand = $txt[array_rand($txt)];
  105.  
  106. foreach($txt_rand as $key => $value) {
  107. echo "<img src=\"http://artarmsantik.com/pliki/numerki/".$key."\" class=\"kontaktowyobrazek\"/>
  108. <input type=\"hidden\" name=\"obrazek\" value=\"".$value."\" >
  109. <p class=\"kontaktowy4\">Kod z obrazka:</p>
  110. <input type=\"text\" name=\"wartosc\" maxlength=\"50\"/ class=\"kontaktowykod\" >";
  111. }
  112. echo "<p class=\"kontaktowy4\"><button type=\"submit\" name=\"submit\" value=\"send\" class=\"kontaktowy5\"/><img src=\"../pliki/wyslij.jpg\" /></button>
  113. </form>";
  114. }
  115. }
  116.  
  117. echo"
  118. </body>
  119. </html>";}
  120. ?>
[PHP] pobierz, plaintext


czy coś pokręciłem
nospor
4.04.2013, 13:28:49
Nie, nie kombinuj z tym co ci podał poprzednik, bo jak pojedziesz do klienta i bedzie błąd to znowu nic nie zrobisz.

Zrób to co pisałem, wstaw tylko to
$id=$_GET['id'];
i już.

A najlepiej zrób:
$id=(int)$_GET['id'];

i juz skrypt będzie 100razy bezpieczniejszy.
matiit
4.04.2013, 13:29:02
Raczej tak:
http://pastebin.com/jpMihX4c

Edit:
@nospor a co w przypadku jeśli id nie bedzie zdefiniowane nigdzie.. (aczkolwiek masz racje, skoro jest to działający system to powinno być o to jakoś zadbane w inny sposócool.gif

deserteagle
4.04.2013, 13:37:42
Ok, dzięki Panowie za pomoc, dam znać czy działa.
nospor
4.04.2013, 13:46:52
Cytat
@nospor a co w przypadku jeśli id nie bedzie zdefiniowane nigdzie..
Skoro działało do tej pory a nagle przestało, znaczy że ktoś wyłączył register_globals. I to id brało się automatycznie albo z post, albo z get, albo od biedy z $_SESSION lub $_COOKIE. na 95 było to jednak post lub get. No ale na odległośc mogę jedynie zgadywać.
deserteagle
6.04.2013, 16:30:20
Zadziałało, dzięki jeszcze raz, duże, mocno chmielowe dla Ciebie nospor i dla matiit oczywiście też. Można zamknąć.
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.