רוטר - מה לא טוב בקוד PHP שעשיתי ?

יומן פעילות	לוח שנה	מבזקי חדשות	תקנון	RSS	כותרות	כותרות לפי תגובה	ניקוי קוקיז	IP	להורדת אפליקציה
eBay	AliExpress	GearBest	Amazon	Booking	Kiwi	SkyScanner	Trip Advisor

"מה לא טוב בקוד PHP שעשיתי ?"

גירסת הדפסה

קבוצות דיון בניית אתרים נושא #10222	מנהל סגן המנהל מפקח Winner צל"ש מומחה

אשכול מספר 10222

גלית
חבר מתאריך 15.11.16
487 הודעות, 2 פידבק

23:11 17.07.11

מה לא טוב בקוד PHP שעשיתי ?

עבר עריכה לאחרונה בתאריך 18.07.11 בשעה 23:11:51 על-ידי Ice Cold (מנהל הפורום)

<HTML>
<HEAD>
<TITLE>
</TITLE>
</HEAD>
<BODY>
<CENTER>
<INPUT TYPE="TEXT" ID="NUMBER">
<form action="11.php method="GET">
<input type="submit" value="שנה">
<?php
$connection = mysql_connect("localhost", "root", "")
or die ("Couldn't connect to server");
$db = mysql_select_db("produce",$connection)
or die ("Couldn't select database");

$query = "UPDATE fruit SET number=isset($_GET['NUMBER']) WHERE name = 'pears'";
$result = mysql_query($query)
or die("Query faild: ".mysql_error());
?>
</body>
</html>

יש את השגיאה הזו:
Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in D:\Xampp\htdocs\12.php on line 18

שתף

מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

האשכול	מחבר	תאריך כתיבה	מספר
אוקי אז ככה,	asco88	17.07.11 23:56	1
תודה רבה על התשובה וההסבר :) , אך יש לי שגיאה נוספת	גלית	18.07.11 00:45	2
שימי לב	asco88	18.07.11 06:56	3
תשתמשי ב XHTML VALIDATOR למצוא שגיאות ב HTML	Sylver	18.07.11 20:42	4
תודה :)	גלית	19.07.11 00:10	5

asco88
חבר מתאריך 17.6.04
26757 הודעות

23:56 17.07.11

1. אוקי אז ככה,
בתגובה להודעה מספר 0

שימי לב שכאן השגיאה:

$query = "UPDATE fruit SET number=isset($_GET) WHERE name = 'pears'";
עשית שני דברים בשורה אחת, יצרת משתנה מסוג מחרוזת המכיל את השאילתה, וגם בדקת האם משתנה מוגדר.
אי אפשר לעשות את זה ככה, חייבים קודם לבדוק ורק אח"כ להציב במחרוזת.
למשל:

if (isset($_GET{'fruits'})
$new_number = $_GET{'fruits'};

$query = "UPDATE fruit SET number = '$new_number' WHERE name = 'pears'";
$result = mysql_query($query)
or die("Query faild: ".mysql_error());
*את הסוגריים המסולסלים יש להחליף במרובעים
שימי לב שבדקתי את המשתנה ורק לאחר שהצבתי אותו בתוך משתנה הצבתי אותו בתוך השאילתה.
דבר נוסף, שימי לב שלא ניתן לפנות למשתנה מסוג GET בלי להתייחס למיקום במשתנה.

בהצלחה.

(ניהול: מחק תגובה)

מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

גלית
חבר מתאריך 15.11.16
487 הודעות, 2 פידבק

00:45 18.07.11

2. תודה רבה על התשובה וההסבר :) , אך יש לי שגיאה נוספת
בתגובה להודעה מספר 1

משום מה.

<HTML>
<HEAD>
<TITLE>
</TITLE>
</HEAD>
<BODY>
<CENTER>
<INPUT TYPE="TEXT" ID="NUMBER">
<form action="11.php method="GET">
<input type="submit" value="שנה">
<?php
$connection = mysql_connect("localhost", "root", "")
or die ("Couldn't connect to server");
$db = mysql_select_db("produce",$connection)
or die ("Couldn't select database");
if (isset($_GET)
$new_number = $_GET;

$query = "UPDATE fruit SET number = '$new_number' WHERE name = 'pears'";
$result = mysql_query($query)
or die("Query faild: ".mysql_error());
?>
Parse error: syntax error, unexpected T_VARIABLE in D:\Xampp\htdocs\12.php on line 19
החלפתי את הסוגריים

(ניהול: מחק תגובה)

מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

asco88
חבר מתאריך 17.6.04
26757 הודעות

06:56 18.07.11

3. שימי לב
בתגובה להודעה מספר 2

שבטופס שלך לא סגור אחד הגרשיים:

<form action="11.php method="GET">
איפה שאת קוראת לעמוד 11.php הגרשיים נשארו פתוחים ולכן שאר הטופס נחשב המשך.

חוץ מזה שלא סגרת את הטופס </form> ושמת input אחד לפני התחלת הטופס, שזה לא טוב.

(ניהול: מחק תגובה)

מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

Sylver
חבר מתאריך 1.7.02
11915 הודעות, 2 פידבק

20:42 18.07.11

4. תשתמשי ב XHTML VALIDATOR למצוא שגיאות ב HTML
בתגובה להודעה מספר 0

(ניהול: מחק תגובה)

מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

גלית
חבר מתאריך 15.11.16
487 הודעות, 2 פידבק

00:10 19.07.11

5. תודה :)
בתגובה להודעה מספר 4

(ניהול: מחק תגובה)

מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד