ابي تصحيح للاخطاء في هالبرنامج c++

بسم الله الرحمن الرحيم

السلام عليكم ورحمة الله وبركاته

مع أني ماخذ أجازة من الإنترنت نهاية شهر ستة بس ما قدرت إني ما أدخل على المنتدى

أخي البرونزي أود أن أقول لك عن الأخطاء التالية :

1-

int small,large; 
determine (small,large);

هنا أنت قمت بتمرير المتحولات إلى البرنامج الجزئي علما بإنك لم تضع قيمة إبتدائية لهذه القيم وفي الفنكشن قمت بجعل المتحول number مساويا للمتحول largest علما بأنه يحتوي على قيمة إفتراضية

2-

  
if ((large>=0) && (small>=0)) 
{ 
cout<<small<<":"<<large<<endl; 
} 
else 
{cout<<"NOTHING TO PROCESS"<<endl; 
}return 0; 
}

في باقي البرنامج هنا قمت بالإعتماد على المتحولين small و large علما بإنها لا تزال قيم فارغة بدون آي قيمة (وذلك لأن القيمة عندما تدخل في فنكشن لا تتغير نهائيا إلا إذا مررت للفنكشن بالمرجع آي يكون شكل البروتوتايب هكذا

void determine (int &smallest, int &largest);

وعندها فقط كل تغيير يطرأ على المتحولات الداخلة في الفنكشن سوف يحدث تغييرا في المتحولات الأساسية التي دخلت في الفنكشن

3- في الفنكشن قمت أو كنت تريد أن تجعل جميع القيم مساوية للصفر إذا لم أخطئ وهكذا ستكون أصغر قيمة هي دائما مساوية للصفر وذلك لأنه لا يمكن إدخال رقم أصغر من الصفر

4- أخي لقد قمت بتعديل البروتوتايب لكي يعمل البرنامج على أكمل وجه حيث تستطيع أخذ فكرة الحل من هذا البرنامج بالإضافة إلى تغيير المعادلة الخاصة بالبرنامج

فكرة الحل : هي أن يقوم المستخدم بإدخال رقم قبل الحلقة حيث يكون هذا الرقم هو الرقم الإفتراضي لجميع المتحولات ثم أدخل إلى حلقة شرطها عنصر غير مساوي للصفر ثم ندخل فيها رقم أخر من المستخدم ثم ندخل في شرط إذا كان الرقم المدخل أكبر من الصفر يدخل ثم نقوم بوضع الشروط التي وضعتها أنت في برنامجك ثم نضع الحالة الأخرى من الشرط وهو أن العنصر يساوي الصفر لكي يخرج من الحلقة في حال لم يوافق الشروط

على كل حال أخي هذا البرنامج سيوضح كلامي :

#include <iostream> 
using namespace std; 
void determine (int &smallest, int &largest); 
int main() 
{ 

int small,large; 
determine (small,large); 
cout <<large<<"  "<<small;

if ((large>=0) && (small>=0)) 
{ 
cout<<small<<":"<<large<<endl; 
} 
else 
{cout<<"NOTHING TO PROCESS"<<endl; 
}return 0; 
} 




void determine (int &smallest, int &largest) 
{ 
int number=0; 
number=smallest=largest; 
cout<<"Please enter a number (negative value to end):"; 
cin >>number;
smallest=largest=number;
do {

cin>>number;
if (number!=0&&number>0)
	{
if (number>largest) 
largest=number; 

if (number<smallest) 
smallest=number;  
	}
	else
	number=0;
}while (number >0);


}

أتمنى أن أكون قد أفدت

أخوك في الله سامر

#include <iostream>

using namespace std;

void determine (int smallest, int largest);

int main()

{

int small,large;

cout<<"Please enter a number (negative value to end): ";

cin >> small>>large;

if ((large>=0) && (small>=0))

{

while ((small>0)&&(large>0))

{

determine (small,large);

cout<<"Please enter a number (negative value to end): ";

cin >> small>>large;

}

}

else

{

cout<<"NOTHING TO PROCESS"<<endl;

}

return 0;

}

///--------------------------------------------

void determine (int number1, int number2)

{

int largest=number1;

int smallest=number2;

if (number1<smallest)

{

largest=number1;

smallest=number2;

}

if (number2>largest)

{

largest=number2;

smallest=number1;

}

cout<<largest<<":"<<smallest<<endl;

}

هذا هو تصيحي ... لا تنسونا من صالح دعائكم ...أخوانكم في عمان