Solved

Programming Stumper...

Posted on 2004-04-28
6
491 Views
Last Modified: 2006-11-17
The following code wasted 20 minutes of my time... Find the BUG!

// m_DEBUGFILE is in the class header as a CFile*
m_DEBUGFILE = new CFile();
CFileException fileException;

if ( !m_DEBUGFILE->Open( pszFileName, CFile::modeCreate | CFile::modeNoTruncate |
          CFile::modeReadWrite ), &fileException )
{
      // fileException.m_cause;
      delete m_DEBUGFILE;
      m_DEBUGFILE = NULL;
}
else
{
      m_DEBUGFILE->SeekToEnd();
      CString msg = "START DEBUGGING";
      m_DEBUGFILE->Write(msg,  msg.GetLength());
      m_DEBUGFILE->Write("\r\n", 2);
}

-- Thanks to microsoft for their wonderful API docs:
See: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vccore98/HTML/_core_opening_files.asp

-- Bubba
0
Comment
Question by:bganoush
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
6 Comments
 
LVL 3

Accepted Solution

by:
CoolBreeze earned 30 total points
ID: 10938440
lol.
this part does looks funny :)

if ( !m_DEBUGFILE->Open( pszFileName, CFile::modeCreate | CFile::modeNoTruncate |
          CFile::modeReadWrite ), &fileException )

the extra bracket
0
 
LVL 10

Assisted Solution

by:Mercantilum
Mercantilum earned 20 total points
ID: 10938856
In you if, the default 3rd argument of Open (exception) was set to NULL, and your if was always true, as in

  if (a, b, c)

the condition to be evaluated will be c.

The &fileException should have been in the Open statement,

  m_DEBUGFILE->Open( pszFileName, CFile::modeCreate | CFile::modeNoTruncate | CFile::modeReadWrite, &fileException)
0
 

Author Comment

by:bganoush
ID: 10939214

Good replies... and quick too!

Did you look at the sample code from Microsoft?   I always wondered why I can't copy some files ...  maybe microsoft actually based Windows on that sample?

-- Bubba
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
LVL 3

Expert Comment

by:CoolBreeze
ID: 10939357
yup, looked at the sample code, the mistake was from the it.
I guess, probably they won't be basing on that code sample? after all, the documentation group is a different group from the coding group (I think).
0
 
LVL 10

Expert Comment

by:Mercantilum
ID: 10940099
Lol :)
It seems their sample are specifically written for the doc, as the Open would be always in error!
0
 
LVL 62

Expert Comment

by:☠ MASQ ☠
ID: 10943696
lol!
The bug is here -----> microsoft.com

M :o)
0

Featured Post

Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Building a cohesive image for your brand is vital to making an impression on consumers. When the economy is tough, brands do better than unbranded  products. This can have a huge impact on your long-term profits, as the economy goes up and down.
Adults who share images on social media aren’t the only ones who need to worry about their privacy. Our culture’s tendency to share every move and celebration affects the privacy of our children, too.
Finds all prime numbers in a range requested and places them in a public primes() array. I've demostrated a template size of 30 (2 * 3 * 5) but larger templates can be built such 210  (2 * 3 * 5 * 7) or 2310  (2 * 3 * 5 * 7 * 11). The larger templa…

740 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question