בעיה ביצירת מערכת העלאת קבצים

29-03-2005, 15:14

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

index.php

קוד PHP:

<html> <body> <form action="upload.php" method="post"> <input name="file" type="file"> <input name="submit" type="submit" value="העלה!"> </form> </body> </html>

upload.php

קוד PHP:

<?PHP if(!$_POST["file"]) { copy($_POST["file"],"/www/files"); echo "ההעלאה בוצעה בהצלחה"; } else { echo "העלאה נכשלה"; } ?>

עכשיו, כשאני מנסה להעלות קובץ תמונה למשל, זה מעביר אותי לupload.php ורושם לי העלאה נכשלה... למה? אני רוצה שהוא יעתיק לי את הקובץ אל התיקיה files.
יש משהו לא נכון בקובץ?

בתגובה להודעה מספר 1 שנכתבה על ידי -TAL- שמתחילה ב "בעיה ביצירת מערכת העלאת קבצים"

כל מה שקשור להתעסקות עם העלאת קבצים אתה יכול למצוא כאן.
שים לב שבתוך הטאג form צריך להופיע "enctype="multipart/form-data במידה ואתה מתעסק עם קבצים.
בנוסף, שים לב שמשתמשים בפונקציה move_uploaded_file ולא copy כמו שאתה השתמשת.

בברכה, דקל

_____________________________________

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

דקל....אני מתפלא עליך...

בתגובה להודעה מספר 2 שנכתבה על ידי דקל שמתחילה ב "[font=Arial]כל מה שקשור..."

שאני נאלץ לתקן אותך ב-PHP.

1. כן משתמשים ב-copy להעלאת קבצים, אני בעצמי בחנתי הרבה מערכות שהשתמשו בפונקציה זו (אני אישית משתמש move_uploaded_file ) אבל לומר שלא משתמשים זה לא נכון.

2. כדי לבדוק אם הקובץ נשלח (בוצע post לקובץ), לא נבצע בדיקה:

קוד PHP:

if($_POST["file"])

(אני לא מבין למה השימוש ב-!, למעשה התנאי שלך הפוך)

אלא נבצע בדיקה אחרת לגבי:

קוד PHP:

$_FILE

בכל מקרה, הנה דוגמה ל-move_upload_file:

קוד PHP:

<?php // In PHP versions earlier than 4.1.0, $HTTP_POST_FILES should be used instead // of $_FILES. $uploaddir = '/var/www/uploads/'; $uploadfile = $uploaddir . basename($_FILES['userfile']['name']); echo '<pre>'; if (move_uploaded_file($_FILES['userfile']['tmp_name'], $uploadfile)) { echo "File is valid, and was successfully uploaded.\n"; } else { echo "Possible file upload attack!\n"; } echo 'Here is some more debugging info:'; print_r($_FILES); print "</pre>"; ?>

ניתן למצא פה את כל המידע על העלאת קבצים ב-PHP (כולל בעזרת copy)
http://il.php.net/manual/en/features.file-upload.php

_____________________________________

[ זיו ]
[ fat fish ]

נכון, רק שכחת לציין ש...

בתגובה להודעה מספר 3 שנכתבה על ידי fat fish שמתחילה ב "דקל....אני מתפלא עליך..."

גם הטופס שגוי, יש להוסיף פרמטר בתגית FORM:

קוד PHP:

enctype="multipart/form-data"

על-מנת לקבל $_FILES ולא $_POST...

בתגובה להודעה מספר 3 שנכתבה על ידי fat fish שמתחילה ב "דקל....אני מתפלא עליך..."

קראתי את מה שכתוב באתר על $_FILES ועל move_uploaded_files (למרות האנגלית החלשה שלי), והבנתי פחות או יותר אך עדיין יש לי כמה שאלות:

1. בקוד הנ"ל:

קוד PHP:

