It reminds me my last school project how I did not include a highscore system. I was making a adventure type game and they wanted us to put a highscore system in our game. WTF? I just decided to ditch it. It costed me 20 pts out of 100. However I didn't care because I was still the best person in the class in average for the entire year and it did not affect my overall school classes average enough. I got 80% on the game, btw

I think I should point out before my teacher gets bashed too bad that he was the teacher that taught me TI-Basic...

Lol

If he does it again just turn in one of your calc games to him instead since he apparently knows TI-BASIC

I showed him Exodus a couple of weeks ago.  I got the same response.
Text sprites FTW!

What Quigibo wrote is very true. You're talking to a self-taught programmer that went through the trials and tribulations of learning everything, that he already knew, in AP Computer Science. As you grow as a programmer you will come to realize the reasons why they teach the way they do.

At any rate, at face value those two pieces of code appear to do the same thing correct? But have you compiled it on every compiler new and old? This is an issue faced by lots of companies in the real world that are supporting legacy code that may need to be built with older compilers, etc. Believe it or not there is a compromise! You can keep your optimized version while simultaneously making your code easier to understand and more portable. Have the compiler do the pointer arithmetic for you:
#include <fstream>
#include <iostream>

void main()
{
char string[25];
char header[5][10] = {
{"Name"},
{"Address"},
{"City"},
{"State"},
{"ZIP Code"}
};

int offset = 0;
file_ptr.open("ADDRFILE.DAT", ios::in);
if (file_ptr)
{
do
{
file_ptr.get(string, 25);
file_ptr.ignore(80, '\n');
if (!file_ptr.eof())
cout << header[offset % 5] << ":" << string << "\n";
++ offset;
}
...etc
So I guess the main point here is, don't over-complicate things, and remember that portability is an issue. Other than that, I would like to say good job on taking the incentive to go above and beyond the accepted normalities; keep the drive going!

I'm going to agree with Halifax, In a programming job, readability, maintainability, and portability are likely to be more important than speed most of the time because other people will have to read and edit your code and very few apps (mainly those with sound or graphics) will max out a processor, so the trade off probably isn't worth it. Imagine a project of 6k lines of this and the migraine it would cause, even if its faster. Also, comments are always a good idea whenever clever code is employed.

What kind of class is it?

Haha, hey DJ_Omnimaga. Yeah, it's been a while since I've gotten time to get back into calculators.

Imagine a project of 6k lines of this and the migraine it would cause, even if its faster. Also, comments are always a good idea whenever clever code is employed.

I can imagine; I just printed out the source to The Penguin 77's Tetris Marathon (64 pages) and Quigibo's Puyopuyo (49 pages) in an attempt to learn Asm.   It's pretty cryptic, but it's fast.   (*ZTrumpet is just weird... )
Despite the comment I just made, I completely agree with you, necro.   I should have probably commented and coded better earlier.

Code: [Select]

#include <fstream>
#include <iostream>

void main()
{
char string[25];
char header[5][10] = {
{"Name"},
{"Address"},
{"City"},
{"State"},
{"ZIP Code"}
};

int offset = 0;
file_ptr.open("ADDRFILE.DAT", ios::in);
if (file_ptr)
{
do
{
file_ptr.get(string, 25);
file_ptr.ignore(80, '\n');
if (!file_ptr.eof())
cout << header[offset % 5] << ":" << string << "\n";
++ offset;
}
...etc
So I guess the main point here is, don't over-complicate things, and remember that portability is an issue. Other than that, I would like to say good job on taking the incentive to go above and beyond the accepted normalities; keep the drive going!

Halifax, that's pretty great code!  Thanks for sharing it, and I think it better than my version.  Awesome job!

What kind of class is it?

It's an introduction to C++ that I'm taking.  As you can tell, we are learning about data files right now.