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

הוסטס - פורום אחסון האתרים הגדול בישראל (https://hosts.co.il/forums/index.php)
-   פורום תיכנות (https://hosts.co.il/forums/forumdisplay.php?f=14)
-   -   האם הקידוד עומד בתקן? (https://hosts.co.il/forums/showthread.php?t=92682)

eliran2313 11-10-11 21:02

הוא לא שינה כלום, הכול עדיין בתוך טבלה.

אדיר 11-10-11 21:36

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

אלעד:
זה שהעמוד עומד בתקן HTML4,
שווה במקרה הזה לעגלת זהב בלי סוס.

Haimz 11-10-11 21:40

חוץ מזה שרמת הקוד מאוד נמוכה, האתר לא נראה לי טוב ..

elad-o 11-10-11 21:53

ציטוט:

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

אלעד:
זה שהעמוד עומד בתקן HTML4,
שווה במקרה הזה לעגלת זהב בלי סוס.

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

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

אשמח להשכיל קצת מבעלי ידע

BuildDream 11-10-11 22:08

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

בכל מקרה בקשר לקידוד יש לא מעט דברים שכדאי לשפר, כמו הpadding של הבלוקים - הטקסט צמוד מידי למסגרת.
במקום להשתמש באופציית הרקע בCSS שיכולה לשכפל תמונה כדי שתופיע לאורך כל האלמנט - שמתי לב שהמקודד פשוט מיקום את התמונה הזאת - http://www.ecom.freehost.co.il/kidud...s/kidud_31.gif ומתח אותה ומיקם אותה מספר פעמים לאורך האתר כדי להציג את הרקע.
מלבד שהשיטה לא טובה במיוחד ופחות דינמית, אפשר להסתפק באותה תמונה רק ברוחב של 1 פיקסל - דבר שיקטין משמעותית את התמונה (ותמונות רקע נוספות שבוצע בהן שימוש) ולחסוך בתעבורה ובזמן העליה של האתר.

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

IgalSt 11-10-11 22:33

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

ציטוט:

נכתב במקור על ידי almog12 (פרסם 821469)
טבלה לא נמצאת מחוץ לתקן ואין סיבה לא להשתמש בה.

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

DoLet 11-10-11 23:23

ציטוט:

נכתב במקור על ידי Rockets (פרסם 821455)
המתכנת שיפר את הקידוד וזו התוצאה הסופית:
http://www.ecom.freehost.co.il/kidud/kidud.html

אשמח לשמוע תגובות..

תקני מבחינת XHTML זה תקין מבחינת חיתוך ויעילות האתר פחות.
הרבה מדברים איך על קוד תקין מבחינת קידום אתרים שכמו שכבר אמרו זה לא החלק הכי חשוב!.

האתר הזה יכול לנצל לך הרבה מהתעבורה של השרת על סתם,
לדוגמא התמונה הזו: http://www.ecom.freehost.co.il/kidud...s/kidud_02.gif
שהיא לבנה לגמרי זה מיותר.
לא רק זה יש אתה התמונה הבא שהיה אפשר לחסוך אותה בחיתוך נכון:
http://www.ecom.freehost.co.il/kidud...s/kidud_40.gif
כנ"ל לגבי התמונות הבאות:
http://www.ecom.freehost.co.il/kidud...s/kidud_43.gif
http://www.ecom.freehost.co.il/kidud...s/kidud_42.gif
http://www.ecom.freehost.co.il/kidud...s/kidud_41.gif
http://www.ecom.freehost.co.il/kidud...s/kidud_74.gif
http://www.ecom.freehost.co.il/kidud...s/kidud_73.gif
http://www.ecom.freehost.co.il/kidud...s/kidud_76.gif

ואני לא חושב שבדקתי את הכל, כלכך הרבה תמונות מיותרות שמעמיסות על האתר.

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

אז בגדול האתר מקודד מבחינת XHTML תקין מבחינת יעילות חד משמעית לא!.
עדיף לשלם קצת יותר ולקבל הרבה הרבה יותר.

עריכה:
וכמו שאמרו כבר שימוש בDIV יותר יעיל ודינמי!.
מקווה שהצלחתי לעזור בהצלחה.

Rockets 11-10-11 23:34

תודה חברים.. אפשר לנעול.

BuildDream 12-10-11 00:29

ציטוט:

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


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

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

אבל במקרה של הקידוד שהוא הביא פה זה שימוש מוגזם שפוגע בדינמיות של האתר וביעילות של הקוד.

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

IgalSt 12-10-11 02:04

ציטוט:

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

אבל במקרה של הקידוד שהוא הביא פה זה שימוש מוגזם שפוגע בדינמיות של האתר וביעילות של הקוד.

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

התג table קיים ומבורך. עם זאת, מבחינתי הוא קיים אך ורק להצגת תוכן טבלאי.

בחיים אני לא אעביר code review אם יש שם טבלאות ליצירת layout.
גם אם זה בטופס או בתפריט.
בטופס חובה להשתמש ב- label עם הגדרת for בהתאם לשדה.
בתפריט מאוד מאוד מומלץ להשתמש ברשימה (ממוינת או לא ממוינת) כאשר כל קישור יושב בתוך LI.


כל הזמנים הם GMT +2. הזמן כעת הוא 05:41.

מופעל באמצעות VBulletin גרסה 3.8.6
כל הזכויות שמורות ©
כל הזכויות שמורות לסולל יבוא ורשתות (1997) בע"מ