Forum PHP.pl > [php] Maly problem z wstawianiem zmiennych

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php] Maly problem z wstawianiem zmiennych

eXido

9.05.2006, 14:13:12

Witam,

Ponizej jest kod na którym pracuje, nic wielkiego, uzytkownik wpisuje pare rzeczy które php potem przekazje MySQLowi:

[PHP] pobierz, plaintext 
<html>
<head><title>Add a Hirer!</title>
</head>
<body background="img19.gif">
 
<center>
<p1><B>Add a New Hirer to the Village Hall Booking System!</B></p1>
</center>
 
 
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
 
<br><table><tr><td width="200">
 
Title</td><td width="200"><select name="title">
<option>Choose
<option>Doctor
<option>Father
<option>Master
<option>Miss
<option>Mr
<option>Mrs
<option>Professor
<option>Sister
</select>
 
</td><td width="200"><b>Further Details</b></td></tr>
 
<tr><td width="200">
 
Surname</td><td width="150"><input type="text" name="surname"><br></td><td width="200">Time of Booking:</td>
<td width="200"><select name="Time">
<option selected>Choose
<Option>Morning
<Option>Afternoon
<Option>Evening
<Option>Evening Extension
</select>
</td></tr><tr><td width="200">Forename</td><td width="200"><input type="text" name="forename"></td>
<td width="200">Type of Booking:</td><td width="200"><select name="Type">
<option selected>Choose
<option>Chess Club
<option>Cubs & Scouts
<option>Golden Wedding Anniversary
<option>Guides & Brownies
<option>Line Dancing
<option>Play Group
<option>Silver Wedding Anniversary
<option>Women's Institute
<option>18th Birthday Party
<option>Other
</select>
</td></tr><tr><td width="200">Street Number</td><td width="200"><input type="text" name="street_number"></td>
<td width="200">Date:</td><td width="200">
 
<select name="DateNumber">
<option selected>1
<option>2
<option>3
<option>4
<option>5
<option>6
<option>7
<option>8
<option>9
<option>10
<option>11
<option>12
<option>13
<option>14
<option>15
<option>16
<option>17
<option>18
<option>19
<option>20
<option>21
<option>22
<option>23
<option>24
<option>25
<option>26
<option>27
<option>28
<option>29
<option>30
<option>31
<option>32
</select>
 
<select name="DateMonth">
<option selected>January
<option>February
<option>March
<option>April
<option>May
<option>June
<option>July
<option>August
<option>September
<option>October
<option>November
<option>December
</select>
 
<select name="DateYear">
<option selected>2005
<option>2006
<option>2007
</select>
 
</td></tr><tr><td width="200">Street Name</td><td width="200"><input type="text" name="street_name"></td>
<td width="200">Day Of Week:</td><td width="200"><select name="Day_of_week">
<option selected>Choose
<option>Monday
<option>Tuesday
<option>Wednesday
<option>Thursday
<option>Friday
<option>Saturday
<option>Sunday
</select>
</td></tr><tr><td width="200">Town/City</td><td width="200"><input type="text" name="town_city"></td>
<td width="200">Regular Customer:</td><td width="200"><select name="Regular">
<option selected>Choose
<option>Yes
<option>No
</select>
</td></tr><tr><td width="200">County</td><td width="200"><input type="text" name="county"></td>
<td width="200">Paid:</td><td width="200"><select name="Paid">
<option selected>Choose
<option>Yes
<option>No
</select>
</td></tr><tr><td width="200">Post Code</td><td width="200"><input type="text" name="PostCode">
</td></tr><tr><td width="200">Home Tel. No.</td><td width="200"><input type="text" name="home_tel">
</td></tr><tr><td width="200">Mobile Tel. No.</td><td width="200"><input type="text" name="mobile_tel">
</td></tr><tr><td width="200">Email</td><td width="200"><input type="text" name="email">
</td></tr> 
 
 
</table>
 
<br><br><input type="submit" value="Submit"><br><br>
</form>
<font color="orange"><font size="4"><B>
<?php
 
