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

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

DorOaki 25-09-08 18:08

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

אחרי מספר שבועות, המערכת כמעט מוכנה (למעט עיצוב) וכוללת את האפשרויות הבאות:
  • כל תמונה שמועלת לשרת מקבלת ID משלה דרך המסד, שם התמונה, גודל התמונה, אורך, רוחב וכתובת התמונה
  • תנאי שגיאות (שגיאות קובץ גדול מידי, קובץ לא נבחר, סיומת לא נתמכת, אימייל לא תקין - למערכת ה"הערות והצעות" שיפורט במהשך ועוד)
  • לאחר העלאת התמונה לשרת, הקובץ מקבל שם חדש, עם 4 ספרות רנדומליות בתחילת השם על מנת שלא יווצר מצב של שם אותו דבר
  • תווי מיוחדים (רווח, גרשיים, גרש וכו') נהפכים ישירות אל התו _
    לדוגמא, קובץ בשם image name.gif ייהפך אוטומטית אל image_name.gif
  • דף הצגת התמונה, הכולל אפשרויות מיוחדת כמו אורך התמונה, גודל, תאריך העלאה, שעת העלאה וכו'
  • כל תמונה שמועלת לשרת, מועלת אל שתי תיקיות בו זמנית, תיקיה אחת בשם images בגודל התמונה המקורי, ואחת מועלת לתיקיה thumbs בגודל 150 על 150 פיקסלים על מנת להציג את התמונה בגודל קטן.
  • מערכת הערות והצעות, עובד על פי מסד, כל תגובה שמתקבלת מקבלת דרך המסד ID משלה, שם השולח, אימייל, נושא ותוכן ההודעה

התגובות נשלחות לפאנל ניהול של המערכת (לא עבדתי עליו ממש, מספר אפשרויות קטן - בהמשך ייגדל), שאותו תוכלו לראות כאן:
http://musicalz.smartracker.net/tester/admin/
שם משתמש:
admin
סיסמא:
1234
יש ללחוץ התחבר ולא אינטר

בנוגע לאבטחה, כפי שראיתם אני כרגע מתלמד בתיכנות PHP ולא התעסקתי באבטחה,
יש אבטחה בסיסית של חסימת תווים מיוחדים, htmlspeicalchars, mysql_real_escape_string וכו' אך לא יותר מזה, אז אני מבקש נא לא להגיב על האבטחה - המערכת גם נוצרה למטרות לימוד בלבד ולא למכירה/פרסום.

טוב, אז זאת המערכת ואני מקווה שלא שכחתי עוד איזה אפשרות,
אני אשמח לקבל ביקורת על התכנות בלבד ולא על העיצוב שאין;P
ומקווה לקבל ביקורות בונות, כל הצעה תתקבל בברכה
הרי אני בטוח שעשיתי כמה דברים לא טובים/תקניים

יום טוב לכולם

XxXYonIXxX 25-09-08 18:27

אחלה של עבודה!
יחסית למתלמד, המערכת ממש טובה(יש כמה דברים שצריך לסדר, אבל אני מתחשב בעובדה שאתה מתלמד).
המשך כך =]
אגב, אני אשמח אם תוכל לדבר איתי במסן:
yoni_levi1221@hotmail.com

DorOaki 25-09-08 20:00

תודה רבה,
הוספתי אותך

Noam 25-09-08 21:42

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

DorOaki 25-09-08 21:54

ציטוט:

נכתב במקור על ידי Noam (פרסם 669413)
בסיסי מאוד אבל כל הכבוד..
שמעלים יש שגיאה.. אי אפשר לבדוק אותה ככה.

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

daMn 25-09-08 23:50

בהצגת תמונה, תעבוד עם GD ותשנה רזולוציה לתמונה ואל תעבוד עם הפרמטרים של אורך ורוחב של תגית img.
אני חושב שאתה בודק סיומות של קבצים ולא Mime Types כי העלתי קובץ טקסט שהוספתי לו jpg בסוף,
http://musicalz.smartracker.net/test.../4244_Jobs.jpg
אז תעבוד על הקטע הזה.

Erez | TrustMedia.co.il 26-09-08 11:26

יש שגיאה בהעלאה
הקובץ אבל מועלה בהצלחה לשרת
אני רואה שהשתמשת הרבה בקודים שלי חח
פאנל ניהול מאוד דומה לשלי וגם הקוד אבטחה שלי
התמונה מקוטנת מעט מפגרת
אם אני מעלה תמונה שהגודל שלה קטן מ100*100 אז זה ישנה את הגודל ובעצם יגדיל אותה במקום להקטין-תסדר

DorOaki 26-09-08 12:42

ציטוט:

נכתב במקור על ידי daMn (פרסם 669424)
בהצגת תמונה, תעבוד עם GD ותשנה רזולוציה לתמונה ואל תעבוד עם הפרמטרים של אורך ורוחב של תגית img.
אני חושב שאתה בודק סיומות של קבצים ולא Mime Types כי העלתי קובץ טקסט שהוספתי לו jpg בסוף,
http://musicalz.smartracker.net/test.../4244_Jobs.jpg
אז תעבוד על הקטע הזה.

תודה רבה, תוקן
עשיתי תנאי שבודק את ה-Mime Types של הקבצים, עכשיו לא ניתן לעלות קבצים שה-Mime Types שלהם לא שווה אל של gif png jpg וכו'
אשמח לקבל עוד הצעות לשיפור
אגב,
EAStyle,
היה שגיאה בגלל שמישהו העלה קובץ txt לשרת, והוא לא יכל היה לקחת את האורך וגובה של התמונה ובגלל זה הוא הציג שגיאה
עכשיו זה כבר לא יהיה - זה תוקן
בקשר לקודים שלך, במה השתמשתי?
אני לא אגיד שלא נעזרתי בך, כי כן נעזרתי בך ואין שום בושה בזה
אבל קודים שלך? ממש לא

