Error: no `int ClMath::colorRegulator(int, int)' member function declared in class

Hey all,

I've searched on the internet, a found about 4 sites/forums where people had the same problem, except none of the solutions that were listed worked for me. I really don't know what's wrong, so either my compiler is acting up or I'm missing something so obvious that I don't even realize it. Here's the code:

Code:

//ClassMath.h

#ifndef CLASS_MATH_H
    #define CLASS_MATH_H

#include <cmath>
#include <cstdlib>
#include "ClassRange.h"

using namespace std;

class ClMath {
    public:

    static long double dPI;

    //Rounds value up or down
    static long int round(long double dNum);

    //Returns random value in range
    static long int randNum(ClRange<long int> rnRange);

    static int colorRegulator(int x, int r);
};

#endif

Code:

//ClassMath.cpp

#include "ClassMath.h"

using namespace std;

long double ClMath::dPI = 4 * atan(1);

long int ClMath::randNum(ClRange<long int> rnRange)
{
    if((rnRange.getUpper() > RAND_MAX) && ((rnRange.getUpper() - rnRange.getLower()) > RAND_MAX)) {

        long int RandNum = rand();
        long int nNumRands = static_cast<long int>((rnRange.getUpper() - rnRange.getLower()) / RAND_MAX);

        RandNum += (rand() % nNumRands ) * RAND_MAX;

        if ((rnRange.getUpper() - rnRange.getLower()) != RAND_MAX * nNumRands)
            RandNum += rand() % ((rnRange.getUpper() - rnRange.getLower()) - (RAND_MAX * nNumRands) + 1);

        return RandNum + rnRange.getLower();
    }

    else
        return rand() % (rnRange.getUpper() - rnRange.getLower() + 1 ) + rnRange.getLower();
}

long int ClMath::round(long double dNum)
{
    if (static_cast<long int>(ceil(dNum + 0.5)) > static_cast<long int>(ceil(dNum)))
        return static_cast<int>(ceil(dNum));
    else
        return static_cast<int>(floor(dNum));
}

int ClMath::colorRegulator(int x, int r)
{
    return static_cast<int>((pow(pow(255.0, x) * x / r, 1/x)));
}

And the error is C:\...\ClassMath.cpp|35|error: no `int ClMath::colorRegulator(int, int)' member function declared in class `ClMath'.

I'm using CodeBlocks with MinGW, and I've never had any problems before, so I hope this isn't the first.

Cheers,

Gabe

Is this the only error and the first error?

Yup. The program works fine, I'm just tweaking it a bit so it'll look better, and once I add that function it won't compile.

Perhaps you could also post "ClassRange.h" and the main function?

I don't have any problems with that (as one file, but ordered to minimize dependancies. E.g ClMath doesn't need to include any headers.)

Code:

template <class T>
class ClRange;

class ClMath {
    public:

    static long double dPI;

    //Rounds value up or down
    static long int round(long double dNum);

    //Returns random value in range
    static long int randNum(const ClRange<long int>& rnRange);

    static int colorRegulator(int x, int r);
};


template <class T>
class ClRange
{
   T from, to;
public:
    ClRange(T from, T to): from(from), to(to) {}
    T getLower() const { return from; }
    T getUpper() const { return to; }
};

#include <iostream>
int main()
{
    std::cout << ClMath::randNum(ClRange<long int>(0, 128)) << '\n';
    std::cout << ClMath::colorRegulator(34, 45) << '\n';
}



#include <cmath>

using namespace std;

long double ClMath::dPI = 4 * atan(1);

long int ClMath::randNum(const ClRange<long int>& rnRange)
{
    if((rnRange.getUpper() > RAND_MAX) && ((rnRange.getUpper() - rnRange.getLower()) > RAND_MAX)) {

        long int RandNum = rand();
        long int nNumRands = static_cast<long int>((rnRange.getUpper() - rnRange.getLower()) / RAND_MAX);

        RandNum += (rand() % nNumRands ) * RAND_MAX;

        if ((rnRange.getUpper() - rnRange.getLower()) != RAND_MAX * nNumRands)
            RandNum += rand() % ((rnRange.getUpper() - rnRange.getLower()) - (RAND_MAX * nNumRands) + 1);

        return RandNum + rnRange.getLower();
    }

    else
        return rand() % (rnRange.getUpper() - rnRange.getLower() + 1 ) + rnRange.getLower();
}

long int ClMath::round(long double dNum)
{
    if (static_cast<long int>(ceil(dNum + 0.5)) > static_cast<long int>(ceil(dNum)))
        return static_cast<int>(ceil(dNum));
    else
        return static_cast<int>(floor(dNum));
}

int ClMath::colorRegulator(int x, int r)
{
    return static_cast<int>((pow(pow(255.0, x) * x / r, 1/x)));
}