$Time = $_REQUEST['Time'];
$Price = $_REQUEST['Price'];
$Regular = $_REQUEST['Regular'];
$Paid = $_REQUEST['Paid'];
$Type = $_REQUEST['Type'];
$Day_of_week = $_REQUEST['Day_of_week'];
 
 
if ($Type == 'Golden Wedding Anniversary' OR $Type == 'Silver Wedding Anniversary' OR $Type == '18th Birthday Party' AND $Day_of_week == 'Friday' OR $Day_of_week == 'Saturday' AND $Time == 'Evening') {
$Price="10.00";
echo "Your booking qualifies for our special rate of 10.00 pounds including the e
vening extension!";
 
}
 
 
elseif ($Time == 'Morning') {
$Price="12.00";
echo "The price is 12.00 pounds.";
 
}
 
elseif ($Time == 'Afternoon') {
$Price="12.00";
echo "The price is 12.00 pounds.";
 
}
 
elseif ($Time == 'Evening') {
$Price="15.00";
echo "The price is 15.00 pounds.";
 
}
 
elseif ($Time == 'Evening Extension') {
$Price="5.00";
echo "The price is 5.00 pounds.";
 
}
 
 
 
else {
 
echo "The price will be quoted based on your selection.";
 
}
 
 
if ($Regular == 'Yes') {
$Regular="Yes";
}
else {
$Regular="No";
}
 
if ($Paid == 'Yes') {
$Paid="Yes";
}
else {
$Paid="No";
}
 
 
?>
<br>
<?php
 
$dbcnx = @mysql_connect('localhost', 'anglopol_admin', 'pass');
if (!$dbcnx) {
exit('<p>Unable to connect to the database server at this time.</p>');
}
 
if (!@mysql_select_db('anglopol_VillageHall')) {
exit('<p>Unable to locate the Village Hall database at this time.</p>');
}
 
 
if (isset($_POST['title'])) {
 
 
$title = $_POST['title'];
$surname = $_POST['surname'];
$forename = $_POST['forename'];
$street_number = $_POST['street_number'];
$street_name = $_POST['street_name'];
$town_city = $_POST['town_city'];
$county = $_POST['county'];
$PostCode = $_POST['PostCode'];
$Home_Tel = $_POST['home_tel'];
$Mobile_Tel = $_POST['mobile_tel'];
$email = $_POST['email'];
$Time = $_POST['Time'];
$Type = $_POST['Type'];
$DateNumber = $_POST['DateNumber'];
$DateMonth = $_POST['DateMonth'];
$DateYear = $_POST['DateYear'];
$Price = $_POST['Price'];
$Regular = $_POST['Regular'];
$Paid = $_POST['Paid'];
 
$sql = "INSERT INTO Hirer SET
	title='$title',
	surname='$surname',
	forename='$forename',
	street_number='$street_number',
	street_name='$street_name',
	town_city='$town_city',
	county='$county',
	PostCode='$PostCode',
	Home_Tel='$Home_Tel',
	Mobile_Tel='$Mobile_Tel',
	Email='$email'
	Time='$Time',
	Type='$Type',
	DateNumber='$DateNumber',
	DateMonth='$DateMonth',
	DateYear='$DateYear',
	Day_of_week='$Day_of_week',
	Price='$Price',
	Regular='$Regular',
	Paid='$Paid'";
 
if (@mysql_query($sql)) {
 
echo '<p>Your Hirer & Booking Details have been added.</p>';
} else {
echo '<p>Error submitting hirer details: ' .
mysql_error() . '</p>';
}
}
 
$HirerID = mysql_insert_id();
?>
 
 
<br><a href="mainpage.php"> Back to Main Page</a>
 
 
</body>
</html>
[PHP] pobierz, plaintext

Ta stronka juz mi wczesniej dzialala w 100%, tyle ze potem dodalem pare dodatkowych informacji które maja znalezc sie w bazie danych, wszystkie te po $_POST['Email'].
I teraz problem, przestalo dzialac i wyrzuca mi taki blad:

[PHP] pobierz, plaintext 
<?php
 
Error submitting hirer details: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Time='Choose' Type='Choose' DateNumber='1' DateMonth='January' DateYear=' at line 13
 
?>
[PHP] pobierz, plaintext

Widze ze cos jest nie tak z tymi nowymi informacjami które dodalem ale siedzie juz nad tym od wczoraj i przegladam caly kod nie widzac co jest nie tak.
Bylbym naprawde wdzieczny gdyby którys ze znawców mógl pokazac w czym problem

Pozdrawiam.

nospor

9.05.2006, 14:18:06

bo widzisz, pola ktore wymieniasz w zapytaniu, musisz oddzielac przecinkiem, ale akurat o tym zapomniales w pewnym miejscu. w ktorym? o tym juz ci mowi komunikat bledu.

