כלומר, יש לי מערך של נתונים ואני רוצה שכל נתון במערך יכנס לפי הסדר לשמות השדות שאני אתן.
למשל המערך:
array(1,2,3)
יכנס אל:
field_1,field_2,field_3
לפי סדר הנתונים במערך.

תודה לעוזרים.

קוד PHP:

<?php
function query_insert($table ,$array ,$mysql_link ,$appendix = '' ,$debug = false) {
    $query = "INSERT INTO `".$table."` ";

    $into = "(";

    $insert = "(";

    foreach($array as $into_k => $value) {

        $into .= "`".$into_k."`,";

        $insert .= "'".$value."',";

    }

    $into{strlen($into)-1} = ")";

    $insert{strlen($insert)-1} = ")";



    $query .= $into." VALUES ".$insert." ".$appendix.";";

    $query_bool = mysql_query($query ,$mysql_link);

    if($debug == true)
        echo $query;
        

    if(!$query_bool)
        return mysql_error();

return 0;
}
?>

אתה זורק ל$array:

קוד PHP:

 $array = array("feild1" => "data1" ,"feild2" => "data2"); 


GNU/Linux - Use the source!

[התמונה הבאה מגיעה מקישור שלא מתחיל ב https ולכן לא הוטמעה בדף כדי לשמור על https תקין: http://petition.mac-it.co.il/promote/buttons/petition_text_less.gif]

הפונקציה שלך נחמדה, אך אפשר לעשות אותה הרבה יותר עדינה ויפה, לדעתי.
הנה איך שאני הייתי מדמיין את הקוד הזה (אם בתוך פונקציה או סתם זרוק בקוד):

קוד PHP:

 ...
$fields = "`" . implode("`, `", array_keys($array)) . "`";
$values = "'" . implode("', '", $array) . "'";
 
$query = mysql_query("INSERT INTO `my_table` ($fields) VALUES ($values);");
... 


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

אתה צודק אבל שלי באה כחלק אינטגרלי מקלאסס שכולל בתוכו את הפונקציות SELECT UPDATE INSERT DELETE
אני משתמש בקלאס ולא כותב שיאלתות בכלל והוספתי פיצרים בשביל דיבאג בגלל שמתי שיש לך 20 קלאסים ואתה לא בידיוק יודע איפה הבעיה זה מפשט את העיניין למעשה חיבור השדות שלך הרבה יותר "סקסי" בעיניי אני מייד משנה אצלי תודה רבה...
יש לציין שהפרמטר
mysql_link
במקור לא מועבר בהגדרה של הפונקצייה אלא :

קוד PHP:

 $this->mysql_link[$connection_id] 


GNU/Linux - Use the source!

[התמונה הבאה מגיעה מקישור שלא מתחיל ב https ולכן לא הוטמעה בדף כדי לשמור על https תקין: http://petition.mac-it.co.il/promote/buttons/petition_text_less.gif]

חחח.. קלאס זה סקסי..

כן, מחלקה זה קלאסי, אבל לא עד כדי כך..
יש גבול לכמה שאפשר לפשט דברים, סתם חוסר יעילות לעשות כל-כך הרבה פונקציות - פונקציה אחת להוספה, פונקציה אחת למחיקה וכו', במחלקת DB זה פשוט לא יעיל.

EVERYTHING SHOULD BE MADE AS SIMPLE AS POSSIBLE, BUT NOT ONE BIT SIMPLER
ALBERT EINSTEIN

למרות שזה פחות יעיל, ברגע שזה מוכן זה פחות פגיע לשגיאות, פחות חיפוש טעוית = פחות זמן פיתוח. ובמקרה הזה, הפסד היעילות הוא ממש לא קריטי.

אני לא רואה איך זה יותר נוח. מה שאני רואה פה זה קוד פחות קריא, כאשר רובו זה קוד קבוע שאם לא הייתה שגיאה בפעם הראשונה, לא צריכה ליהיות שגיאה בכל ההרצות שאחריה, ואם כבר יש שגיאה בשאליתא עצמה, איך הפונקציה הזאת אמורה לעזור לי לאתר את הבעיה? mysql_error אמור לעזור לי לאתר את הבעיה..

אני לא נגד מחלקות, אבל אני מוצא את זה נוח מסיבות אחרות מאלו שציינתם פה, שנראות לי לא בהכרח קשורות. מחלקה נוחה לשם יצירת שינויים מידיים בקוד ארוך, בבת אחת, בדומה לפונקציות בודדות, עם הנוחיות שבאה עם מחלקה בניגוד לפונקציות מתואמות (כל הטריקים הנחמדים של פונקציות, משתנים, אובייקטים וכו').

היתרון במחלקות מבחינת כתיבת קוד, הוא שהקוד יותר אינטואיטיבי, מאפשר חלוקה מודולרית של התוכנית, ואפשר לטפל בחלקים באופן עצמאי. אם למי שכותב את הקוד נוח יותר לראות

קוד PHP:

 $db->insert(...); 


ולא להריץ שאילתות, זה בחירה שלו.

חוץ מזה, שדווקא במקרה הזה יכולים להיות עוד שיקולים חוץ מ-"האם הייתה שגיאה"...

חוץ מזה, לא דווקא במקרה הזה, אבל הרבה פעמים יש שוניות כאשר בונים שאילתות למסדים נתונים שונים, ומבנה המחלקה הזה מאפשר למתכנת לא לחשוב על זה בכלל ברגע שהוא סיים לבנות את המחלקה שתואמת את מסד הנתונים.

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

כמובן שמצד שני, ככל שיש יותר יותר מחלקות / שיטות / פונקציות הקוד פחות יעיל, ולכן אני אישית מעדיף לעשות סוג של "אופטימיזציה" לקוד שלי כשאני מסיים.

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

נכון, יש אנשים שיותר נוח להם לקרוא את זה ככה, אבל אני באמת לא רואה את ההבדל הגדול בין לרשום:

קוד PHP:

 $db->insert("QUERY"); 


לבין

קוד PHP:

 $db->query("INSERT INTO QUERY"); 


נראה לי סתם מיותר לבנות כל כך הרבה פונצקיות שעושות פעולה שהמתמש יכול היה לפתור בחוסר עצלנות של כתיבת שתי מילים. ואם כבר, אז אני הייתי עושה פרמטר שדרכו אפשרי להעביר מספר ביקורת, או טקסט קבוע מראש שמכריז על סוג השאליתא.

בקשר לתאימות של מסדים, בדיוק לזה התכוונתי שאמרתי שנוי מידי. שינוי בסוג המסד, בצורת הפלט של השגיאות, באירגון התוצאות (אם זה ע"י השמת MYSQL_NUM, לדוגמא).

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

המחלקה שלו פשוט יותר מדי, לעשות פונקציה במיוחד לINSERT?! אני אישית חושב שזה ממש לא יעיל.
2. כן, גם אני עשיתי מחלקה (נראה לי שפרסמתי פה) שכוללת, הודעות שגיאה מותאמת אישית,
בחירה בין לשלוף את כל הנתונים או לשלוף רק שורה אחת, בחירה, איך להוציא את הנתונים (assoc, array, object) וכדו'.
גם כן שהוספתי אפשרות נחמדה לשאילתה - sprintf.
אפשר לשאול מה בדיוק עשית בMYSQL_ASSOC (איזו תוספת)?

EVERYTHING SHOULD BE MADE AS SIMPLE AS POSSIBLE, BUT NOT ONE BIT SIMPLER
ALBERT EINSTEIN

אני רואה את זה ביזבוז בלכתוב מחדש פור וחוצמזה כתבתי פונקציה לכל דבר כי כל שאילתה מסודרת אחרת...

GNU/Linux - Use the source!

[התמונה הבאה מגיעה מקישור שלא מתחיל ב https ולכן לא הוטמעה בדף כדי לשמור על https תקין: http://petition.mac-it.co.il/promote/buttons/petition_text_less.gif]

בדיוק מה שאני אומר!
שזה כבר לא יעיל לעשות כל-כך הרבה פונקציות.
מה הבעיה לרשום את השאילתה?! SQL זו לא שפה מסובכת.
זו כבר הגזמה לעשות לכל דבר קטן פונקציה.

EVERYTHING SHOULD BE MADE AS SIMPLE AS POSSIBLE, BUT NOT ONE BIT SIMPLER
ALBERT EINSTEIN

קוד PHP:

 #!/usr/bin/php -q
<?php

$array = array();

for($i=0;$i<1000;$i++)
        $array[$i] = chr($i);

$start_for = microtime();
        foreach($array as $into_k => $value) {
                $into .= "`".$into_k."`,";
                $insert .= "'".$value."',";
        }
        $into{strlen($into)-1} = ")";
        $insert{strlen($insert)-1} = ")";
$end_for = microtime();

$start_implode = microtime();
        $fields = "(`" . implode("`, `", array_keys($array)) . "`)";
        $values = "('" . implode("', '", $array) . "')";
$end_implode = microtime();

echo $for_time = $end_for - $start_for ."\n";
echo $implode_time = $end_implode - $start_implode;
?> 


יצא ששלך יותר מהיר:

קוד:

leon@Dserv2:~$ ./benchmark.php
0.005009
0.004319
leon@Dserv2:~$ ./benchmark.php
0.005102
0.00456
leon@Dserv2:~$ ./benchmark.php
0.005076
0.00439
leon@Dserv2:~$ ./benchmark.php
0.005108
0.004352
leon@Dserv2:~$ ./benchmark.php
0.005219
0.005707
leon@Dserv2:~$ ./benchmark.php
0.005131
0.004304
leon@Dserv2:~$ ./benchmark.php
0.005027
0.005067
leon@Dserv2:~$

עברתי לשיטה שלך...

GNU/Linux - Use the source!

[התמונה הבאה מגיעה מקישור שלא מתחיל ב https ולכן לא הוטמעה בדף כדי לשמור על https תקין: http://petition.mac-it.co.il/promote/buttons/petition_text_less.gif]