האם אתם -באמת- מתכנתים כמו שצריך?

[BlueNosE]

שלום, אני רוצה לדבר איתכם על נושא בתכנות שנראה לכם כמובן מאליו.
במקרה הזה אני אדבר על PHP, אבל זה תקף לכל השפות באינטרנט - JS, ASP וכו'.

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

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

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

קוד בPHP שיראה כך:

PHP קוד:

if ($_GET['id'] == 3)
echo "addfh"; 


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

הסינטקס הנכון הוא

PHP קוד:

if ((!empty ($_GET['id'])) and ($_GET['id'] == 3))
/*OR*/
if ((isset ($_GET['id'])) and ($_GET['id'] == 3)) 


ואני שואל - האם אתם באמת מתכנתים ככה?
כדי לבדוק את זה, תכתבו בתחילת הקוד
error_reporting (E_ALL);

[RS324]

חשבת להגדיר EROR_REPORTING במערכת ? ככה שבכל אופן גם אם יש NOTICE
הוא יתעלם מהם ?

[BlueNosE]

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

חשבת להגדיר EROR_REPORTING במערכת ? ככה שבכל אופן גם אם יש NOTICE
הוא יתעלם מהם ?

חשבתי, והחלטתי שהרבה יותר חכם ונכון לתכנת בסינטקס נכון.

[Derey22]

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

חשבתי, והחלטתי שהרבה יותר חכם ונכון לתכנת בסינטקס נכון.

בסינטקס נכון,אסור לך להשתמש בסוג של הif המרובה

PHP קוד:

if  )(
}
{ 


אז איך אני יכול להחליף אותו?

[BlueNosE]

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

בסינטקס נכון,אסור לך להשתמש בסוג של הif המרובה

PHP קוד:

if  )(
}
{ 


אז איך אני יכול להחליף אותו?

מה?? לא הבנתי. תסביר.

1. לא ביטלת את ה ERROR_REPORTING שיתעלם מ NOTICE
כי NOTICE זה לא בעצם שגיאה זה רק אומר לך שעשית שימוש במשהו שהוא לא קיים...
2. לגבי ה REGISTER_GLOBAS לא הבנתי מה הקשר לאתחול המשתנה של ה GET
3. אם זה באמת חשוב לך אני משתמש ב ERROR_HANDLER משלי...ככה שזה לא משפיע עלי בכל מקרה...

1. נו ברור שלא, בגלל זה אני אומר שכדאי לא להסתמך על זה.
2. אמרתי שאני אף פעם לא משתמש ב$var אלא תמיד ב$_GET['var'].
3. איך יצרת ERROR HANDLER? אפילו בSQL לא הצלחתי ליצור אחד שנותן יותר מmysql_error().
הייתי שמח לפירוט / הסבר קטן.

[BlueNosE]

^^^
לא מדבר על HTML בהקשר כלשהו.

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

חשבת להגדיר EROR_REPORTING במערכת ? ככה שבכל אופן גם אם יש NOTICE
הוא יתעלם מהם ?

אגב, תעשה סקר ותברר מי מאלה שלא כותב בסינטקס נכון משתמש בERROR_REPORTING, ותגיע לסכום ששואף ל0.

[RS324]

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

^^^
לא מדבר על HTML בהקשר כלשהו.


אגב, תעשה סקר ותברר מי מאלה שלא כותב בסינטקס נכון משתמש בERROR_REPORTING, ותגיע לסכום ששואף ל0.

ותעשה סקר כמה אנשים פה מתמודדים ע register_globals ועם magic_qoutes ותגיע גם לסכום השואף ל 0

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

PHP קוד:

if($_GET['id'] == 3)
{
echo 'hey fucker';
} 


זה יעבוד בכל שרת... מניסיון...

[BlueNosE]

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

אם אני עושה

PHP קוד:

if($_GET['id'] == 3)
{
echo 'hey fucker';
} 


זה יעבוד בכל שרת... מניסיון...

http://www.hill.4save.net/Huh.php
הקוד שלך מועתק 1:1, עם הוספת תווי <?php ?>.

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

בקשר לMAGIC QUOTES, אני משתמש במשהו אחר בשביל לבצע פעולת סינון.

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

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

[RS324]

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

http://www.hill.4save.net/Huh.php
הקוד שלך מועתק 1:1, עם הוספת תווי <?php ?>.

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

בקשר לMAGIC QUOTES, אני משתמש במשהו אחר בשביל לבצע פעולת סינון.

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

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

1. לא ביטלת את ה ERROR_REPORTING שיתעלם מ NOTICE
כי NOTICE זה לא בעצם שגיאה זה רק אומר לך שעשית שימוש במשהו שהוא לא קיים...
2. לגבי ה REGISTER_GLOBAS לא הבנתי מה הקשר לאתחול המשתנה של ה GET
3. אם זה באמת חשוב לך אני משתמש ב ERROR_HANDLER משלי...ככה שזה לא משפיע עלי בכל מקרה...

[nevo]

תמיד מבצע בדיקה לGET

$Id = (isset( $_GET['Id']))?intval( $_GET['id']):0;