יש לי טבלה בשם flash_news.
שמות העמודותד הינם: id, date, time, update_by, category, title, flash

אני מנסה לבנות טופס לעדכון הטבלה (UPDATE לא INSERT).
בטופס יש שדה שבו רושמים באיזה ID רוצים לבצע את העכדון, ושדות נוספים שממלאים רק במקרה שמעוניינים לעדכן שדה זה. (category, title, flash)

היות ואי אפשר לדעת איזה שדה יעדכנו, כתבתי את הקוד כך:

קוד PHP:

 if($_GET['update_by'])
 {
     $update_by="update_by=".$_POST["update_by"].",";
 }
 if($_GET['category'])
 {
     $category="category=".$_POST["category"].",";
 }
 if($_GET['title'])
 {
     $title="title=".$_POST["title"].",";
 }
 if($_GET['flash'])
 {
     $flash="flash=".$_POST["flash"].",";
 }
 
 $sql=    "
     UPDATE news_flash SET "
     .$update_by
     .$category
     .$title
     .$flash
     ." WHERE id=".$_POST['id']
     ; 


ניסיתי לעדכן, וקיבלתי את השגיאה

קוד:

 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 'WHERE id=9' at line 1
 

כמו שהנכם בוודאי יודעים, לפי התחביר לא כותבים פסיק (,) רק אם יש שדה נוסף שרוצים לעדכן. ובקוד שכתבתי יש פסיק גם לאחר תוכן השדה האחרון ("column=content,").
האם זה מהווה בעיה ?
(לפי השגיאה אני מבין שיש בעיה אחרת).

אשמח גם לשמוע איך אוכל לכתוב קוד יעיל יותר.

לצורך דיבוג הקוד תוסיף הדפסה של תוכן המשתנה sql לפני שאתה שלוח אותו לדטהבייס וכך תראה מה הכנת לו.

בנתיים נראה שיש פסיק מיותר בשורה:

קוד PHP:

 $flash="flash=".$_POST["flash"].","; 


למעשה אתה כותב:

קוד:

UPDATE news_flash SET update_by=something,  category=soemthing,  title=something,flash=something, WHERE id=something;

אז אתה מקבל שגיאה שליד ה WHERE יש טעות

_________________________________________________

אזהרה: משרד הבריאות קובע כי העישון מזיק לבריאות !
תראו, אפילו החייזר נהיה ירוק מזה

מנהל פורום מדע בדיוני ופנטסיה ופורום מדע טכנולוגיה וטבע בפרש

רשמת לו WHERE id=something כאשר הID מן הסתם הוא INT אז תעשה WHERE id=1

קוד PHP:

 $id = $_POST['id'];
$sql = "UPDATE news_flash SET update_by='$update_by' , catagory='$category' , title='$title' , flash='$flash' WHERE id='$id' "; 


תודה לכולם, השאלה אם אני יכול לכתוב את השאילתה בצורה כזאת:

קוד PHP:

 $sql = "UPDATE news_flash SET update_by='$update_by' , catagory='$category' , title='$title' , flash='$flash' WHERE id='$id' "; 


כשיתכן שחלק מהמשתנים רקים, ואני גם לא רוצה לעדכן אותם !
במילים אחרות, האם התחביר הבא תקין ב-MySQL.

קוד:

 UPDARE news_flash SET title=title, flash='' WHERE id=9
 

1. האם התחביר חוקי ?
2. זה ישנה את התוכן של title ו-flash ?
3. זה ישנה רק את התוכן של title ?

לא ידעתי שאין בכוונתך לעדכן שדות מבלי ערכים שכן זה יאפס את השדות, הנה הקוד שתעשה:

קוד PHP:

 $id = $_POST['id'];
$resultset = mysql_query ("SELECT * FROM news_flash WHERE id='$id' " , $linkid) or die (mysql_error($linkid));
while(list( $id2, $date2, $time2, $update_by2, $category2, $title2, $flash2 ) = mysql_fetch_rows($resultset)) {
 if (!isset($date))
  $date = $date2;
 if (!isset($time))
  $time = $time2;
 if (!isset($update_by))
  $update_by = $update_by2;
 if (!isset($catagory))
  $catagory = $catagory2;
 if (!isset($title))
  $title = $title2;
 if (!isset($flash))
  $flash = $flash2;

 $sql = "UPDATE news_flash SET update_by='$update_by' , catagory='$category' , title='$title' , flash='$flash' WHERE id='$id' ";
} 


שיניתי את הפקודות, אני משרש הכל למשתנה אחד, ובמקרה שכבר קיים שדה לתיקון אני מוסיף פסיק, אם לא אני לא מוסיף פסיק.
כך שלאחר השדה האחרון לא יהיה פסיק.

קוד PHP:

 if($_GET['update_by'])
   {
       $sql0="update_by=".$_POST["update_by"];
   }
   if($_GET['category'])
   {
       if($sql0){$sql0=$sql0.','}
       $sql0=$sql0."category=".$_POST["category"];
   }
   if($_GET['title'])
   {
       if($sql0){$sql0=$sql0.','}
       $sql0=$sql0."title=".$_POST["title"];
   }
   if($_GET['flash'])
   {
       if($sql0){$sql0=$sql0.','}
       $sql0=$sql0."flash=".$_POST["flash"];
   }
   
   $sql="UPDATE news_flash SET ".$sql0." WHERE id=".$_POST['id']; 


חשבתי גם אולי להוסיף תמדי את הפסיק (כמו שעשיתי בתחילה), ורק אח"כ להריץ בדיקה אם התו האחרון במחרוזת הוא פסיק, ואם כן, להסירו.
איך אני מבצע זאת ?

אם מה שרשמתי לך לא מתאים לך אז פשוט תעביר את הכל בתוך מערך ותרוץ רק על התאים שמחזירים לך TRUE...

$flash['cell']

תודה לכולכם !

ב"ה סיימתי לגהץ את הבאגים, הקוד המתוקן:

קוד PHP:

 if($_POST['update_by'])
 {
     $sql0="update_by='".$_POST["update_by"]."'";
 }
 if($_POST['category'])
 {
     if($sql0){$sql0=$sql0.',';}
     $sql0=$sql0."category='".$_POST["category"]."'";
 }
 if($_POST['title'])
 {
     if($sql0){$sql0=$sql0.',';}
     $sql0=$sql0."title='".$_POST["title"]."'";
 }
 if($_POST['flash'])
 {
     if($sql0){$sql0=$sql0.',';}
     $sql0=$sql0."flash='".$_POST["flash"]."'";
 }
 
 $sql="UPDATE news_flash SET ".$sql0." WHERE id=".$_POST['id'];