C / C++ / MFC

26-Mar-08 9:46

I have to create a class called controller. I do not know what I need to include in this class: attributes and methods/getters/setters/operations?????? I understand the other classes involved in hangman, but not this one?????

Demonstrate the use of a class. It is strongly recommended that you choose a class to implement from the Hangman Tutorial, such as the Controller class.

Class must have a constructor.
Class must have two class member variables of different data types and getters and setters to access these variables.
Program must have 3 files, one header file, one .cpp file for that class and one Main.cpp.

#ifndef CONTROLLER_H

#define CONTROLLER _H

#include <string>

using namespace std;

class Controller

{

private:

string filename; // stores name of file where words are stored

// Add other member variables here

public:

Controller();

void start();

// Add accessor and mutator methods here for your member values

};

#endif // CONTROLLER_H

Now add the source file (Controller.cpp) & code the constructor.

#include "Controller.h"

Controller::Controller(string filename)

{

cout << "Inside the Controller Default Constructor << endl;

}

// Finish implementing methods here

First compile and execute the project to make sure you have typed everything correctly. Then modify your main() function to access your Controller class and use all methods.

Here is an example main() function*:

int main(int argc, char *argv[])

{

Controller controller;

// Call accessor and mutator methods for setting filename

// Start the game

return 0;

}

The Menu is not part of the actual hangman game itself, but rather allows us to add words to the game, play the game, and exit so it is a good candidate for its own class. The game itself can be a separate class so that we are not writing all of the code in main(). As a matter of a fact, let's plan on having very little code in main(), and let a separate class manage the interactions between the menu, game and the dictionary. We will call this class "Controller".

David Crow26-Mar-08 9:51

David Crow

26-Mar-08 9:51

djrasa wrote:
Class must have two class member variables of different data types and getters and setters to access these variables.

For example:

class Controller
{
public:
    void setFilename( const char *s ) { filename = s; }
    string getFilename( void ) { return filename; }
};

"Normal is getting dressed in clothes that you buy for work and driving through traffic in a car that you are still paying for, in order to get to the job you need to pay for the clothes and the car and the house you leave vacant all day so you can afford to live in it." - Ellen Goodman

"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne

djrasa26-Mar-08 14:32

26-Mar-08 14:32

I do not see how this will control the menu, dictionary, and game/ and no attributes (I am trying to understand, not trying to imply that it is incorrect{you would know before me : ) }). I thought that I needed two attributes (eg. int, char) and two functions for each to manipulate them. Sorry I am pretty new to this (retorical). Thank you for the fast response, much abliged.

David Crow26-Mar-08 17:13

David Crow

26-Mar-08 17:13

djrasa wrote:
I do not see how this will control the menu, dictionary, and game...

It won't. I was just providing you an example of how to add "getters and setters" to your existing class.

djrasa wrote:
I thought that I needed two attributes (eg. int, char) and two functions for each to manipulate them.

Which is what I showed you, only using the single member variable you already had.

djrasa27-Mar-08 4:35

27-Mar-08 4:35

