Repetitiveness

What, in your opinion, is the easiest way of removing the repetitiveness of this FizzBuzz program?

I hate having to repeat if statements, it annoys me.

Also, general fizzbuzz thread.

#include <iostream>



using namespace std;



int main() {

	int count = 1;

	do {

		if(!(count%3)) {

			if(!(count%5)) {

				cout << "FizzBuzz" << endl;

			}

		}

		if(!(count%3)) {

			cout << "Fizz" << endl;

		} if (!(count%5)) {

			cout << "Buzz" << endl;

		} else {

			cout << count << endl;

		}

		count++;

	} while(count<100);

}

Also, if you think my code looks bad, then suck a dick because I did it in Vim because I'm too lazy to install a real text editor. So I don't have syntax highlighting, and compilation errors in g++ are cryptic at best.

Anyway, yep. Anyone know any other programming interview challenges that are fun?

You don't have to end the line with every cout.

if (!(count % 3))

    cout << "Fizz";

if (!(count % 5))

    cout << "Buzz";

if (count % 3 || count % 5)

    cout << count;

cout << endl;

Also you don't need syntax highlighting to properly space your code. I did this in the quick reply box.

GAMEchief wrote:

You don't have to end the line with every cout.

if (!(count % 3))

    cout << "Fizz";

if (!(count % 5))

    cout << "Buzz";

if (count % 3 || count % 5)

    cout << count;

cout << endl;

Also you don't need syntax highlighting to properly space your code. I did this in the quick reply box.

Oh hey, that's a pretty good idea, just appending the buzz onto it if it needs it, or printing the number if it's neither, that's pretty sweet. +10lgg

And yeah, good point about the formatting haha.

Using a do while loop is a tad redundant, particularly since you're just incrementing . You're better off using a for loop.