|
Thank you very much for the response. Unfortunately, my problem is unchanged. 
Sorry if my question is vague.
To clarify: I only need to save these values to the file.
It is reassurring to see that you are using the same method that I have for passing the values to the ofstream. Notice that portion in the code below.
The function definition and function call (with arguments) are as previously posted and as you have described.
Now I am beginning to think that the problem does not exist in the
function call --> function --> file open and output.
Rather, I am beginning to think the problem lies in the:
assigning values --> function call --> function
Please, take a look at the way I am assigning the values prior to handing them to the function call as arguments.
-snip-
//from bread.h
class Bread <br />
{<br />
public:<br />
<br />
void setValues(double fat,double carb,int calor,double fiber);<br />
<br />
std::string strFileName;<br />
double dblFat;<br />
double dblCarb;<br />
double dblFiber;<br />
int intCalor;<br />
};
//from Bread.cpp
<br />
void dataOut(std::string foodName, int Calor, double wtr, double prt, double fat,double carb,double fib,double vita,double vitb, double vitc, double calci)<br />
{<br />
std::fstream<br />
appendFile("Tally.txt");<br />
<br />
<br />
if(!appendFile.is_open())<br />
{<br />
std::ofstream<br />
outputFile("Tally.txt");<br />
<br />
if(!outputFile.is_open())<br />
{<br />
std::cerr << "\nNot enough memory to create file\n";<br />
exit(1);<br />
}<br />
else<br />
{<br />
outputFile << << "\n\nFood: "<< foodName << "\n"<br />
<< "Calories: "<< Calor << "\n"<br />
<< "Water (oz): "<< wtr << "\n"<br />
<< "Protein: "<< prt << "\n"<br />
<< "Fat: "<< fat << "\n"<br />
<< "Carbohydrates: "<< carb << "\n"<br />
<< "Fiber: "<< fib << "\n"<br />
<< "Vitamin A: "<< vita << "\n"<br />
<< "Vitamin B: "<< vitb << "\n"<br />
<< "Vitamin C: "<< vitc << "\n"<br />
<< "Calcium: "<< calci << "\n";<br />
}<br />
}<br />
else<br />
{<br />
appendFile.seekp(0,std::ios::end);
<br />
appendFile << "\n\nFood: "<< foodName << "\n"<br />
<< "Calories: "<< Calor << "\n"<br />
<< "Water (oz): "<< wtr << "\n"<br />
<< "Protein: "<< prt << "\n"<br />
<< "Fat: "<< fat << "\n"<br />
<< "Carbohydrates: "<< carb << "\n"<br />
<< "Fiber: "<< fib << "\n"<br />
<< "Vitamin A: "<< vita << "\n"<br />
<< "Vitamin B: "<< vitb << "\n"<br />
<< "Vitamin C: "<< vitc << "\n"<br />
<< "Calcium: "<< calci << "\n";<br />
}<br />
}<br />
<br />
int main()<br />
{<br />
breadobject.setValues( 1.1 , 11.4 , 60 , 0.9 );<br />
breadobject.strFileName="White Slice";<br />
<br />
dataOut(breadobject.strFileName, breadobject.intCalor, 2.2, 0.0, breadobject.dblFat, breadobject.dblCarb, breadobject.dblFiber, 0.0, 0.0, 0.0, 0.0); <br />
<br />
return 0;<br />
}
WHEW!
Travis
|
|
|
|
|
Hi Travis,
You asked me to check the way you're assigning values
prior to sending them to the function call as arguements. I assume you are doing that in the setValues function member of Bread. Regrettably I see the declaration of setValues but not its definition so I can't help you there.
What I'd like you to do for me is to cut your code back to pretty much the smallest amount that succesfully compiles and demononstrates your problem, and post that complete code for me to compile and debug. I reckon that will be the fastest way of getting to the bottom of this.
Cheers,
David
|
|
|
|
|
Now Im getting SOMETHING anyway...
I put an amperstand in front of the calling argument like so:
SendThisInfoToTextFile( yada.yada, &yada.yada);
ANd I placed an asterisk at the receiving end:
SendThisInfoToTextFile( string yada1, int* yada2 ) {...send stuff to text file...}
now, my text file is receiving this:
CorrectYada1 <---correct output on line 1
0012EF44 <---Looks like the address of ...umm...Yada2
New theory: ofstream is for strings and not ints or doubles
feedback please?
If you check the posting times, you will see that I have been working on this all night.
must ... keep eyes... open ...
Travis
|
|
|
|
|
Hi Travis
I have big doubts about your new theory (ofstream is for strings and not ints or doubles). Streams should handle all data types. In the code I provided to you, streams handled string, int and double data types properly - both for storing to a file and reading it back.
You are right about 0012EF44 being the address of something. I would expect this to see this though if you stream the address of variables or pointers to variables to a file.
Once again, see if you could post a small amount of code which demonstrates your problem and is complete
enough to compile, and I'll check it for you.
Cheers,
David
|
|
|
|
|
Hi Travis,
I compiled your code, (after inventing a few missing bits) and it works fine for me. Your definition of setValues should look like this.
void Bread::setValues(double fat,double carb,int calor,double fiber)
{
dblFat = fat;
dblCarb = carb;
intCalor = calor;
dblFiber = fiber;
}
The contents of Tally.txt looks like this...
Food: White Slice<br />
Calories: 60<br />
Water (oz): 2.2<br />
Protein: 0<br />
Fat: 1.1<br />
Carbohydrates: 11.4<br />
Fiber: 0.9<br />
Vitamin A: 0<br />
Vitamin B: 0<br />
Vitamin C: 0<br />
Calcium: 0
One possible explanation for your problem....
Did you declare dblFat, dblCarb, intCalor and dblFiber as local variables inside setValues. If you did, you will be setting the values of local variables, and not member variables of Bread. The member variables of Bread would remain undefined and you would indeed get garbage if you streamed them to a file.
In any case, it is a good idea to set reasonable default values for member variables of a class in its constructor to avoid this sort of trap.
Cheers
David
|
|
|
|
|
I inserted your code and it works now.
![Rose | [Rose]](https://codeproject.global.ssl.fastly.net/script/Forums/Images/rose.gif) 
Im still waking up. Il give some details shortly.
Travis
|
|
|
|
|
Your version...
void Bread::setValues(double fat,double carb,int calor,double fiber)
{
dblFat = fat;
dblCarb = carb;
intCalor = calor;
dblFiber = fiber;
}
My old version...
void Bread::setValues(double fat,double carb,int calor,double fiber)
{
double
tmpfat = fat,
tmpcrb = carb,
tmpfbr = fiber;
int
tmpclr = calor;
}
...  Sad is the day when a man realizes that he is an idiot! 
Thank you so much for helping me to get this working. My program should be completed on time.
Travis
|
|
|
|
|
Hi Travis,
I'm happy to help, and I'm glad your code is working now.
One final suggestion if I may.
Its a little unusual in C++ to create a class object and then set it's values. Normally you would use a constructor to either assign values when the object is created, or set values to a sensible default. That way, whenever the object is used, it's member variables are guaranteed to have assigned values, and the problem you experienced would never arise.
In your case, your code could look something like this.
class Bread
{
public:
Bread(double fat,double carb,int calor,double fiber,
std::string filename);
void setValues(double fat,double carb,int calor,double
fiber);
std::string strFileName;
double dblFat;
double dblCarb;
double dblFiber;
int intCalor;
};
Bread.cpp might look something like this....
Bread::Bread(double fat,double carb,int calor,double
fiber,std::string filename)
{
dblFat = fat;
dblCarb = carb;
intCalor = calor;
dblFiber = fiber;
strFileName = filename;
}
int main()
{
Bread breadobject(1.1, 11.4, 60, 0.9, "White Slice");
dataOut(breadobject.strFileName, breadobject.intCalor,
2.2, 0.0, breadobject.dblFat, breadobject.dblCarb,
breadobject.dblFiber, 0.0, 0.0, 0.0, 0.0);
return 0;
}
Its your code of course, and this approach might not suite your purposes. Its all up to you.
Cheers
Dave
|
|
|
|
|
This may sound simple for you, but can anayone tell me how to insert a dialog box in an MDI app. I want the dialog box with several options to appear and float in front of the MDI!
|
|
|
|
|
Your question is not very clear. Do you want to insert dialogbox over an open MDI window. If yes, then create a new dialog in resources and set its type to child and invoke it at whatever event you want.
Otherwise, if you want to have a dialog appeared at some event. Just DoModal it.
It would be better if you elaborate your question a little.
"Aerodynamically, the bumble bee shouldn't be able to fly, but the
bumble bee doesn't know it so it goes on flying anyway." - Mary Kay Ash
My Articles
|
|
|
|
|
If I understand you correctly, you need to create a modeless dialog.
Maximilien Lincourt
Your Head A Splode - Strong Bad
|
|
|
|
|
Hi,
I want to enable all the hot keys in my MFC application (in menu items, dialogs etc) after launching the application without pressing the ALT key. Is this possible, if yes how and where do I write the code.
eg. MS-Word shows the hot keys for menus when launched without pressing the ALT key.
At present the hot keys in my application are enabled only after I hit the ALT key.
Thanks,
PRJ
|
|
|
|
|
I think you should look for accelarators instead of Hotkeys.
i.e. you think ,you want your application to repond to Ctrl+s key combination for saving like it in MSWord.
is i am right??
"I Think this Will Help"
[Vote One Here,.....]
<h5
alok gupta="" <br=""> visit me at http://www.thisisalok.tk
|
|
|
|
|
Right_Clk_Desktop->Properties->Appearance->Effects.
Uncheck ->"Hide underlined letters for Keyboarduntill Alt Key Pressed"
suhredayan
There is no spoon.
|
|
|
|
|
Here is the scenario. I'm wondering why it won't work and how to fix it. Ms Visual C++.net. Create dialog based MFC Application called MyProject. Add function called MyFunction declared at top of ProjectDlg.cpp as follows:
int MyFunction(int num1, int num2);
Add defintion to bottom of ProjectDlg.cpp file:
int MyFunction(int a, int b)
{
UpDateData(TRUE);
return a+b;
}
Use as follows inside a button:
sum = MyFunction(int num1, int num2);
This doesn't work. I get the following message:
error C3861: 'UpDateData': identifier not found, even with argument-dependent lookup
Is there some way I should declare or define MyFunction in order that it can use the UpdateData function within my function?
|
|
|
|
|
a simple way is make MyFunction() a member function of your CProjectDlg class.
suhredayan
There is no spoon.
|
|
|
|
|
Thank you for the response. Here's what I did.
in the MyprojectDlg.h file I put the following:
class CMyProjectDlg : public CDialog
{
int CMyProjectDlg::MyFunction(int num1, int num2);
}
Alas, same error message:
error C3861: 'UpDateData': identifier not found, even with argument-dependent lookup
(this error occurs on the line using UpdateData() within MyFunction() but not when I use it outside of MyFunction)
Is this not the correct way to make MyFunction() a member of the MyProjectDlg class or is there some other problem, perhaps with the function definition? I would greatly appreciate any further help.
ss
|
|
|
|
|
SeanSheehan wrote:
error C3861: 'UpDateData': identifier not found, even with argument-dependent lookup
it is CWnd::UpdateData()
Not UpDateData()
Jetli
conclusion means Coming to wrong Decision with confidence
|
|
|
|
|
As per the suggestion I changed MyFunction() definition to:
int MyFunction(int a, int b)<br />
{ <br />
CWnd::UpDateData(TRUE);<br />
return a+b;<br />
}
However I now get two error messages:
error C2039: 'UpDateData' : is not a member of 'CWnd'
error C3861: 'UpDateData': identifier not found, even with argument-dependent lookup
|
|
|
|
|
SeanSheehan wrote:
class CMyProjectDlg : public CDialog
{
int CMyProjectDlg::MyFunction(int num1, int num2);
}
Change as follows:
class CMyProjectDlg : public CDialog
{
int MyFunction(int num1, int num2);
}
And in ur cpp file the definition should look like this:
int CMyProjectDlg::MyFunction(int num1, int num2)
{
..
..
..
}
suhredayan
There is no spoon.
|
|
|
|
|
Made the suggested changes to both the delclaration and definition. The declaration is now:
class CMyProjectDlg : public CDialog<br />
{<br />
int MyFunction(int num1, int num2);<br />
}
in MyProjectDlg.h, and the definition is now:
int CMyProjectDlg::MyFunction(int a, int b)<br />
{ <br />
UpDateData(TRUE);<br />
return a+b;<br />
}
in MyProjectDlg.cpp, yet the same error message persists:
error C3861: 'UpDateData': identifier not found, even with argument-dependent lookup
Maybe this is not such a simple problem.
ss
|
|
|
|
|
SeanSheehan wrote:
UpDateData(TRUE);
Change to UpdateData(TRUE);
suhredayan
There is no spoon.
|
|
|
|
|
Thats what i have told earlier with but he doesn't get the Simplest thing.
Jetli
Constant Thing In World Is Change.
|
|
|
|
|
Jetli Jerry wrote:
he doesn't get the Simplest thing.
I still remember those days of
suhredayan
There is no spoon.
|
|
|
|
|
He's probably not observing the change in case.
Artificial intelligence is no match for natural
stupidity.
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