I guess my real question is what is the point of the controller class (what has to be in it)?? What actions of the menu, dictionary, and game will it(controller class) control (won't these classes control there own actions???) It seems to me that you would not need this class (if not for assignment). Sorry, thanks for the response, I am just having a hard time understanding this class (for the most part I understand the menu, game, and dictionary classes).

Thanks again for being patient with me, and answering my questions.

Member 75496027-Mar-08 5:01

Member 754960

27-Mar-08 5:01

You are given a clue about the controller class in main: "Start the game"

If you are the game controller and I tell you "Start the game" what do you need to do?

djrasa27-Mar-08 7:55

27-Mar-08 7:55

call the function/method to display game board/word.

This is what I have come up with so far, need to add variables to methods still.
Is this close?????

<code></code>
class Controller //Class for Controller.
{
public:

void getgame();
void setgame ();
Controller();
void getmenuinput();
void setmenuinput ();

};
#endif //CONTROLLER_H
<code></code>

Member 75496027-Mar-08 8:35

Member 754960

27-Mar-08 8:35

Normally I would say "Stop trying to write code until you understand what you are trying to accomplish." However, here that is the only way you can demonstrate effort. When you know what you are tying to do it will be easier to see how the various classes/objects function.

Maybe I'd like to start a new game.
Maybe I'd like to quit the program.
Maybe I'd like to save the game I'm in.
Maybe...

Who displays these to the user? When?
Who gets the user selection? When?

So, controller says,

While not quit
    Display menu
    Get menu selection

Where are all these menu things located?

djrasa28-Mar-08 9:41

28-Mar-08 9:41

in the controller class? something like this?

<code></code>

class Controller
{

private:

string filename; //Store filename.
int Menu; //Store Menu.

public:

void setFilename(const char *s); //Method to set filename.
string getFilename(void); //Method to get filename.
int getmenu(int choice); //Method to get menu.
void setmenu(const int m); //Method to set menu.
Controller(); //Default constructor.
};
#endif
<code></code>

djrasa28-Mar-08 10:13

28-Mar-08 10:13

Controller.cpp file

<code></code>
#include "Controller.h"

Controller::Controller()
{
Menu = 0;
}

int Controller::getmenu(int choice)
{
return Menu;
}

void Controller::setmenu(const int m)
{
Menu = m;
}

string Controller::getFilename()
{
return filename;
}

void Controller::setFilename(const char *s)
{
filename = s;
}
<code></code>

djrasa28-Mar-08 10:14

28-Mar-08 10:14

Main.cpp

<code></code>
#include <iostream>
#include "Controller.h" //include controller header file

using namespace std;

int main()
{
int choice; //To hold screen to display message, until menu class is added.
int menu = 0;
const char *word = "Hangman";

Controller controller;
Controller display;

display.setmenu(menu); //Method to set the menu.

display.getmenu(menu); //Method to get the menu.

controller.setFilename(word); //Set filename to Hangman.

cout << "Coming soon - " << controller.getFilename() << endl; //Display temporary message.
cout << "To be continued......" << endl;
cout << endl;
cout << "Please click X in top right corner to exit." << endl;
cin >> choice; //Hold screen to display message.

return 0;

}
<code></code>

Exception when assigning char * to CString

led mike26-Mar-08 9:52

led mike

26-Mar-08 9:52

Thank you for posting your course assignment. We enjoy keeping abreast of the current CIS curriculum.

led mike

habs26-Mar-08 8:46

habs

26-Mar-08 8:46

I am catching an exception that is thrown when a char* is assigned to a CString. {The line - m_strResponse += CString(_T(pszOutBuffer));} I can print the char * pszOutBuffer to the screen, and it looks fine. Any ideas on how to get past this or how to further debug? The exception is caught in the second catch (...), and GetLastError() returns 0.
Thanks.

Code segments
from .h file:
public CString m_strResponse;
from .c file:

BOOL CMessage::HandleResponse(HINTERNET hRequest)<br />
{           <br />
        CString strError = "";<br />
        DWORD dwSize, dwDownloaded;<br />
        char *pszOutBuffer;<br />
<br />
        try<br />
        {<br />
        TRY<br />
        {<br />
            do <br />
            {<br />
                // Check for available data.<br />
                dwSize = 0;<br />
                if (!WinHttpQueryDataAvailable( hRequest, &dwSize))<br />
                {<br />
                    strError.Format("Error %u in WinHttpQueryDataAvailable.",GetLastError() );<br />
                    RecordError( strError, NULL ,"", "" );<br />
	   }<br />
<br />
                // Allocate space for the buffer.<br />
	   pszOutBuffer = new char[dwSize+1];<br />
<br />
                if (!pszOutBuffer)<br />
                {<br />
                    strError.Format("Error allocating buffer for HandleResponse()\n");<br />
                    RecordError( strError, NULL ,"", "" );<br />
                    return false;<br />
                }<br />
                else<br />
                {<br />
                    // Read the Data.<br />
                    ZeroMemory(pszOutBuffer, dwSize+1);<br />
                    <br />
                    if (!WinHttpReadData( hRequest, (LPVOID)pszOutBuffer,    dwSize, &dwDownloaded))<br />
                    {<br />
                        strError.Format ("Error %u in WinHttpReadData.", GetLastError() );<br />
                        RecordError( strError, NULL ,"", "" );<br />
                    }<br />
                    else<br />
                    {<br />
                        strError.Format("Try 1 -  outbuffer:%s, downloaded%u", pszOutBuffer, dwDownloaded);<br />
                        AfxMessageBox(strError);<br />
                        <br />
	           pszOutBuffer[dwSize] = '\0';<br />
	           m_strResponse += CString(_T(pszOutBuffer));<br />
                       <br />
                        strError.Format("response:%s", m_strResponse);<br />
                        AfxMessageBox( strError ) ;<br />
                    }<br />
<br />
                    // Free the memory allocated to the buffer.<br />
                    delete [] pszOutBuffer;<br />
                    pszOutBuffer = NULL;<br />
                            <br />
                }<br />
<br />
            } while (dwSize>0);<br />
<br />
                <br />
            // write response to tblAppDetInterfaceMessages.MessageReceived<br />
            CDBAccess::Instance()->RecordTransaction( m_strXMLMessage, m_strResponse);<br />
<br />
        }<br />
        CATCH_ALL(pEx)<br />
        {<br />
            RecordError( "Error occured while processing HTTP response", NULL ,"", (pEx->ReportError()) );<br />
<br />
            if( pszOutBuffer != NULL )<br />
                        delete[] pszOutBuffer;<br />
<br />
           return false;<br />
        }<br />
        END_CATCH_ALL <br />
        <br />
        }<br />
        <br />
        catch (...)<br />
        {<br />
            strError.Format ("Caught GetLastError: %u.",GetLastError());//at Index %d", GetLastError(), nTestIndex );<br />
            AfxMessageBox(strError);<br />
            return false;<br />
        }<br />
        <br />
        return true;<br />
}

Mark Salsbery26-Mar-08 9:39

Mark Salsbery

26-Mar-08 9:39

holly.sherman wrote:
m_strResponse += CString(_T(pszOutBuffer));

I can't readily see the cause of the exception, but using _T() in that fashion is incorrect.

You're reading data synchronously, right?

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

led mike26-Mar-08 9:48

led mike

26-Mar-08 9:48

Mark Salsbery wrote:
but using _T() in that fashion is incorrect.

I can't believe you got that far, I stopped looking after seeing this bowl of worms:

try<br />
{<br />
TRY<br />
{<br />
    do <br />
    {

led mike

David Crow26-Mar-08 9:53

David Crow

26-Mar-08 9:53

Come on, Mike, it's the old adage, "If at first you don't succeed, try and try again." Wink | ;)