Thread: "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?

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;
};

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;
}

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 }

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())

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