thank you for your advice!! as I said in my ending post, this was my final project in my introduction to c language in my college and needed to be improved. Although my professor is so strict at that because he discussed 1/4 of the Introduction to C language and it was merely a beginner's project... I appreciate the advice
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
You really should just return the value:
int menu(void);
If
choice
is a pointer, just write:scanf("%d", choice);
You repeat the call to
menu()
in everycase
. You could more simply do:while ( (choice = menu()) <= 5 ) {
thank you for your advice!! as I said in my ending post, this was my final project in my introduction to c language in my college and needed to be improved. Although my professor is so strict at that because he discussed 1/4 of the Introduction to C language and it was merely a beginner's project... I appreciate the advice