basename($_FILES['userfile']['name']

מה עושה הפונקציה basename? ואילו משתנים/ערכים אמורים להיות במשתנה $uploadfile?

2. בקוד הבא:

קוד PHP:

$_FILES['userfile']['tmp_name']

לא הבנתי מה אמור להיות במקום ה-userfile, וה-tmp_name...

3. איזה פרמטרים/מאפיינים (תקראו לזה איך שתרצו...) אמור לקבל move_uploded_file?

תודה רבה למי שיעזור לי

נערך לאחרונה ע"י -TAL- בתאריך 29-03-2005 בשעה 18:13.

בתגובה להודעה מספר 5 שנכתבה על ידי -TAL- שמתחילה ב "תגובות לכולם"

לפני כל שאלה, תראה שאתה קורא על כל פונקציה שיש לך שאלה עליה פה:
http://www.php.net
ואם אחרי זה עדיין יהיו לך שאלות, נשמח לענות עליהן.

ו-web, דקל כבר ציין את הנושא של הטופס, אז לא הרגשתי צורך להדגיש שוב....אבל תודה.

_____________________________________

[ זיו ]
[ fat fish ]

נערך לאחרונה ע"י fat fish בתאריך 29-03-2005 בשעה 23:13.

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

בתגובה להודעה מספר 6 שנכתבה על ידי fat fish שמתחילה ב "RTFM"

הנה הקודים:

index.php

קוד PHP:

<html> <body> <form action="upload.php" method="" enctype="multipart/form-data"> <input name="file" type="file"> <input name="submit" type="submit" value="העלה!"> </form> </body> </html>

upload.php

קוד PHP:

<?PHP $upload_dir = "/files/"; $file_name = basename($_FILES['userfile']['name']); if (move_uploaded_file($file_name,$upload_dir)) { echo "true"; } else { echo "false"; } ?>

זה מחזיר לי False. למה? ומה אני צריך לעשות?

אופסי, לא שמתי לב, חחחח(ל"ת)

בתגובה להודעה מספר 6 שנכתבה על ידי fat fish שמתחילה ב "RTFM"

לא כל כך ברור לי מה אתה מתפלא :)

בתגובה להודעה מספר 3 שנכתבה על ידי fat fish שמתחילה ב "דקל....אני מתפלא עליך..."

דבר ראשון - כולם טועים. להיות תמיד צודק זה קצת משעמם... אתה לא חושב ?
בנוגע להתעסקות עם copy. יש המון דרכים לעשות כל מיני דברים... חלק מהדרכים מוצלחות יותר, וחלק פחות. זה ש copy עובד זה עדיין לא אומר שזאת הדרך המוצלחת ביותר לעשות את מה שהוא צריך (ובמקרה הזה דווקא השימוש ב move_uploaded_file זאת הדרך הנכונה כיוון שהיא מבצעת עוד בדיקות בנוגע לקובץ המקור שנמצא בתיקיה temp).
דבר שני - אני אנצל את התגובה הזה בשביל להגיב לטל (מקווה שזה לא יעשה בלאגן )
אז ככה טל:
1) השתמשת בטופס בשם file לשם הקובץ, ואח"כ אתה מנסה לקחת את הנתונים של userfile. אתה צריך להיות החלטי
2) הפונקציה move_uploaded_file מקבלת כערך ראשון את ['FILES['userfile']['tmp_name_$ ולא שום דבר אחר (שים לב שיש לך דוגמא בלינקים שנתתי), וכפרמטר שני את השם החדש של הקובץ החדש שאתה רוצה ליצור בשרת (כולל נתיב מלא).
קרא שוב את המניואל... הכל רשום שם.

בברכה, דקל

_____________________________________

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

בתגובה להודעה מספר 9 שנכתבה על ידי דקל שמתחילה ב "לא כל כך ברור לי מה אתה מתפלא :)"

1. אההה... השם של התיבה אמור להיות מוחלף בזה... חשבתי שזה משהו אחר..
2. אוקיי זה דווקא אחד הדברים שלא הבנתי לאחר הקריאה..

קוד PHP:

$_FILES['userfile']['tmp_name']

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

בתגובה להודעה מספר 10 שנכתבה על ידי -TAL- שמתחילה ב "1. אההה... השם של התיבה אמור..."

אז ככה. המשתנה הגלובאלי FILES_$ מכיל את הנתונים על הקבצים שאתה מעלה בטופס.
עבור כל אחד מהקבצים שהועלה נוצר לך מערך חדש ב FILES_$ שהשם שלו הוא ['FILES['userfile_$. מערך זה מכיל מספר פרמטרים על הקובץ שהעלית:

ציטוט:

במקור נכתב על ידי php.net

$_FILES['userfile']['name']
The original name of the file on the client machine.

$_FILES['userfile']['type']
The mime type of the file, if the browser provided this information. An example would be "image/gif".

$_FILES['userfile']['size']
The size, in bytes, of the uploaded file.

$_FILES['userfile']['tmp_name']
The temporary filename of the file in which the uploaded file was stored on the server.

$_FILES['userfile']['error']
The error code associated with this file upload. This element was added in PHP 4.2.0

לדעתי זה דיי מובן (וזה גם רשום לך בלינק שנתתי לך בהתחלה).

בברכה, דקל

_____________________________________

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

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

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

קוד PHP:

$_FILES['userfile']['name']

userfile - השם שנתתי לשדה ההעלאה בטופס.
name - משאירים את זה ריק... שם הקובץ של המשתמש שהעלה.

קוד PHP:

$_FILES['userfile']['type']

userfile - השם שנתתי לשדה ההעלאה בטופס.
type - סוג הקובץ. אני מגדיר את זה כך? /gif/jpg/png/

קוד PHP:

$_FILES['userfile']['size']

userfile - השם שנתתי לשדה ההעלאה בטופס.
size - גודל הקובץ בביטים.

קוד PHP:

$_FILES['userfile']['tmp_name']

userfile - השם שנתתי לשדה ההעלאה בטופס.
tmp_name - אני לא בטוח בנוגע לזה... זמן הקובץ על השרת?

הודעת השגיאה לא ניראת לי כל-כך חשובה, לכן אני לא אציין אותה.

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

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

בתגובה להודעה מספר 12 שנכתבה על ידי -TAL- שמתחילה ב "את זה בדיוק קראתי, אבל אני לא בטוח שהבנתי את הכל,"

אוקיי.. אז ככה:
name מחזיר לך את השם של הקובץ אותו המשתמש העלה.
tmp_name מחזיר לך את השם של הקובץ הזמני אשר הועלה.

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

קוד PHP:

move_uploaded_file($_FILES['fupload']['tmp_name'], $dir . $_FILES['fupload']['name']);

כאשר $dir מכיל את המיקום שבו ישמר הקובץ החדש.
אני מקווה שזה יותר ברור ככה.

אוקיי, הבנתי הכל, אבל הקוד לא פועל לי גם לאחר ששיניתי אותו:

בתגובה להודעה מספר 13 שנכתבה על ידי Fixxxer שמתחילה ב "אוקיי.. אז ככה: name מחזיר לך..."

Index.php

קוד PHP:

<html> <body> <form action="upload.php" method="" enctype="multipart/form-data"> <input name="file" type="file"> <input name="submit" type="submit" value="העלה!"> </form> </body> </html>

Upload.php

קוד PHP:

<?PHP $upload_dir = "/files/"; $file_name = $upload_dir . basename($_FILES['file']['name']); if (move_uploaded_file($FILES['file']['tmp_name'],$upload_dir)) { echo "true"; } else { echo "false"; } ?>

בתגובה להודעה מספר 14 שנכתבה על ידי -TAL- שמתחילה ב "אוקיי, הבנתי הכל, אבל הקוד לא פועל לי גם לאחר ששיניתי אותו:"

אתה צריך לתת את המיקום המלא של הקובץ החדש, כלומר:
"/files/" לא מספיק אלא צריך את כל הנתיב של השרת עד התיקייה הנ"ל.
אם אתה לא יודע את המיקום, אתה תמיד יכול להשתמש ב realpath.

אוקיי, שיניתי ועכשיו זה מציג את השגיאה הבאה:

בתגובה להודעה מספר 15 שנכתבה על ידי Fixxxer שמתחילה ב "אתה צריך לתת את המיקום המלא..."

קוד PHP:

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in c:\documents and settings\Tal\שולחן העבודה\MyServer\upload.php on line 3

בתגובה להודעה מספר 16 שנכתבה על ידי -TAL- שמתחילה ב "אוקיי, שיניתי ועכשיו זה מציג את השגיאה הבאה:"

עדיף שכל הזמן תתן את הקוד העדכני ביחד עם השגיאות, שיהיה יותר ברור.
לפי מה שאני רואה עכשיו, יכולות ליהיות 2 בעיות בקוד שלך.
1. ב realpath אל תשים גרש לפני שם התיקייה, ומומלץ גם לא לשים אחרי, אלא רק את השם התיקייה המבוקשת.
2. בפקודה move_upload.. רשמת בחלק שבו אמור ליהיות שם הקובץ החדש ומיקומו $upload_dir במקום $file_name.

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

בתגובה להודעה מספר 17 שנכתבה על ידי Fixxxer שמתחילה ב "עדיף שכל הזמן תתן את הקוד..."

index.php

קוד PHP:

<html> <body> <form action="upload.php" method="" enctype="multipart/form-data"> <input name="file" type="file"> <input name="submit" type="submit" value="העלה!"> </form> </body> </html>

upload.php

קוד PHP:

<?PHP $upload_dir = "C:\Documents and Settings\Tal\שולחן העבודה\MyServer\files\"; $file_name = $upload_dir . basename($_FILES['file']['name']); if (move_uploaded_file($FILES['file']['tmp_name'],$upload_dir)) { echo "true"; } else { echo "false"; } ?>

בתגובה להודעה מספר 18 שנכתבה על ידי -TAL- שמתחילה ב "[u]index.php [/u]..."

אני חוזר על מה שאמרתי מקודם:

קוד PHP:

if (move_uploaded_file($FILES['file']['tmp_name'],$upload_dir))

במקום $upload_dir אתה צריך לשים את $file_name .
כנראה סתם התבלבלת, אבל זה בהחלט מקור הבעיה. ואם לא "הבעיה" אז סתם בעיה

עדייו לא עובד, אני מיואש כבר...

בתגובה להודעה מספר 19 שנכתבה על ידי Fixxxer שמתחילה ב "אני חוזר על מה שאמרתי..."

index.php

קוד PHP:

<html> <body> <form action="upload.php" method="" enctype="multipart/form-data"> <input name="file" type="file"> <input name="submit" type="submit" value="העלה!"> </form> </body> </html>

upload.php

קוד PHP:

<?PHP $upload_dir = "C:\Documents and Settings\Tal\שולחן העבודה\MyServer\files\"; $file_name = $upload_dir . basename($_FILES['file']['name']); if (move_uploaded_file($FILES['file']['tmp_name'],$file_name)) { echo "true"; } else { echo "false"; } ?>

זה מציג לי את השגיאה הבאה:
קוד:
Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE,  expecting T_STRING or T_VARIABLE or T_NUM_STRING in c:\documents and  settings\Tal\שולחן העבודה\MyServer\upload.php on line 3

בתגובה להודעה מספר 20 שנכתבה על ידי -TAL- שמתחילה ב "עדייו לא עובד, אני מיואש כבר..."

האא.. אני לא מאמין איך לא שמתי לב :O
בכל מקרה, יש לך את הטעות המרכזית ועוד אחת קטנה.
המרכזית: בנתיב לתיקייה - רשמת בסופו סלאש ולאחר מכן גרשיים. כפי שאתה כנראה יותר, פעולה זאת אומרת לPHP שהגרשיים הן לא חלק מהקוד ולכן הוא כביכול לא סגר את המחרוזת.
האפשרויות שלך: או להוריד את הסלאש, או להוסיף עוד אחד אחריו (השניה מומלצת במקרה שלך).
עוד בעיה קטנה: בהעברת הקובץ למקומו החדש - רשמת שם $FILES במקום $_FILES (הקו התחתון).

וזהו.. זה צריך לעבוד

אוקיי, עכשיו אין שגיאה אבל זה מחזיר לי False :|

בתגובה להודעה מספר 21 שנכתבה על ידי Fixxxer שמתחילה ב "האא.. אני לא מאמין איך לא..."

index.php

קוד PHP:

<html> <body> <form action="upload.php" method="" enctype="multipart/form-data"> <input name="file" type="file"> <input name="submit" type="submit" value="העלה!"> </form> </body> </html>

upload.php

קוד PHP:

<?PHP $upload_dir = "C:\Documents and Settings\Tal\שולחן העבודה\MyServer\files\\"; $file_name = $upload_dir . basename($_FILES['file']['name']); if (move_uploaded_file($_FILES['file']['tmp_name'],$file_name)) { echo "true"; } else { echo "false"; } ?>

בתגובה להודעה מספר 22 שנכתבה על ידי -TAL- שמתחילה ב "אוקיי, עכשיו אין שגיאה אבל זה מחזיר לי False :|"

עכשיו אני מסתכל על ה index.php .
השארת את ה method ריק. למה זה?
תרשום שם post וזה יעבוד. אי אפשר, בשימוש ב file, להעביר נתונים דרך get.

סוף סוף עובד!!! תודה.

בתגובה להודעה מספר 23 שנכתבה על ידי Fixxxer שמתחילה ב "עכשיו אני מסתכל על ה..."

ד"א אני חשבתי שרושמים Post רק אם משתמשים בו..

בתגובה להודעה מספר 10 שנכתבה על ידי -TAL- שמתחילה ב "1. אההה... השם של התיבה אמור..."

הממ..... נסה לעשות echo ל tmp_name.....
רק תראה מה יודפס לך...

אם לא הבנת למה זה מה שמודפס זה משום שבPHP הקובץ בעצם מועלה לתיקייה זמנית שכזו שעלייך להעלות את הקובץ מהמקום הזמני שלו, לשם כך משתמשים ב move_uploaded_files.

הייתי גם מציע לך כדי ללמוד מה מחזיק כל מערך FILES לעשות את הדבר הבא:

קוד PHP:

<?php echo "<pre>"; print_r($_FILES); echo "</pre>"; ?>

בגלל שאני משתמש בשורות הללו...

בתגובה להודעה מספר 25 שנכתבה על ידי Web שמתחילה ב "הממ..... נסה לעשות echo ל..."

בתדירות מטורפת, לבדוק את תוכנם של מערכים, יש לי קובץ lib.inc.php שהוא סט פונקציות שאני מוסיף לכל מערכת שאני כותב, הפונקציה הזו זכתה שם לציון גבוה

קוד PHP:

function print_array($arr) { echo "<pre>"; print_r($arr); echo "</pre>"; }

_____________________________________

[ זיו ]
[ fat fish ]

למה יש לקובץ שם מוזר כזה?

בתגובה להודעה מספר 26 שנכתבה על ידי fat fish שמתחילה ב "בגלל שאני משתמש בשורות הללו..."

ואולי תפרסם באשכול חדש כל מיני פונקציות שימושיות?

תודה!

בתגובה להודעה מספר 27 שנכתבה על ידי דור שמתחילה ב "למה יש לקובץ שם מוזר כזה?"

מה מוזר בשם קובץ?...
lib - ספרייה , של פונקציות במקרה הזה
inc - שמטרתו היא include
php - מממ זה אהממ PHP? חחח

הדבר היחיד שלא הבנתי זה ה-inc... תודה!(ל"ת)

בתגובה להודעה מספר 28 שנכתבה על ידי שי משעלי שמתחילה ב "מה מוזר בשם קובץ?... lib -..."

כלי אשכול	חפש באשכול זה
הצג גירסת הדפסה שלח דף זה ב E-Mail	חפש באשכול זה: חיפוש מתקדם
מצבי תצוגה	דרג אשכול זה
עבור למצב לינארי מצב כלאיים עבור למצב משורשר	דרג אשכול זה:

הדף נוצר ב 0.11 שניות עם 10 שאילתות
צור קשר - Fresh - למעלה