[ تمت الإجابة ]فئة لحفظ الأرقام الزوجية والفردية

اهلين اخوي ...

ممكن انك تكتب الكود بشكل منسق ..لاني لا استطيع قرائته :)

السلام عليكم

ما دمت تتعامل مع أرقام صحيحة .. لماذا تستخدم char ؟؟

وعليكم  السلام 

صححتها و استخدمت int   و بالخطأ نسخت لكم  الحل السابق 

اعذرني على ارفاق الكود كل ما اعمله يطلع شكله مو صح 

هل المطلوب هو التأكد من زوجية أو فردية العدد أم الأرقام المكونة للعدد ؟

المطلوب هو عند ادخال رقم من عدة خانات ان يطبع بدل الرقم الزوجي 1 و الرقم الفردي 0

يعني مثلا, إذا كانت المدخلات هي 2914 ستكون المخرجات 1001 ؟

الدوال المستخدمه هي Instance Functions و بالتالي لا داعي لتمرير الـ setType، فقط قم بإستخدام this داخل الـ function او قم بإستخدام العناصر الـ private مباشرة.

أيضا استخدامك لـ firstValue و lastValue كـ indexes للمصفوفة خاطئ و ذلك لأن قيمة firstValue ليست صفر و إنما القيمة المقابلة للحرف صفر و هي 48 و هذا سيسبب Overflow وقت التشغيل.

الدالة IsElementOf ستسبب Overflow إذا كانت القيمة المدخلة ليست بين صفر و 9.

الدالة SetAssign لا تقوم بشي جديد و بالتالي لا داعي منها حيث ان المترجم سيقوم بإنشاء copy constructor سيتكفل بهذا الأمر، فقط قم بإستخدام = لعملية الـ assignment.

عندما تقوم بتمرير كائن لدالة إستخدم التمرير بالمرجع حتى لا يتم نسخ الكائن.

#include <iostream>using namespace std;const int universe = 10;class setType {    bool set [universe];public:	setType() {		for (int i = 0; i < universe; i++)			set = false;	}    bool IsElementOf(char E) {		E -= '0';		if (E < 0 || E >= universe) return false;		return set[E];	}    bool SetEmpty() {		for (int i = 0; i < universe; i++)			if (set == true) return false;	}    bool SubSetOf(setType& T) {		for (int i = 0; i < universe; i++)			if (set && !T.set) return false;	}    void Union(setType& S, setType& R) {		for (int i = 0; i < universe; i++)			set = (R.set || S.set);	}    void AddElement(char U) {		U -= '0';		if (U < 0 || U >= universe) return; // failed should do something		set = true;	}    void SetDisplay() {		for (int i = 0; i < universe; i++)			cout << set << " ";	}};int main() {    //Create a Set to represent Even Numbers	setType EvenNumbers;    EvenNumbers.AddElement('0');    EvenNumbers.AddElement('2');    EvenNumbers.AddElement('4');    EvenNumbers.AddElement('6');    EvenNumbers.AddElement('8');    //Create a set to represent Odd Numbers	setType OddNumbers;    OddNumbers.AddElement('1');    OddNumbers.AddElement('3');    OddNumbers.AddElement('5');    OddNumbers.AddElement('7');    OddNumbers.AddElement('9');    //Create a set of Even and Odd Numbers	setType Numbers;    Numbers.Union(OddNumbers, EvenNumbers);    Numbers.SetDisplay(Numbers);    cout << endl << endl << endl;    //Create a set InLineNumbers which will be used to store any   Numbers that we find in the  //line of text	setType InLineNumbers;    char ch = 0;    while (ch != '0') {        cin >> ch;        if (Numbers.IsElementOf(ch)) InLineNumbers.AddElement(ch);    }    //Display the Set InLineNumbers that appeared in the line    InLineNumbers.SetDisplay();}

قارن الكود الجديد بالخاص بك حتى تعرف ما الأخطاء الأخرى التى وقعت بها.

و الله ولي التوفيق

شكرا جزيلا لك ..

جزاك الله كل خير