ps: jestes pewien ze ta zabojcza ilosc kodu html ze wszystkimi opcjami selecta jest nam potrzebna?

eXido

9.05.2006, 14:53:44

Alez ja jestem....chyba sie zaciukam. Taki malutki blad i tyle czasu stracilem. Ale w tym piekno programowania

Dzieki za podpowiedz!

Co do tych selectów - chce zeby byla jak najmniejsza szansa ze uzytkownik sie pomyli i wpisze jakas niedobra date - a tak jak dam selecty to nie dosc ze musi wybrac cos poprawnego to jeszcze nie musze potem weryfikowac czy dane sa dobre

Jedna z opcji to Data zamówienia (booking date) w której mam select dla dnia (od 1-32), select na miesiac i potem select na rok. Próbowalem wczesniej miec jedna kolumne na to wszystko w MySQL (rodzaj kolumny "Date") ale jak cokolwiek wprowadzalem do tej kolumny przez forumlarz taki jak ten tu, nic sie nie zapisywalo i w MySQL w kolumnie Date wszystkie daty byly 00-00-0000, mimo iz wprowadzalem wszystkimi mozliwymi formatami ta date.
Jesli móglbys podpowiedziec czemu takie cos nie dzialalo, moze i spróbuje tak bo rzeczywiscie byloby latwiej

nospor

9.05.2006, 14:59:49

Cytat

a tak jak dam selecty to nie dosc ze musi wybrac cos poprawnego

tja, jasne, szczegolnie jak twoj select pozwala na wybranie 32 dni

to sie nazywa poprawnosc danych.

Daj inputa na wpisanie daty w formacie rrrr-mm-dd a potem waliduj. jest mnostwo kodów do walidacji - zaden problem.
Mozesz nawet zrobic kontrolke, w ktorej sie bedzie wybieralo konkretną date z kalendarza, ale to juz inna bajka (ale skrypty tez sa

)

Cytat

to jeszcze nie musze potem weryfikowac czy dane sa dobre

I tu jestes w wielkim bledzie. Dane mozna przeslac nie tylko z Twojego formularza. No chyba ze zastosujesz jakies zabezpieczenie `ala token

eXido

9.05.2006, 15:26:52

Cytat(nospor @ 2006-05-09 13:59:49)

Daj inputa na wpisanie daty w formacie rrrr-mm-dd a potem waliduj. jest mnostwo kodów do walidacji - zaden problem.
Mozesz nawet zrobic kontrolke, w ktorej sie bedzie wybieralo konkretną date z kalendarza, ale to juz inna bajka (ale skrypty tez sa

)

A jak taki input by wygladal, zeby bylo tak jak piszesz w formacie rrrr-mm-dd? Czy móglbys wkleic cos co móglbym wstawic?
I jak walidowac?
Jak widac, jestem jeszcze poczatkujacy

Cytat(nospor @ 2006-05-09 13:59:49)

I tu jestes w wielkim bledzie. Dane mozna przeslac nie tylko z Twojego formularza. No chyba ze zastosujesz jakies zabezpieczenie `ala token

Wiem ale na moje potrzeby (jest to projekt do szkoly, jeszcze nie oczekuja ze bede mial super zabezpieczenia itd) to wystarczy. Ale dobrze ze mówisz, czlowiek codzienniego czegos nowego sie dowiaduje

krzyszbi

9.05.2006, 15:52:56

witam
ja to robie tak ( raczej bezpieczne - user może wybrać tylko date - nie możejej wpisac )
index.html

[HTML] pobierz, plaintext 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html>
<head>
  <link rel="stylesheet" type="text/css" href="styl.css">
  <title>Twoja strona</title>
  <script type="text/javascript">
