[PHP] פרסום פונקציה חמודה ביותר... ממליץ להשתמש!

[reXtea]

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

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

PHP קוד:

function ISnumber($vars)
{
   foreach(explode(";",$vars) as $k => $val)
   {
      if(!is_numeric($val))
      {
         die("ERROR");
      }
      else
      {
         return true;
      }
   }
}

ISnumber("id;page;number"); 


זה הפונקציה... לשימושכם החופשי.
הפונקציה לא נבדקה ככה שלא בטוח שהיא עובדת עם השיטה של הexplode אני לא ממש בטוח ב100% שזה יעבוד... אבל נראה לי שכן D:

בכ"מ הרגשתי צורך לפרסם את זה... לדע למה.
תהנו :]

[4ior]

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

[Daniel]

עדיף שיהיה INPUT של ARRAY,
explode + implode לוקחים יחסית דיי הרבה משאבים

[reXtea]

נכתב במקור על ידי 4ior

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

מ"ז להכניס את כל המשתנים? זה בסך הכל עושה בדיקה שמשתנים שאמורים להיות רק מספרים לא יהיו תווים.

והיא מאוד שימושית, ויעילה גם כן... ואממ אם זה לוקח יותר זמן מאשר explode אז באמת עדיף מערך.

[-roee-]

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

מציע לך לעבור על מדריכי אבטחה שונים ואז תבין את היעילות הרבה שאין להחליף אותו בשום דבר זול של הפונקציה is_numeric או intval

[Tomer]

PHP קוד:

function checkForNumbers($inputArray,$excludeArray="") {
    foreach ($inputArray AS $key=>$val) {
        if (!$excludeArray[$key] && !is_numeric($inputArray[$key])) $inputArray[$key] = intval($inputArray[$val]);
    }
} 


אני מאמין שזו דרך נחמדה ואפשרית לשימוש.

$Exclude = array('subject','body');
checkForNumbers($_POST,$Exclude);

המערך Exclude מכיל את הרשימה של משתנים שלא צריכים להבדק. כמובן שאפשר לא להעביר את המערך ובכך לבדוק את כל המערך שניתן (הדוגמא כאן - $_POST)

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

[reXtea]

נכתב במקור על ידי roeenoy

מציע לך לעבור על מדריכי אבטחה שונים ואז תבין את היעילות הרבה שאין להחליף אותו בשום דבר זול של הפונקציה is_numeric או intval

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

נכתב במקור על ידי Tomer

PHP קוד:

function checkForNumbers($inputArray,$excludeArray="") {
    foreach ($inputArray AS $key=>$val) {
        if (!$excludeArray[$key] && !is_numeric($inputArray[$key])) $inputArray[$key] = intval($inputArray[$val]);
    }
} 


אני מאמין שזו דרך נחמדה ואפשרית לשימוש.

$Exclude = array('subject','body');
checkForNumbers($_POST,$Exclude);

המערך Exclude מכיל את הרשימה של משתנים שלא צריכים להבדק.

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

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