Davieh 26-09-08 13:07

אני מנסה להעלות תמונה JPG רגילה, זה רושם לי שהסיומת לא טובה, הכוונה לפורמט שלה ..
טפל בזה :)

DorOaki 26-09-08 14:23

אני אשמח אם תשלח לי קישור לתמונה כאן או בפרטי
על מנת שאוכל לראות את הפורמט שלה ולעדכן בתנאי שבודק את הפורמטים, הבעיה נוצרה מכיוון שהפורמט של התמונה לא הוכנס לתנאי (וזה כי אני לא מכיר את כל הפורמטים של התמונות חוץ מ-image/jpeg, image/x-png)

Tomer 26-09-08 14:38

image/x-png
image/png
image/gif
image/jpeg
image/pjpeg

אצלי באתר זה לא עושה בעיות.

DorOaki 26-09-08 14:40

ציטוט:

נכתב במקור על ידי Tomer (פרסם 669524)
image/x-png
image/png
image/gif
image/jpeg
image/pjpeg

אצלי באתר זה לא עושה בעיות.

תודה רבה,
עכשיו אני מוסיף את השאר

DorOaki 27-09-08 09:40

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

Basloman 01-10-08 19:38

אחלה מערכת
כל הכבוד

DorOaki 01-10-08 19:39

תודה רבה.

מספר שינויים שביצעתי במערכת,
הוספתי עיצוב CSS לטעמי נוחות בלבד,
שם הקובץ החדש השתנה, כל קובץ שמועלה לשרת מקבל 12 תווים של מספרים ואותיות באנגלית רנדומליות - וזה על מנת שתווים אסורים כמו % ^ & * וכו' לא יועלו לשרת.
התווסף בעמוד הצגת התמונה, טבלה מסודרת של 5 התמונות האחרונות שהועלו לשרת

שיפורים בפאנל ניהול:
- התווסף אפשרות הוספת מנהלים
- התווסף אפשרות עריכת מנהלים - עריכת פרטים (שם משתמש, אימייל, סיסמא(מוצפנת ב-MD5))
- התווסף אפשרות מחיקת מנהלים
- התווסף אפשרות הוספת לינקים
- התווסף אפשרות עריכת לינקים (שם האתר, כתובת)
- התווסף אפשרות מחיקת לינקים
- תצוגת התמונות השתנה לטבלה מסודרת הכוללת שם התמונה,כתובת,גודל,תאריך העלאה,IP ומחיקה
- תצוגת התגובות השתנתה לטבלה מסודרת הכוללת שם השולח, אימייל,תאריך שליחה,שעה,כותרת, תוכן ומחיקה

לינק למערכת:
http://musicalz.smartracker.net/tester
לינק לפאנל ניהול:
http://musicalz.smartracker.net/tester/admin/
שם משתמש:
hosts
סיסמא:
hosts
נא לא לשנות סיסמא, תודה.

ASTeam 02-10-08 15:19

בהעלאת תמונה
ציטוט:

Warning: imagesx(): supplied argument is not a valid Image resource in /home/doroaky2/domains/musicalz.smartracker.net/public_html/tester/upload.php on line 133

Warning: imagesy(): supplied argument is not a valid Image resource in /home/doroaky2/domains/musicalz.smartracker.net/public_html/tester/upload.php on line 134

Warning: Division by zero in /home/doroaky2/domains/musicalz.smartracker.net/public_html/tester/upload.php on line 138

Warning: imagecreatetruecolor() [function.imagecreatetruecolor]: Invalid image dimensions in /home/doroaky2/domains/musicalz.smartracker.net/public_html/tester/upload.php on line 141

Warning: imagecopyresized(): supplied argument is not a valid Image resource in /home/doroaky2/domains/musicalz.smartracker.net/public_html/tester/upload.php on line 144
--
בהתחברות
פשוט בטופס של שינוי פרטים תעשה בלי form
--
תעשה שאם התמונה גדולה מידי - שישאל אם להקטין, ויקטין
--
תוותר על -
5 תמונות אחרונות

DorOaki 02-10-08 15:24

ציטוט:

נכתב במקור על ידי ASTeam (פרסם 670652)
בהעלאת תמונה

--
בהתחברות

פשוט בטופס של שינוי פרטים תעשה בלי form
--
תעשה שאם התמונה גדולה מידי - שישאל אם להקטין, ויקטין
--
תוותר על -
5 תמונות אחרונות

עשיתי הרשאות למשתמשים - אין באפשרותם עכשיו לערוך או למחוק דברים בפאנל אלא רק למשתמש הראשי (admin)

--
בנוגע להודעות אזהרה - תוקן
--
בנוגע ל-5 תמונות אחרונות שיישאר בינתיים:-D

ASTeam 02-10-08 15:30

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

DorOaki 02-10-08 15:32

ציטוט:

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

איפה אתה רואה פה עיצוב?
כבר כתבתי שאני מבקש ביקורת על התיכנות ולא על העיצוב - כי בעצם אין פה עיצוב, אני באתי להראות עבודת תכנות
בכל מקרה תודה - אני אוסיף עוד אפשרויות בהמשך

ASTeam 02-10-08 15:34

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


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

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