[marshaj]

Hello:
I am new to C++ and I need to write a simple program to print out consecutive numbers from m to n given m=4 n=10 .
i have tried various code and nothing seems to work

this a the code
Please help ,give me a hint what might be wrong....

include <stdio>
int main() {
	//prints numbers from m to n
	int m,n;
	int begin =4;
		int end=10;
	int m=4;
	int n=10;
	int count=0
		int sum=begin +1;
	
	
		while(count<=10){
			count +1=count;

			printf("%d\n",count);
	         
		
			}
	

}

Mod edit - Added tags

[macosxnerd101]

Hint- use a for loop.

for(int i = /*initial value*/; i <= /*end value*/; i++){
    //now print i
}

[BetaWar]

First off you have a lot more variables than are needed. All you should have to worry about is an m and n integer. Second, you redeclare m and n, that will cause problems. Third, you are trying to assign variables like so count+1=count;, that is invalid in C/C++, you have to do it like so: count=count+1; (though you could also do it like this: count += 1; or count++;

Now, given that you are learning I won't post code, I'll just give you some hints and let you try to come up with something that works from there.

I would create a loop starting at m and going until n that prints out whatever you use to keep track of your current number each time through (this may be m or another variable).

Hope that is of use.

[marshaj]

Thanks
I tried a little different code and compiled the program
Actually I am trying to learn to use the while with this code
It didn't give me exactly what I wanted but I'm still working on it
this what i got so far

#include <stdio.h>
int main() {
	//prints numbers from m to n
		
	int m=4;
	int n=10;
	int count=0;
		
		while(count<=10){
			count +=1;
			printf("m:%d\n"n);
	         
		
			}
}

This post has been edited by macosxnerd101: 27 September 2010 - 04:35 PM
Reason for edit:: Added code tags. Please use them in the future.

[eker676]

You're over complicating this. All you need is one variable.

Btw, That code is C not C++

In C++

#include <iostream>

int main()
{
  int count = 4;
  while(count <= 10)
    std::cout << count << std::endl;

  return 0;
}

[ishkabible]

no need for count,macosxnerd101 gave you a big hint. ask yourself what is the "initial value" and what is the "end value" then look at his code, not sure if should have even given that to you. if you required to use a while loop then initialize an int before the loop and increment that int at the end of each loop.

[marshaj]

eker676, on 27 September 2010 - 03:41 PM, said:

You're over complicating this. All you need is one variable.

Btw, That code is C not C++

In C++

#include <iostream>

int main()
{
  int count = 4;
  while(count <= 10)
    std::cout << count << std::endl;

  return 0;
}

Thnaks much
I think your while loop is missing the open and closed curly brackets???

[eker676]

Quote

I think your while loop is missing the open and closed curly brackets???

Nope it isn't. Well, yes it is but the code will run correctly. When all you have is one line you can skip the brackets. It can be a cause of some confusion but when the code is indented correctly it's easy to see what's happening.

This post has been edited by eker676: 27 September 2010 - 05:22 PM

[pkjm17]

#include <iostream>
using namespace std;

int main()
{
  int n = 10;

  for(int m = 4; m <= n; m++)
    cout<<m <<endl;

    cout<<endl;
    //system("pause");

   return 0;
}

[Alex6788]

pkjm17, on 27 September 2010 - 09:58 PM, said:

#include <iostream>
using namespace std;

int main()
{
  int n = 10;

  for(int m = 4; m <= n; m++)
    cout<<m <<endl;

    cout<<endl;
    //system("pause");

   return 0;
}

You shoudn't use

system("pause");

to pause the console instead use

 cin.get();

and sometimes that dosn't work if there is already input in the buffer then use

cin.ignore(); 
cin.get();

Hope i was of help.

This post has been edited by Alex6788: 28 September 2010 - 08:31 AM

[marshaj]

Alex688:
Thanks for your post but you have totally confused me on the system pause.
I just need to print numbers from m to n with those variables.
I need to keep it as simple as possible by using the while loop and the printf functions.

Mj

[Alex6788]

marshaj, on 28 September 2010 - 09:26 AM, said:

Alex688:
Thanks for your post but you have totally confused me on the system pause.
I just need to print numbers from m to n with those variables.
I need to keep it as simple as possible by using the while loop and the printf functions.

Mj

This isn't about the program and cin.get(); is simple, i just saw system("pause"); in your code and whenever i see that i always tell them they should use cin.get(); because system("pause"); is bad for many reasons and shouldn't be used really and cin.get(); is so much better.

This post has been edited by Alex6788: 28 September 2010 - 08:31 AM

[anonymouscodder]

You shouldn't use system("pause"); because it's a call to another program (on your system) called "pause".

This affect your program performance and portability.

[masoug]

system("pause"); is okay, but like anonymouscoder said, it calls another program, which is not recommended. If its just a simple program like this, then it really wouldn't matter too much. On the contrary, don't use this on a larger, more complicated program. Plus, if you are going to use system("pause"); then put it at the end;

#include <iostream>
using namespace std;

int main()
{
  int n = 10;

  for(int m = 4; m <= n; m++)
  {
    cout<<m <<endl;
  }
  system("pause");
  return 0;
}

PS. Might need some verification: Is the pause command Windows only?

-Masoug

This post has been edited by masoug: 28 September 2010 - 12:58 PM

[Alex6788]

masoug, on 28 September 2010 - 01:55 PM, said:

system("pause"); is okay, but like anonymouscoder said, it calls another program, which is not recommended. If its just a simple program like this, then it really wouldn't matter too much. On the contrary, don't use this on a larger, more complicated program. Plus, if you are going to use system("pause"); then put it at the end;

#include <iostream>
using namespace std;

int main()
{
  int n = 10;

  for(int m = 4; m <= n; m++)
  {
    cout<<m <<endl;
  }
  system("pause");
  return 0;
}

PS. Might need some verification: Is the pause command Windows only?

-Masoug

Yeah I'm pretty sure it's a windows command, One bad thing is that it prints "Press any key to continue" on the screen and you might not want that.

This post has been edited by Alex6788: 28 September 2010 - 01:02 PM