Using classes in function outside main

Please help me figure out what I'm doing wrong.

I have a class, declared in a header file:

Code:

// std_dice.h 
// header file for standard system include files,
// project include files 

#include <stdio.h>
#include <tchar.h>

// additional headers 
#include <iostream>
#include <fstream>
#include <stdlib.h>
#include <time.h>
namespace DICE {
  class SingleDie {
    public:
    int nRolled;
    public:
    void RollDie();
  };
}

void fStupidFunction();

Also declared was a stupid little function, to be used in main. It replaces a larger function, for the question, that was / is going to collect some statistics about rolled dice.

Here is main.cpp, including the class definition, the main function, and the function where I can't properly use the class.

Code:

// main.cpp 
#include "std_dice.h"
using namespace std;

int main(){ 
/***************Classes here********************************/
  // set up the SingleDie class
  class SingleDie {
    public:
    int nRolled;
    
    // constructor
    SingleDie() {
      cout << "Constructor performed \n\n";  // for testing
      nRolled = 0;
    } // SingleDie constructor
    
    public: 
    void RollDie() {
      nRolled = 1;
      nRolled = 10 + rand() % 60;
      nRolled /= 10;
    } // RollDie
  
  }; //SingleDie class

/***************Main testing here*******************************************/  
  using namespace DICE;
  SingleDie ssss;
  ssss.RollDie();
  cout << ssss.nRolled << "  = Value of nRolled in main \n\n";
  fStupidFunction();  
 
// pause after executing console application
 system("PAUSE");
    return EXIT_SUCCESS;
return 0;
}//main

/*********************Functions Here****************************/  
  void fStupidFunction() {
    //using namespace DICE;
    DICE::SingleDie sOneDice;
    sOneDice.nRolled = 7;  // Works: member variable is public for now
    sOneDice.RollDie();    // Doesn't work!
    // If I comment out the above line, the program compiles.
    cout << sOneDice.nRolled << " In the stupid little function! \n\n";
  };

As per the comments in the function: fStupidFunction: I can access the member variable .nRolled just fine. (I'll later fix the class so that there is a member function to set .nRolled, make it private, and make sure it never gets set to 7.)

In fact, if I comment out the line SOneDice.RollDie(); then the program compiles and runs. Therefore, I can suppose that my namespace is okay and that the class variable is declared okay--or can I? The problem comes when I try to use the member function RollDie().

RollDie() works in main but not in fStupidFunction.

It doesn't make sense to me that one cannot use a class variable in a function--only in main. I've looked in Bjarne's book (which is way above my head anyway), in C++ for Dummies (which doesn't cover the issue), and in various tutorials. If someone can help me to understand what I've done wrong and how to fix it, I would be very appreciative.

Thanks,
Keith

Your function RollDie is not defined. (Yes, I can see a definition in main(), but that definition is local to main() and therefore doesn't count.) All the class definition stuff should be in a separate file, not included in some other random function.

Originally Posted by tabstop

Your function RollDie is not defined. (Yes, I can see a definition in main(), but that definition is local to main() and therefore doesn't count.) All the class definition stuff should be in a separate file, not included in some other random function.

Thanks!!! That fixed it... I moved the line with main(){ down to where I had the comment about main testing. (Makes sense, but I'm still very new at this stuff!!)

Then I had to move the constructor to the header file and I left the other class member function up at the top. Probably ought to move it into the header file later...

Anyway, I beat my head against the wall for hours not noticing what you told me about, so

THANKS!!!!!

Are you trying to save memory by excluding the "c" from "dice" in "RollDie" and "SingleDie"?

J/k... but still, readability can sometimes be good. One could think this was a class about a dog which has the ability to roll over and die.

Originally Posted by DrSnuggles

Are you trying to save memory by excluding the "c" from "dice" in "RollDie" and "SingleDie"?

J/k... but still, readability can sometimes be good. One could think this was a class about a dog which has the ability to roll over and die.

LOL! One die plus one die equals two dice.

FWIW, the result will be some stats about this silly little dice game some friends taught us after dinner a few weeks ago. It's similar to Cosmic Wimpout, but you use 6 regular dice. Aces are worth 100. Fives are worth 50. Three of a kind: three deuces = 200. Four deuces = 400. Five of them = 600. Six would be 1,200. Three sixes = 600. Three fives = 500. Three aces = 1,000. Game is 15,000. We've been playing it since then, my wife and I, at dinner or at breakfast, and the odds don't always work out the way I would expect them to. Thus, a good excuse to get back into some programming for fun. It's been almost a decade since I took a c++ intro course (for fun) and I was just looking for an excuse. The dice were it.

I may even have something worthy of sharing when I get done--with any luck.

Keith

The problem is that you aren't implementing your class, but rather defining it twice. One inside the DIE namespace and one outside.
There are class tutorials on the site. I suggest you read them for better understanding on how to create and implement classes.

Originally Posted by Bladeforger

LOL! One die plus one die equals two dice.

FWIW, the result will be some stats about this silly little dice game some friends taught us after dinner a few weeks ago. It's similar to Cosmic Wimpout, but you use 6 regular dice. Aces are worth 100. Fives are worth 50. Three of a kind: three deuces = 200. Four deuces = 400. Five of them = 600. Six would be 1,200. Three sixes = 600. Three fives = 500. Three aces = 1,000. Game is 15,000. We've been playing it since then, my wife and I, at dinner or at breakfast, and the odds don't always work out the way I would expect them to. Thus, a good excuse to get back into some programming for fun. It's been almost a decade since I took a c++ intro course (for fun) and I was just looking for an excuse. The dice were it.

I may even have something worthy of sharing when I get done--with any luck.

Keith

Aha I didn't know that one "die" could also be one dice. When I translate the swedish word for one "tärning" I get "dice", but if i look up the english word die there is also that translation.

I know that game you're talking about, I used to play it a lot one summer with my friends. Do share if you get something working, dice games can be a good timepasser.