"return function" doesn't work

here's my simplified codes:

Code:

---

class Drug
{
          public:
                  Drug(): avai(0) {};
                  bool& get_avai()
                  {
                            return avai;
                  }
          protected:
                  bool avai;
};

if (somedrug.get_avai())
{
            //do something
}

---

my compiler (Dev-C++) says " cannot convert Drug::get_avai()' to `

how come?:confused:

You're missing a semi-colon at the end of your class.

Code:

---

class Drug
{
          public:
                  Drug(): avai(0) {};
                  bool& get_avai()
                  {
                            return avai;
                  }
          protected:
                  bool avai;
};

---

Quote:

---

Originally posted by MrWizard
You're missing a semi-colon at the end of your class.

Code:

---

class Drug
{
          public:
                  Drug(): avai(0) {};
                  bool& get_avai()
                  {
                            return avai;
                  }
          protected:
                  bool avai;
};

---

---

well, ya....sry, but that wasn't the problem (i just typed it wrong)

The error message is weird; is that all you get?
You might wanna omit the ampersand (&) as the return type

Code:

---

// bool& get_avai()
bool get_avai()
{
      return avai;
}

---

Quote:

---

Originally posted by alphaoide
The error message is weird; is that all you get?
You might wanna omit the ampersand (&) as the return type

Code:

---

// bool& get_avai()
bool get_avai()
{
      return avai;
}

---

---

Umm....tried it. Stilll doesn't work (same error) :(

I tried to compile your class with an empty main and the only line that gave me problems was

Code:

---

Drug(): avai(0) {};

---

there is a colon after the function Drug. I think that is supposed to be a constructor? If it is there is no colon after it just braces with whatever constructor code. The next part with the avai function gives me a syntax error for a constant? Not sure if that helps or not. If I take out the line above it compiles with no errors. I don't see where you have any destructors?

Code:

---

Drug(paramater list here for constructor)
{ constructor code here }

~Drug()
{ destructor code here or empty if no clean up needed }

---

Maybe you should post the full code where you get this problem, and the full error. For example, maybe you passed const Drug& somedrug to a method, then called somedrug.get_avai(). That might cause a "cannot convert" error because of the const. This code works fine in MSVC++6.0:

Code:

---

class Drug
{
public:
    Drug(): avai(0) {};
    bool& get_avai()
    {
        return avai;
    }
protected:
    bool avai;
};

int main()
{
    Drug somedrug;
    if (somedrug.get_avai())
    {
    }
}

---

By the way, the constructor is fine, and there is no need to write a destructor if it isn't going to do anything. The default will work just fine.

:( I tried to recreate a similar class instead of copy and paste. I will avoid making comments in the future until I have a better understanding of the problem. Sorry.

Okya, I 'm gonna post up the actual code that makes the error

Code:

---

//TOWN.H
#ifndef TOWN_H
#define TOWN_H
#include <string>
#include <iostream>
using namespace std;

int rand_between(int min, int max);        //returns random number
void cho_destin();          //prompts user to choose destination

class Drug
{
    public:
        Drug():avai(0) {}
        Drug(bool Avai):avai(Avai){}       
        bool& get_avai();                 
   
    protected:     
        bool avai;      //availabilty
};

class Town
{
    public:
        Town();      //constructor       
        void refresh();        //refreshes each drug's price       
               
    protected:             
        Drug drugs[8];       

};   

#endif

--------------------------------------------
//TOWN.CPP
#include "town.h"
#include <iostream>
#include <ctime>
using namespace std;

bool& Drug::get_avai()
{
    return avai;
}
     
void Town::refresh()
{
    if (drugs[0].get_avai)                                //ERROR!
        {
              cout << "ERROR!!!";
        }
   
}

---

town.cpp In member function `void Town::refresh()':
line 13 town.cpp could not convert `this->Town::drugs[0].Drug::get_avai()' to `bool
[Build Error] [town.o] Error 1

> if (drugs[0].get_avai) //ERROR!

To call the function you need parentheses:
if (drugs[0].get_avai())

Quote:

---

Originally posted by swoopy
> if (drugs[0].get_avai) //ERROR!

To call the function you need parentheses:
if (drugs[0].get_avai())

---

OMFG....thx alot