:1power: نبدأ التحليل والاقتراحات:
===============
:court: السطر 16 يقوم بإدراج حقل مخفي... لا أرى ضرورة لوجوده
كود PHP:
<input type="hidden" name="action" value="edit1">
حيث أرى أنه استخدمه لمعرفة إذا تم التحميل بواسطة النموذج.. في السطر 23
كود PHP:
if ($action=="edit1")
يمكن حذف السطر 16 واستبدال السطر 23 بـ
كود PHP:
if ($_POST[['file1'])
===============
:court: في السطر 22 يوجد اسم المجلد
هذا خطأ... ضع كلمة pic بين علامتي اقتباس ليصير
===============
:court: يوجد خطأ أيضاً في السطر 28... حيث يوجد الآتي
كود PHP:
if($file1 != "none" )
وهو يقصد أن يتحقق إذا كان حقل الملف خاليا ... ولكن لن يعمل هذا أبدا
لأن none أصبحت قيمة معينة
وأرى أنه ظن أن ذلك يتحقق من التصاريح لتحميل الملف ولكن لن ينجح ذلك
استبدله بالآتي:
كود PHP:
if(trim($file1) != "" && !empty($file1))
===============
:court: والآن الخطأ في السطر 30 .. حيث يوجد الآتي
كود PHP:
if ($file1_size > 200000)
وذلك للتحقق من حجم الملف المصرح بتحميله
وهذا لن ينجح أيضاً .... استبدله بالآتي ليتم التحقق من حجم الملف
كود PHP:
if ($_FILES['file1']['size'] > 1000)
حيث 1000 هو الحجم الأقصى المطلوب مع ملاحظة أن ذلك بالبايت
===============
:court: في السطر 35 يوجد الآتي
كود PHP:
if ($file1_type !="image/pjpeg" && $file1_type !="image/gif" && $file1_type !="image/jpeg" )
وذلك لن يحقق شيئاً،، حيث أراد التحقق من صيغة الملف
بدون إطالة كلام استبدله بالآتي
كود PHP:
$types = array("image/pjpeg","image/gif","image/jpeg");
if(!in_array($_FILES['file1']['type'],$types))
===============
هذه أهم الاقتراحات....لم أقم بعرض جميع الاقتراحات لدي...
لكن هذا سيضمن عمل السكربت بشكل مقبول
حاولوا التعديل بأنفسكم على السكربت لمعرفة المزيد
:app:
تحياتي