[Mr.Ahmad]

المشاركة الأصلية كتبت بواسطة مواليد نكبت1411 هلا حبيبتي انا حليته بif وترى عادي حقوق الطبع مو محفوظه >>>int Grenache; int x; char city; cout<<"city\tcode\n\n"; cout<<"jeddah\tj\n"; cout<<"makkah\tm\n"; cout<<"cairo\tc\n"; cout<<"enter your choice:"; cin>>city; cout<<"enter the time in grenache:"; cin>> Grenache; if(Grenache>=1 && Grenache<=24) { if(city='j') { x=(Grenache+3) if (Grenache>=1 && Grenache<12) cout<<"the time in jeddah: "<<x <<"a.m."; else cout<<"the time in jeddah: "<<x <<"p.m."; } else if(city='m') { x=(Grenache+3); if (Grenache>=1 && Grenache<12) cout<<"the time in makkah: "<<x <<"a.m."; else cout<<"the time in makkah: "<<x <<"p.m."; } else { x=(Grenache+2); if (Grenache>=1 && Grenache<12) cout<<"the time in cairo: "<<x <<"a.m."; else cout<<"the time in cairo: "<<x <<"p.m."; } } else cout<< "wrong entry not in the range of 1 to 24"; getch(); }

ملاحظات ( انتقادات بنائة ) :
1- يجب أخذ في الإعتبار إدخال المستخدم حرف كبير أو صغير .
مثلاً عندما يدخل المستخدم J راح يطلع له خطأ بالنسبة لحلك ,,, أما لو كتب j يعتبرها صح ,,, يجب الأخذ في الإعتبار لهذه النقطة .
2- ماذا لو أدخل المستخدم حرف ليس ( j و m و c ) ؟! لا يفعل البرنامج شيء !!!
... إذاً يجب الأخذ في الإعتبار هذه النقطة .
3- ماذا لو أدخل المستخدم مدينة جدة ووضع وقت 24 ... سيظهر الوقت الناتج 27 . فهل يوجد وقت كهذا ؟!!!
إذاً يجب الأخذ في الإعتبار لهذه النقطة .
4- تجنبوا تكرار الكود لتغيير بسيط .
مثل :

 
cout<<"the time in makkah: "<<x <<"p.m.";

و

 
cout<<"the time in makkah: "<<x <<"a.m.";

5- أجعل من حلك مرونة وتجنب الإطالة في الكود .

هذا ,, بالتوفيق .