beep when even?

I need help with this
Write a program that will get a number from user and print ""Hello world" that time. But remember if the entered number is even then the program should add 1 to that value and print the value that times and your program should only print the output odd times.
E.g. If user entered 2 or any even value then add 1 to that value and print the string that times and also show a beep at last output if and only if the entered value is even.


my code

Code:

---

#include<iostream.h>
#include<conio.h>
void main()
{
int num;
clrscr();
cout<<"\n How many times do you want to print Hello World ?  ";
cin>>num;
for(int i=1;i<=num;i++)
        {
        if(num%2==0)
        num+=1;    //this will increment if num is even
        cout<<"\n Hello World ";
        }
        cout<<"\a";
getch();
}

---

I cant do the beep part if the num is even. Though it does gives u beep at the end wether if its even or odd...

That's exactly what you tell it to do. The compiler cannot read your mind.

It is also strange why you should check and increment num within the loop.

You severely need to learn indentation: https://apps.sourceforge.net/mediawi...le=Indentation
Then you need to get rid of void main: http://apps.sourceforge.net/mediawik...itle=Void_main
Then you need to build a flowchart as what you want to do and then translate it to code.
You could even translate your current code to a flowchart as an exercise to see what is wrong.

i do use int main too but i am used to void main though il switch to int main from now on ;)

Quote:

---

You severely need to learn indentation:

---

Ahmed,

But, it's not the indentation that's causing your bug. It's the placement of the curly braces.

You have a pair of braces following your if-statement. Everything inside the braces depends on the if-condition. The "beep" code is not inside the braces, so it gets executed every time through the loop, regardless of the if-condition.

but when i but it in the loop body , the beep sounds as many times u want to repeat hello world...

And that is why you need to make a flowchart. This is a logic error, one we should not have to help you out with if you wish to be a programmer.

Code:

---

<iostream.h> // old as the hills - use <iostream>

<conio.h> // do not even need it - and its also non-standard

getch() // replace it with std::cin.get();

---