|
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.
|
|
|
|
|
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?
|
|
|
|
|
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>
|
|
|
|
|
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?
|
|
|
|
|
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>
|
|
|
|
|
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>
|
|
|
|
|
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>
|
|
|
|
|
Thank you for posting your course assignment. We enjoy keeping abreast of the current CIS curriculum.
led mike
|
|
|
|
|
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 />
dwSize = 0;<br />
if (!WinHttpQueryDataAvailable( hRequest, &dwSize))<br />
{<br />
strError.Format("Error %u in WinHttpQueryDataAvailable.",GetLastError() );<br />
RecordError( strError, NULL ,"", "" );<br />
}<br />
<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 />
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 />
delete [] pszOutBuffer;<br />
pszOutBuffer = NULL;<br />
<br />
}<br />
<br />
} while (dwSize>0);<br />
<br />
<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());
AfxMessageBox(strError);<br />
return false;<br />
}<br />
<br />
return true;<br />
}
|
|
|
|
|
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]](https://codeproject.global.ssl.fastly.net/script/Forums/Images/coffee.gif)
|
|
|
|
|
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
|
|
|
|
|
Come on, Mike, it's the old adage, "If at first you don't succeed, try and try again." 
"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
|
|
|
|
|
LMAO
It must be the Texas version: "try and try and do, can't be fooled again".
led mike
|
|
|
|
|
You should post that code to the Coding Horrors one.
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
|
|
|
|
|
 ROTFL. I am just wondering why people like Mark, David, etc., are making fun of the OP. Crazy guys. Your reply is the only one which makes sense. 5 for that.
Nobody can give you wiser advice than yourself. - Cicero
.·´¯`·->Rajesh<-·´¯`·.
Codeproject.com: Visual C++ MVP
|
|
|
|
|
I get the same results when I remove _T().
Yes, I'm reading data synchronously.
I tried writing to the CString character by character too, but it threw the exception at the first character.
for(nTestIndex = 0; nTestIndex <= dwSize; nTestIndex++)
{
m_strResponse += pszOutBuffer[nTestIndex];
}
|
|
|
|
|
m_strResponse should really be a CStringA, not a CString.
Then, maybe try stepping into the CStringA += operator code to see why an exception
is being thrown.
Mark
Mark Salsbery
Microsoft MVP - Visual C++
|
|
|
|
|
Why CStringA? MSDN doc says CString does both ANSI and Unicode, while CStringA does just ANSI. Why would that make a difference?
Stepping through code not exactly an option... Long story, but this is old code (5 yrs) that mysteriously broke at that point when I made a change to the request xml. In other words, this code worked before, but doesn't anymore. I suppose I could write a small test for that one line, but http connection won't work from my testing environment.
|
|
|
|
|
habs wrote: Why CStringA?
To match your incoming data. If the data is coming in Unicode, then use CStringW.
Any other case requires potential conversion of the data, depending on your build configuration.
Is the debugger no help? There's not all that many reasons the CString += operator
should fail.
Mark
Mark Salsbery
Microsoft MVP - Visual C++
|
|
|
|
|
The following test code works without any issues:
int main(int argc, char* argv[])<br />
{<br />
char * pszOutBuffer;<br />
pszOutBuffer = "test";<br />
CString m_strResponse;<br />
m_strResponse += CString(pszOutBuffer);<br />
<br />
<br />
printf(m_strResponse);<br />
<br />
<br />
return 0;<br />
}
I added all this header info from the original program:
#include <afxwin.h>
#include <afxext.h>
<br />
#ifndef _AFX_NO_OLE_SUPPORT<br />
#include <afxole.h>
#include <afxodlgs.h>
#include <afxdisp.h>
#endif // _AFX_NO_OLE_SUPPORT<br />
<br />
<br />
#ifndef _AFX_NO_DB_SUPPORT<br />
#include <afxdb.h>
#endif // _AFX_NO_DB_SUPPORT<br />
<br />
#ifndef _AFX_NO_DAO_SUPPORT<br />
#include <afxdao.h>
#endif // _AFX_NO_DAO_SUPPORT<br />
<br />
#include <afxdtctl.h>
#ifndef _AFX_NO_AFXCMN_SUPPORT<br />
#include <afxcmn.h>
#endif // _AFX_NO_AFXCMN_SUPPORT<br />
<br />
#pragma warning ( push )<br />
#pragma warning ( disable : 4786 )<br />
#include <map><br />
#include <string><br />
#include <strstream><br />
#include <fstream><br />
<br />
std::ostream& operator<< ( std::ostream& os, CString& cs );
I'm having trouble switching this to CStringA... Did I mention this is Visual Studio 6.0?
|
|
|
|
|
Almost always, if you use a catch-all you should re-throw the exception again.
Steve
|
|
|
|
|
I'm assuming that none of you know the answer or have anything helpful to say since you have resorted to making fun of this code... (apologies to Mark, who at least seems to be trying)
I added the extra try{}catch(...){} because the macro didn't catch the exception, and I didn't know if just adding catch at the end of the macro would work. I also tried adding catch(CException &e), but that didn't catch anything. I don't know to get to the exception that is caught in the (...), and GetLastError() only returns 0. I have pulled the assignment code out and put it in a separate program (see above), and it works fine.
This lovely little segment of code even worked before, but doesn't work anymore (yay, maintenance).
So anyways, unless you know how to tell me how to get at the exeception, or how to debug, or how to fix... bug off.
modified on Thursday, March 27, 2008 10:04 AM
|
|
|
|
|
habs wrote: I'm assuming that none of you know the answer or have anything helpful to say since you have resorted to making fun of this code
Speaking only for myself here. Since I am not being paid to analyze that monkey crap you posted I didn't even try to "know the answer". My answer ( and based on the history around here I choose not to post) would be, if I had to maintain that monkey crap, to select the entire thing and press the delete key. Then develop a reasonable solution from scratch.
led mike
|
|
|
|
|
You haven't provided enough evidence that the diagnosis of the problem is correct.
You might try to:
- replace the TRY-CATCH_ALL macros
- use a try-catch on the "Read the Data" block rather than the whole function
- properly handle the case of WinHttpQueryDataAvailable failure.
- verify that m_strResponse is valid before you attempt to concatinate to it.
It may have "worked" before; I would add "by the grace of God." The code is flawed.
|
|
|
|
|
Thanks for some suggestions. I'm not certain how to implement all of them.
Member 754960 wrote: - properly handle the case of WinHttpQueryDataAvailable failure.
Are you talking about missing a return false; ? I can add that in, but I actually get a response, dwSize 753, so I don't think that's the problem.
Member 754960 wrote: - verify that m_strResponse is valid before you attempt to concatinate to it.
How would I validate m_strResponse?
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
