אני צריך הסבר

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

הנה הפונקציה שלא עבדה לי:




function isProper(){
var iChars = "*|,":<>[]{}`';()@&$#%^&!=~)({}><;/-";

for (var i = 0; i < string.length; i++) {
if (iChars.indexOf(string.charAt(i)) != -1)
return false;
}
return true;
}


דיברתי עם כמה אנשים והם אמרו לי לעשות כמה תיקונים.הנה הפונקציה שעובדת:




function isProper(string){
if (!string) return false;
var iChars = "*|,":<>[]{}`';()@&$#%^&!=~)({}><;/-";

for (var i = 0; i < string.length; i++) {
if (iChars.indexOf(string.charAt(i)) != -1)
return false;
}
return true;
}



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