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

הדרך האחרונה שיצרתי היא שימוש ב switch(TRUE) {......}
לדוגמא:

קוד PHP:

 switch (TRUE) {
        case !isset($_REQUEST['registerUser']):
        case !is_string($_REQUEST['registerUser']):
        $report_error[] = 'please specify a user name.';
        break;
        case strlen($_REQUEST['registerUser']) < $_CONFIG['REGISTER']['minLengthUser']:
        $report_error[] = 'please specify a longer user name. (between <b>'.$_CONFIG['REGISTER']['minLengthUser'].' to '.$_CONFIG['REGISTER']['maxLengthUser'].' characters)';
        break;
        case strlen($_REQUEST['registerUser']) > $_CONFIG['REGISTER']['maxLengthUser']:
        $report_error[] = 'please specify a shorter user name. (between <b>'.$_CONFIG['REGISTER']['minLengthUser'].' to '.$_CONFIG['REGISTER']['maxLengthUser'].' characters)';
        break;
        case !preg_match('%^[A-Za-z0-9]+$%', $_REQUEST['registerUser']):
        $report_error[] = 'please specify a user name containing only characters and numbers. (<b>a-z,A-Z,0-9)';
        break;
    } 


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

אתה משתמש בקידוד multibyte כמו UTF-8?

אם כן, הפונקציה strlen אינה מתאימה במקרה שמופיעים תוים שאינם בתחום ה-ASCII.
יש צורך להשתמש בפונקציות mb כמו mb_strlen (אל תשכח להגדיר קידוד בפונקציית ה-mb המתאימה)

ואני משתמש בצורה (הרגילה) הזאת:

קוד PHP:

 if ( expr1 op1 expr2 op2 expr3 ... )

// op = operator
// expr = expression 


PHP בודק את הערכים בתוך תנאי ה-if משמאל לימין, וכאשר הוא נתקל בביטוי שהוא שקר, הוא לא יבדוק את ה- expressions הבאים.

ציטוט:

במקור נכתב על ידי dorM אתה משתמש בקידוד multibyte כמו UTF-8? אם כן, הפונקציה strlen אינה מתאימה במקרה שמופיעים תוים שאינם בתחום ה-ASCII. יש צורך להשתמש בפונקציות mb כמו mb_strlen (אל תשכח להגדיר קידוד בפונקציית ה-mb המתאימה) ואני משתמש בצורה (הרגילה) הזאת: קוד PHP:  if ( expr1 op1 expr2 op2 expr3 ... ) // op = operator // expr = expression  PHP בודק את הערכים בתוך תנאי ה-if משמאל לימין, וכאשר הוא נתקל בביטוי שהוא שקר, הוא לא יבדוק את ה- expressions הבאים.

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

משתמש בכמה if'ים...
יש מקרים שזה בקושי משפיע על הריצה, אבל אולי יש מקרים שבהם תעדיף לייעל גם שם...
אני חושב שבעזרת switch, כמו שעשית, זה הכי עדיף שיש.

דרך אגב עדיף לייעל במקרה הזה את השימוש בפונקציה strlen, שבקוד שמוצג השתמשת בה פעמיים על אותו המשתנה. היה עדיף להריץ פעם אחת ולשמור במשתנה זמני את התוצאה.

ציטוט:

במקור נכתב על ידי dorM משתמש בכמה if'ים... יש מקרים שזה בקושי משפיע על הריצה, אבל אולי יש מקרים שבהם תעדיף לייעל גם שם... אני חושב שבעזרת switch, כמו שעשית, זה הכי עדיף שיש. דרך אגב עדיף לייעל במקרה הזה את השימוש בפונקציה strlen, שבקוד שמוצג השתמשת בה פעמיים על אותו המשתנה. היה עדיף להריץ פעם אחת ולשמור במשתנה זמני את התוצאה.

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

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