Thread: Documentation

I could be just frustrated, but for some reason after I added documentation to my program, it no longer works as it did before. I appreciate anybody taking the time to glance at it and point out what I'm sure is a totally obvious error. I am receiving 'member function redeclaration not allowed' (on my show num2 function) and 'missing function header' on my add function, as well as several related errors. I have attached my header, implementation, and main() (application) file. Thanks so much.
P.S. Knowing that there are umpteen ways to improve this program, unfortunately I procrastinated and no longer have time to make improvements.

Search for "include guards".

Don't comment "end for" or "end if". That's obvious. It's like 3 lines away.

A guard such as pragma once or ifndef? I will remove the end if and end for.

Yes.

The pragma is not portable.

I inserted this into the top of my header file and my implementation file:

Code:

#ifndef FATINTEGERS_H
#define FATINTEGERS_H

and I have a mismatching ifndef error message. How am I doing this incorrectly?

(I should add that I've also tried only inserting it at the top of my implementation file, and in all three files, and this was the version that received the fewest errors.)

Code:

void CFatIntegers::showNum2(char c[])
} <--

heh.

Thank you. I appreciate that immensely.

If you look at your shownum2 function, you may notice that it somehow begins with

Code:

}

Edit: slow.

Still appreciated.

Originally Posted by marQade

(I should add that I've also tried only inserting it at the top of my implementation file, and in all three files, and this was the version that received the fewest errors.)

Include guards should be in header files, not in implementation files.

OK.
How would you have gone about formatting the numbers for display so they look nice? Such as:
1000
+ 500
--------
1500

Put it in a printf statement and put in some newlines (\n).

What have you tried?

I was using the iomanip library and trying to force setw() and setfill() in main(). I didn't, however, include the object name, such as:

Code:

cout << Centipede.setw(Centipede.getLength()+2) << Centipede.setfill(' ') << Centipede.showNum1(Centipede.num1) << endl;

I think the compiler was having a problem with this:

Code:

setw(Centipede.getLength()+2) << setfill(' ') << Centipede.showNum1(Centipede.num1) << endl;

So ... what did you do then?

You didn't define your own setw, so you can't call it through your class object. Maybe this is closer:

Code:

cout << setw(Centipede.getLength()+2) << setfill(' ');
Centipede.showNum1(Centipede.num1);
cout << endl;