<!-- <![CDATA[
function okno(pole) {
  o = window.open("kalendarz.php?p="+pole, "k", "width=200, height=200,"
   + " left=100,top=25,innerwidth=200,innerheight=200,location=0,status=0,"
   + " directories=0, scrollbars=0, menubar=0, toolbar=0, resizable=1");
  o.focus();
  return false;
}
// ]]> -->
</script>
</head>
<body>
 <form action="skrypt.php" id="f1" method="post">
  <table border="0" width="100%">
   <tr>
    <td>
      Data :
    </td>
    <td>
     <input type="text" name="data" readonly="readonly" />
     </td>
     <td>
        <a href="#" onclick="return okno('data')">
          <img src="pic/b_calendar.png" border="0" alt="Data - wstaw.">
        </a>
        </td>
       <td>
         <input name="sort" type="submit" value="Pokaż">
       </td>
     </tr>
   </table>
  </form>
</body>
</html>
[HTML] pobierz, plaintext

kod kalendarz.php

[PHP] pobierz, plaintext 
<? echo '<?xml version="1.0" encoding="ISO-8859-2"?>'; ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
  "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="pl"><head>
<meta http-equiv="Content-type" content="text/xml; charset=iso-8859-2" />
<title>Kalendarz</title>
<style type="text/css"><!--
body, td { font-family: arial; font-size: 13px; }
body {margin:0px; padding:0px; }
a { color:black; }
table { width:200px; height:200px; }
.ciemny	{ background:#f3f3f3; }
.jasny	 { background:#FFFFEE; }
.niedziela { background:#FF9999; }
.dzis	  { font-weight:bold;   }
.center	{ text-align:center;  }
--></style>
</head><body>
 
<?
function kalendarz($ile, $p) {
  $znacznik = mktime(0, 0, 0, date("m")+$ile, 1);
  $iledni   = date("t", $znacznik);
  $start	= date("w", $znacznik);
  $miesiac  = date("m", $znacznik);
  $rok	  = date("Y", $znacznik);
  $miesiace = array(
	"Styczeń", "Luty", "Marzec", "Kwiecień", "Maj", "Czerwiec",
	"Lipiec", "Sierpień", "Wrzesień", "Paxdziernik", "Listopad", "Grudzień"
  );
 
  echo '<table border="1" cellspacing="0" cellpadding="2"><tr><td>';
  echo '<a href="kalendarz.php?ile='.($ile-1).'&amp;p='.$p.'">&lt;&lt;</a>';
  echo '</td><td colspan="5" class="ciemny center">';
  echo $miesiace[$miesiac-1].' '.$rok.'</td><td align="right">';
  echo '<a href="kalendarz.php?ile='.($ile+1).'&amp;p='.$p.'">&gt;&gt;</a>';
  echo '</td></tr>';
  echo '<tr class="ciemny"><td>Nd</td><td>Pn</td><td>Wt</td><td>Śr</td>
		<td>Cz</td><td>Pt</td><td>So</td></tr>';
  echo '<tr align="right">';
  if ($start>0) echo '<td colspan="'.$start.'" class="ciemny">&nbsp;</td>';
  else $pierwszy=true;
  for ($i=1;$i<=$iledni;$i++) {
	$dzien = date("w", mktime(0, 0, 0, $miesiac, $i, $rok));
	if ($dzien==0 and !$pierwszy) echo '</tr><tr align="right">';
	if ($dzien==0) $bg = "niedziela"; else $bg = "jasny";
	$pierwszy = false;
	if (mktime(0, 0, 0, date("m"), date("d"), date("Y"))==
		mktime(0, 0, 0, $miesiac, $i, $rok)) $bg .= " dzis";
	$d = sprintf("%02d", $i);
	echo '<td class="'.$bg.'">';
	echo '<a href="#"
	 onclick="opener.document.getElementById('f1').'.$p.'.value='';
	echo $rok.'-'.$miesiac.'-'.$d.''; window.close();">'.$i.'</a></td>';
  }
  $koniec=7-($start+$iledni)%7;
  if ($koniec<>7) echo '<td colspan="'.$koniec.'" class="ciemny">&nbsp;</td>';
  echo '</tr></table>';
}
kalendarz($_GET["ile"], $_GET["p"]);
?>
</body></html>
[PHP] pobierz, plaintext

daiala - testowalem
i pole $_POST['data'] można bezpośrednio zapisać do bazy jeśli format zapisu danego pola todata
jak coś nie dziala to daj znać

skrypt znaleźiony jakiś czas temu na google ale już nie pamietam gdzie - nie jest mojego autorstwa

nospor

9.05.2006, 15:59:41

Cytat

A jak taki input by wygladal, zeby bylo tak jak piszesz w formacie rrrr-mm-dd?

Jest to najwyzkleszy w swiecie input type="text" do ktorego mozesz wpisac dowolny text. ty tylko potem ten tekst musisz zwalidowac, czy jest odpowiedniego formatu. Jak pisalem mnostwo tego bylo na forum, na sieci. wystraczy przejrzec.

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.