Sure, although I doubt that will help you much; main is a mess, I haven't cleaned it up yet, and ClassRange has been used in a couple of programs before without any problems

Code:

//ClassRange.h

#ifndef CLASS_RANGE_H
    #define CLASS_RANGE_H

using namespace std;

template <typename T>
class ClRange{
    private:
        T m_tLoBound;
        T m_tHiBound;

    public:
        ClRange(T tLoBound, T tHiBound)
                :m_tLoBound(tLoBound), m_tHiBound(tHiBound)
                {}
        ClRange()
                :m_tLoBound(0), m_tHiBound(0)
                {}
        ~ClRange() {}

        T getLower() { return m_tLoBound; }
        T getUpper() { return m_tHiBound; }

        void setLower(T tValue) { m_tLoBound = tValue; }
        void setUpper(T tValue) { m_tHiBound = tValue; }
        void set(T tLower, T tUpper) { m_tLoBound = tLower; m_tHiBound = tUpper; }
};

#endif

Code:

//main.cpp

#include <cstdlib>
#include <ctime>
#include "ClassUniverse.h"
#include "ClassValues.h"

using namespace std;

int main()
{
    allegro_init();

    install_timer();
    install_keyboard();

    set_color_depth(32);
    if (set_gfx_mode(GFX_AUTODETECT_WINDOWED, ClValues::nScreenHight, ClValues::nScreenWidth, 0, 0) < 0) {
        allegro_message("Couldn't set gfx mode: %s\n", allegro_error);
        exit(1);
    }

    ClValues::dblbuffer = create_bitmap(ClValues::nScreenHight, ClValues::nScreenWidth);
    clear(ClValues::dblbuffer);

    srand(time(NULL));

    ClUniverse TheUniverse;

    while(!key[KEY_ESC]) {
        TheUniverse.calculateDisplacement();
        TheUniverse.drawObjects();
    }

    return 0;
}
END_OF_MAIN();

Hope it helps.

Well, when I include the definition in the class, as in

Code:

#ifndef CLASS_MATH_H
    #define CLASS_MATH_H

#include <cmath>
#include <cstdlib>
#include "ClassRange.h"

using namespace std;

class ClMath {
    public:

    static long double dPI;

    //Rounds value up or down
    static long int round(long double dNum);

    //Returns random value in range
    static long int randNum(ClRange<long int> rnRange);

    static int colorReg(int x, int r) { return static_cast<int>((pow(pow(255.0, x) * x / r, 1/x))); }
};

#endif

it works fine. So now I'm really lost.

Not really

There doesn't seem anything technically wrong with these files, so perhaps you could try a full rebuild and see if any other headers introduce circular dependancies.

All I can say is that I don't see why something with only static functions should be a class (global functions/variables, perhaps in a namespace to avoid collisions with other libraries would be just as good).

You can also try reducing dependencies (see code above how header of ClMath can be made depend on no other headers) and get rid of the using namespace std; in headers (as you don't need anything from that anyway).

ClassMath is only dependent on <cmath>, <cstdlib> and "ClassRange.h". It needs the former two and while it doesn't need the last, I still don't see how that could be preventing a successful compilation, especially since the new function doesn't even use ClRange. As for why I'm making a class with only static functions, I guess it's personal preference. I started learning Java some time ago, and since then I prefer having everything enclosed in a class.

EDIT: Oh, and I already tried completely re-building the whole thing; didn't help. Oh well, guess I'll have to live with the fact that one of the function definitions is going to have to be included with the declaration. Thanks for your help.

ClassMath is only dependent on <cmath>, <cstdlib> and "ClassRange.h"

The declaration doesn't depend on <cmath> and <cstdlib>, and the dependancy on "ClassRange.h" can be removed with a forward declaration. It is the implementation that depends on them, which means they can be included in the cpp file.

If you don't mind, you could attach all the files, so it would be possible to take a look.

Oh...well, again because of personal preference, I decided to include all the files in the headers, since the .cpp files include the headers plus everything the headers include. Didn't know it could cause problems.

The next part is just short of being hilarious (if it weren't so serious of course ). I was uploading the files as you requested, and by chance noticed there was a ClassMath.h.gch file in the folder. So I went and removed that, and bingo, the program works. So, thank you for your help AND for your request I guess

On a more serious note, anyone know what that file is? I still have it here, so I can upload it if anyone wants to have a look... a search on Google returned "(a .gch is basically a dump of the internal memory state of GCC, which can differ with each version)."...any idea on how that could have affected the program?

It looks like you are (perhaps unknowningly) using precompiled headers. I haven't used them myself.

I guess this goes hand in hand with a problem I had when I first compiled the program, when for some reason ClassMath.h had "compile file" and "link file" checked in its properties. Anyway, I fixed that, but missed this...I had a feeling that it was somehow going to be linked to this problem, but I didn't see how so I didn't even mention it. Probably should have.

Thanks again