Thread: need help with a program

Code:

 #include<stdio.h>
void function1();
void function2();
void main()
{
char variable1;
char repeat;
do
{
printf("Please press 'b' if you wish to do something \n or\n please press 'v' if you wish to do something else ");
scanf("%c",&variable1);
if (variable1=='b')
{
function1();
}
else
{
if (variable1=='v')
{
function2();
}
printf("Do you wish to repeat the program? (y/n)");
scanf("%c",&repeat);
}
while(repeat=='y');
}
　
　
　
　
　
void function1()
{
printf("function1 has been utilised");
}
　
　
　
　
void function2()
{
printf("function2 has been utilised");
}

my code is actually indented on visual studios but when i copy n paste it here it comes out like that. How does your look more neat and have numberse etc? Did you still use the square bracket code tag?

Do you have any advice concerning the errors with my code and why it isnt working properly?

so my code works perfectly in your compiler? Strange. Is that what you mean to say?

hopefully i will figure out how to post code up neater here. sorry if its messy for anyone to read

what non ascii code did i have?

What is your program supposed to do? Also, void main needs to be int main(void) and return 0; at the end

Originally Posted by camel-man

What is your program supposed to do? Also, void main needs to be int main(void) and return 0; at the end

the program is supposed to ask you whether you'd like to do A or B, and if you choose A its supposed to activate function A, and if you choose B its supposed to activate function B, and once its finished its supposed to ask you whether you'd like to have another go, and if you select yes it will ask you to choose A or B again and if you select no the program ends.

As far as I knew you can void main() but int void is better practice, but the program should still work with void main() am i right in thinking that? Im looking for the reason that the program wont work at all

If I were you I would change it to int main(void) now before you get into any bad habits(also it will save you from hearing it from everyone else on the forum ). Your code will not compile for a number of reasons. One being that you need to supply parameters in your functions at the bottom. If you are not passing anything then put void in the parens. Another thing check your curly braces, make sure none are left open.

ok will change it

Ive added this at the bottom of the main function:

Code:

 fflush(stdin);
getchar();

will that solve one of the issues you mentioned?

will check through the curly brackets. thanks

how do i change it to void main?

Dp I put int Where I initially declare the function, the part where the function is defined, or both?

take out fflush(stdin) and what you should do is put a leading space in your scanf function example scanf(" %c", &variable1); do that for any scanf function that accepts a character input. This allows scanf to ignore any leading whitespace, therefore the '\n' character will be ignored when looping around and getting your next char variable. Just change void main() to int main(void) and return 0; at the bottom of main just inside the ending curly brace.