Error message

[Latina]

Hi I am doing a program of sets using overloaded methods.
But I am having some error because supposedly the method isValid is
undeclared.
He is some part of my program.

#include <iostream>
#include <string>
#include <iomanip>

using namespace std;

class IntegerSet
{
private:
bool set[26];
int element;

public:
//Methods
IntegerSet(); //default
Constructor
IntegerSet(int x[], int k); //overload
constructor

bool isValue(int);
void insertElement(int);
void deleteElement(int);
void setString();
void inputSet();
};

//methos isValid
bool IntegerSet::isValid(int i)
{
return set;
}

Thanks.

 

[Latina]

Hi I am doing a program of sets using overloaded methods.
But I am having some error because supposedly the method isValid is
undeclared.
He is some part of my program.

#include <iostream>
#include <string>
#include <iomanip>

using namespace std;

class IntegerSet
{
private:
bool set[26];
int element;

public:
//Methods
IntegerSet(); //default constructor
IntegerSet(int x[], int k); //overload constructor

bool isValue(int);
void insertElement(int);
void deleteElement(int);
void setString();
void inputSet();

};

This is the error message pointing where the arrow is:
no `bool IntegerSet::isValid(int)' member function declared in class
`IntegerSet'

//methos isValid
bool IntegerSet::isValid(int i) --> {
return set;
}

Thanks.

 

[Michael]

class IntegerSet

{

bool isValue(int);

};

//methos isValid
bool IntegerSet::isValid(int i)
{
return set;

}

isValue is not the same as isValid. One or the other is misspelled.
Or maybe both.

Michael

 

[Latina]

isValue is not the same as isValid. One or the other is misspelled.

Or maybe both.

Michael

oops I didnt notice that, thanks
but I still have some other errors.

error: `isValid' undeclared (first use this function)

(where the arrow is pointing)


IntegerSet operator+(IntegerSet &right)
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
--> if(isValid(element) || right.isValid(element))
j.insertElement(element);
}
return j;
}

 

[Martin Engelmann]

but I still have some other errors.

error: `isValid' undeclared (first use this function)

(where the arrow is pointing)


IntegerSet operator+(IntegerSet &right)
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
--> if(isValid(element) || right.isValid(element))
j.insertElement(element);
}
return j;
}

Is operator+ a member of class IntegerSet? If it is, isValid will be
called with *this. If operator+ is not a member of class IntegerSet you
have to call it with an object.

Martin

 

[Latina]

Latina schrieb:










Is operator+ a member of class IntegerSet? If it is, isValid will be
called with *this. If operator+ is not a member of class IntegerSet you
have to call it with an object.

if i use *this gives me this error
--> invalid use of `this' in non-member function

 

[Martin Engelmann]

if i use *this gives me this error
--> invalid use of `this' in non-member function

So you should either make operator+ a member function of IntegerSet or
add a second parameter to the function.

IntegerSet IntegerSet:perator+(IntegerSet &right)
(my recommendation)
or
IntegerSet operator+(IntegerSet &left, IntegerSet &right)

When using a non-member function you have to change the condition to
if(left.isValid(element) || right.isValid(element))

 

[Latina]

Latina schrieb:







So you should either make operator+ a member function of IntegerSet or
add a second parameter to the function.

IntegerSet IntegerSet:perator+(IntegerSet &right)
(my recommendation)
or
IntegerSet operator+(IntegerSet &left, IntegerSet &right)

When using a non-member function you have to change the condition to
if(left.isValid(element) || right.isValid(element))- Hide quoted text -

Ok, I declared like this and this is the error I am getting
"expected init-declarator before '+' token ":

class IntegerSet
{
public:
//Operator methods.
IntegerSet &operator+(const IntegerSet &right);


//overloaded operator + to compute the union of two sets
--> IntegerSet IntegerSet:erator+(IntegerSet &right)
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
if(isValid(element) || right.isValid(element))
j.insertElement(element);
}
return j;
}

But if I do it like this I am getting the same error
"expected init-declarator before '+' token"

//overloaded operator + to compute the union of two sets
--> IntegerSet perator+( &left, IntegerSet &right)
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
if(left.isValid(element) || right.isValid(element))
j.insertElement(element);
}
return j;
}

 

[Latina]

Ok, I got it but now I am getting this error:
" passing `const IntegerSet' as `this' argument of `bool
IntegerSet::isValid(int)' discards qualifiers "


//overloaded operator + to compute the union of two sets
IntegerSet IntegerSet:perator+(const IntegerSet &right)const
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
--> if(isValid(element) || right.isValid(element))
j.insertElement(element);
}
return j;
}

 

[Victor Bazarov]

Ok, I got it but now I am getting this error:
" passing `const IntegerSet' as `this' argument of `bool
IntegerSet::isValid(int)' discards qualifiers "


//overloaded operator + to compute the union of two sets
IntegerSet IntegerSet:perator+(const IntegerSet &right)const
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
--> if(isValid(element) || right.isValid(element))
j.insertElement(element);
}
return j;
}

Most likely 'isValid' is not declared 'const' (unlike 'operator+'
here, for example)

V

 

[Latina]

Most likely 'isValid' is not declared 'const' (unlike 'operator+'
here, for example)

V

Ok, thanks

 

[Latina]

I have another error where the arrow is.

Error: " `IntegerSet IntegerSet:perator!(const IntegerSet&)' must
take `void' "

class IntegerSet
{
public:
//Operator methods.
IntegerSet operator + (const IntegerSet &)const; //Method
union
IntegerSet operator * (const IntegerSet &)const; //Method
intersection
--> IntegerSet operator ! (const IntegerSet &)const; //Method
complement

//overloaded operator ! to compute the complent of a set
IntegerSet IntegerSet:perator!(IntegerSet &right)const
{
IntegerSet j;

for(int element=0; element<=25; element++)
{
if(!isValid(element))
j.insertElement(element);
}
return j;
}