כתבתי קוד הקטנת תמונה די ארוך (53 שורות), ואני שוקל להכניס אותו לפונקציה.
הקוד שומר את התמונה בתיקיה מסויימת, ולא פולט אותה לדפדפן. מה שהוא כן פולט, זה הודעה "התמונה שונתה בהצלחה בלה בלה בלה".
מה להחזיר ב-return? את ההודעה או את המשתנה שמכיל את התמונה?

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

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

קוד PHP:

 if(img())
    $msg = 'התמונה הוקטנה ונשמרה בהצלחה'; 


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

הכוונה הייתה שאם התמונה שונתה בהצלחה תעשה true בreturn......

תלוי איך כתבת את הקוד, אני הבנתי מדבריך שהפונקציה עצמה שומרת את הקוד, ואז אתה יכול לעשות כמו שפירט web.
אבל למיטב ידעתי, גם אם תחזיר תמונה, תוכל לבצע בדיקת if, למרות שזה לו ערך בוליאני, כי אם תבצע בדיקת if כל ערך חיובי יומר ל-true (ושלילי ל-false), כך שתוכל להחזיר את התמונה במקרה של הצלחה, או false בכשלון.

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

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

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

קוד PHP:

 if (@imagejpeg($thumb, $newfilename))
{
return $newfilename;
} 


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

קוד PHP:

 image_resize($_POST['src'], $_POST['size']); 


כשרציתי להדפיס את הנתיב, עשיתי ככה (זו שורה 77):

קוד PHP:

 echo "<a href=".$newfilename.">".$newfilename."</a>" 


קיבלתי שלוש שגיאות:

קוד:

Notice:  Undefined variable: source in C:\Program Files\Apache Group\Apache2\htdocs\resize.php on line 63
 
Warning: imagecopyresized(): supplied argument is not a valid Image resource in C:\Program Files\Apache Group\Apache2\htdocs\resize.php on line 63
 
 Notice:  Undefined variable: newfilename in C:\Program Files\Apache Group\Apache2\htdocs\resize.php on line 77

זו שורה 63:

קוד PHP:

 imagecopyresized($thumb, $source, 0, 0, 0, 0, $newwidth, $newheight, $width, $height); 


נסה אולי להדפיס (echo, var_dump וכדומה) את כל המשתנים, לבדוק אותם אחד אחד ולוודא שבכל אחד הערך הרצוי.

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

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

אז תבדוק, ותדווח ...

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

מסתבר שהבעיה הייתה במבנה ה-switch הבא:

קוד PHP:

 switch($format)
                {
                    case 'image/jpeg':
                 $source = imagecreatefromjpeg("$filename");
                    break;
                    case 'image/gif':
                 $source = imagecreatefromgif("$filename");
                    break;
                    case 'image/png':
                 $source = imagecreatefrompng("$filename");
                    break;
                } 


(יש בעיה עם ההזחות בעורך של פרש, אין לי מושג איך לסדר את זה.)

כשהסרתי את המבנה והשארתי רק את השורה הזו:

קוד PHP:

 $source = imagecreatefromjpeg("$filename"); 


הכל עבד מעולה.

מה הבעיה במבנה?

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

תראה במיוחד את הקוד שמגדיר את המשתנה format

קוד PHP:

 if (is_string($filename))
        {
            $format = '';
            $format = image_type_to_mime_type(exif_imagetype("$filename")); 


filename מתקבל כארגומנט.

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

ציטוט:

Get Mime-Type for image-type returned by getimagesize, exif_read_data, exif_thumbnail, exif_imagetype

אבל אתה צודק לגבי זה שזה מיותר להשתמש בשתיהן.

עברתי לשימוש רק ב-exif_imagetype, וזה עובד מעולה. אתה גדול!

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

קוד PHP:

 image_resize("gingi.jpg", "222x200");
echo $newfilename; 


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

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


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

קוד PHP:

 $newfilename =  image_resize("gingi.jpg", "222x200");
echo $newfilename; 


באמת עזרת לי וקידמת אותי.
מי יתן ויעזרו גם לך