asked on

Horizontal Bar-Graph C++

I'm a c++ beginner. Can someone PLEASE help me?

Write a program that allows a user to display a horizontal “bar-graph”.
Your program first asks the user to enter a series of positive integers (no more than 20).
They can indicate they are done by entering -1. You should read these numbers into an
array. Be sure you do not allow the user to enter negative numbers. Also make sure the
number entered is less than 70.
Once you have read all the numbers from the user, you are to print a horizontal bar graph,
with each line corresponding to one of the numbers entered. Use iomanip and functions

Ex.
1: 2
2: 4
3: 1

1: **
2: ****
3: *

TommySzalapski

Since this is quite clearly a homework assignment, take a shot at it and post your code when you get stuck. We at EE are not permitted to do your homework for you, but we will certainly help you get it done.

cbginner

ASKER

I apologize, I thought I had added the code, but apparently I didn't lol.

Here is what I got so far.

#include <iostream>
#include <iomanip>
using namespace std;


int main() {
	const int SIZE = 20;
	char n[SIZE];

	for ( int i = 0; i < SIZE; i++ ) {
		cout << "Please enter a number less than 70 (-1 to stop):  ";
		cin  >> n[i];
	}

	int inputFromUser;
		int n = inputFromUser;

	do	{	

		if ( inputFromUser != -1 ) {
			if ( inputFromUser < 0 || inputFromUser >= 70 ) {
				cout << "Invalid entry" << endl;
			}
			else {
				for ( int i = 0; i < inputFromUser; i++ ) {
					cout << "*" ;
				}
				cout << endl;
			}

		}

	} while ( inputFromUser != -1 ) ;

}

Open in new window

TommySzalapski

There are several things.
You declare n twice as two different things. You can't do this. Take out the second one and it will compile then you can see what it does (it doesn't work yet though)

Your for loop keeps going even if they put in a -1 to stop. This is bad.

In your while loop, you never set inputFromUser to anything. Why not use another for loop? See if you can get it working from this. Fix it up and let me know if you hit another snag. I'll be on for a while so I'll try to answer quickly.

cbginner

ASKER

I took the seconde n out and it still wouldn't compile.

As for the for loop. Do you mean the first one? Should I move it into the do-while loop so it will stop at -1 or create another do while loop around it?

I'm sorry , but i'm really bad at this so can you try to simplify it even more. Are you suggesting that I use a for loop to replace the inputFromUser function all together?

TommySzalapski

What compile error is it giving?
I'm suggesting you do not use a while loop at all just a second for loop. Like this (this is not actual code)

Post the new code too.

TommySzalapski

Signing off for a bit. I'll check in with you tomorrow.

cbginner

ASKER

I'm sorry, but I didn't see any example that you posted so I'm still not sure how to go about the while loop for a for loop. Hence, my code hasn't changed much.

As for the compile error, it wasn't even compiling and just said 0 success, but once I clicked rebuild, it was a successful compile. I did also notice that it does not stop when -1 is entered.

Here's the code:

#include <iostream>
#include <iomanip>
using namespace std;


int main() {
	const int SIZE = 20;
	char n[SIZE];

	for ( int i = 0; i < SIZE; i++ ) {
		cout << "Please enter a number less than 70 (-1 to stop):  ";
		cin  >> n[i];
	}

	int inputFromUser;

	do	{	

		if ( inputFromUser != -1 ) {
			if ( inputFromUser < 0 || inputFromUser >= 70 ) {
				cout << "Invalid entry" << endl;
			}
			else {
				for ( int i = 0; i < inputFromUser; i++ ) {
					cout << "*" ;
				}
				cout << endl;
			}

		}

	} while ( inputFromUser != -1 ) ;

}

Open in new window

cbginner

ASKER

* go about replacing the while loop for a for loop

ASKER CERTIFIED SOLUTION

cbginner

membership

This solution is only available to members.

To access this solution, you must be a member of Experts Exchange.

Start Free Trial

cbginner

ASKER

I never received any further responses from my expert.