הוסטס - פורום אחסון האתרים הגדול בישראל

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

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

PHP קוד:


		
			
<?php

// Your code

$query_4 = mysql_query("SELECT * FROM businesses WHERE category = '$id' AND sub_category = '2'");



// My suggestion



// Build the query

$sqlGetBusinesses = <<<SQL

SELECT    *

FROM    `businesses`

WHERE    (`category` = %d) AND

    (`sub_category` = 2)

LIMIT    20

SQL;



// Add the parameters

$sqlGetBusinesses = sprintf($sqlGetBusinesses, $id);



// Execute the query

$qryGetBusinesses = @mysql_query($sqlGetBusinesses);



// Handle errors

if (!$qryGetBusinesses)

{

    // error stuff here

}

else

{

    // Processing here



    // Free memory

    mysql_free_result($qryGetBusinesses);

}

?>

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

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

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

ועוד דבר בPHP, כמו ברוב שפות העילית יש נושא של short-circuit evaluation. מה שזה אומר זה שהתנאי יבדק רק עד איפה שצריך כדי לדעת אם התוצאה שלו היא true או false. אני מציע לך לקרוא עוד קצת על האופרטורים באתר הרישמי.
http://php.net/manual/en/language.operators.logical.php
אם היית מחליף את הסדר של הבדיקות בלולאה הפנימית שלך היית חוסך קריאה מיותרת ל mysql_fetch_array.

ובסופו של דבר, כדאי לך לקרוא עם mysqli. מגיע היום בכל התקנה של PHP5 והוא טוב יותר מהפלאגין הרגיל של mysql.

מקווה שעזרתי במשהו..