happy New Year to all! Can you help me modify the small topic?

298003525

Set an integer to implement simple operations on integers
Written in vector
class IntSet
{
private:
    vector <int> Ints;
int element [100];
int n, seeknumber;
public:
IntSet ()
{
cout << "Enter the following integer:";
cin >> n;
for (long i = 0; i <n; i ++)
{
cin >> element [i];
Ints.push_back (element [i]);
};
cout << endl << Ints [0];
};
Empty () {Ints.clear ();};
Isempty () {if (Ints.empty ()) cout << "The collection is empty!" << endl;};
bool Ismemberof ()
{
cout << "To determine if this number is inside the collection, please enter it:";
cin >> seeknumber;
for (long j = 0; j <n; j ++)
Ranch
if (Ints [j] == seeknumber) return true;
Ranch
};
Add () {};
Sub () {};
Isequal () {};
Intersection () {};
Merge () {};
Print () {};
};
Is there anything wrong, please help me modify it? Thank you all here!

eyesloveyou

for (long i = 0; i <n; i ++)
{
cin >> element [i];
Ints.push_back (element [i]);
};
-------------------------------------------------- -
In fact, the array element is superfluous.


bool Ismemberof ()
{
cout << "To determine if this number is inside the collection, please enter this number:";
cin >> seeknumber;
for (long j = 0; j <n; j ++)
Ranch
if (Ints [j] == seeknumber) return true;
Ranch
};
-------------------------------------------------- --------------------
This lookup can be implemented using find (),
Or use vector <int> :: iterator to locate the value of seeknumber!


The ";" after the class is required, but the ";" after the function is not needed, and it is not wrong to add it, but it is not part of the function, but is regarded as a blank line by the compiler.

If you haven't forgotten the corresponding header file, there should be no problem.

eyesloveyou

cout << endl << Ints [0];
--------------------------------
Should it be changed to
cout << endl << * Ints.Begin ();

??

firegun

Although the usage is not very good, the compilation should pass. Likeeyesloveyou, element is useless here, and the input can be changed to:
 for (long i = 0; i <n; i ++)
{
         long temp;
cin >> temp;
Ints.push_back (temp);
};
To use find (), you need to include the header <algorithm>, as follows:
#include <algorithm>
usage:
vector <int> :: iter = find (Ints.begin (), Ints.end (), seeknumber);
if (iter! = Ints.end ())
return true;
else
return false;



In addition:
cout << endl << Ints [0];
no problem

firegun

Temp is a local variable, because containers such as vector all use value passing, so there will be no problem

298003525

Thank you guys!
So grateful

298003525

What else should be added on this subject? ?
I've changed it, but I feel that there should be something added, who knows

zjwzjw

class IntSet
{
private:
    vector <int> Ints;
int n, seeknumber, element;
public:
IntSet ()
{
cout << "Enter the following integer:";
cin >> n;
for (long i = 0; i <n; i ++)
{
cin >> element;
Ints.push_back (element);
};
// cout << endl << Ints [0];
}
void Empty () {Ints.clear ();}
bool Isempty ()
{
      if (Ints.empty ()) {cout << "The collection is empty!" << endl; return true;}
      else return false;
    }
bool Ismemberof ()
{
cout << "To determine if this number is inside the collection, please enter this number:";
cin >> seeknumber;
for (long j = 0; j <n; j ++)
if (Ints [j] == seeknumber) return true;
return false;
Ranch
};
Ranch
bool Isequal () {};
IntSet&Add () {};
IntSet&Sub () {};
IntSet&Intersection () {};
IntSet&Merge () {};
void Print () {};
};

// test
int main ()
{
    IntSet set1;
    set1.Isempty ();
    // .... other
    system ("pause");
    return 0;
}

zjwzjw

bool Isequal () {};
IntSet&Add () {};
IntSet&Sub () {};
IntSet&Intersection () {};
IntSet&Merge () {};
void Print () {};

These needs continue to be achieved ～～

298003525

How else? I am not very clear!