I could use a favour..

**Iconoklast** · 07-20-2003

Hello,
I recently started learning about C programming, and so far I have written what may very well be, a very basic conversion program. It converts celsius to fahrenheit and vice versa, also miles to kilometres and vice versa.

I basically only know about the basics, conditional code, and looping, so I'm sure there is something I could learn in the future to minimalise my code..
To the favour, however..

I was wondering if someone with rather extened or decent knowledge of C to look over my code and tell me if its decent considering its just a bunch of if else, while statements.. Also, I was wondering if you could give me a way to remove the goto, and use something that'll produce the same effect, as I learned GOTO's are very messy and are not recommended..

i'd have posted the code, but its 160 some lines, so if you're willing to help, you can message me on AIM SN: Into Iniquity
or email me at [email protected]

Thank you!

**ygfperson** · 07-20-2003

Code:

 I recently started learning about C programming, and so far I have written what may very well be, a very basic conversion program. It converts celsius to fahrenheit and vice versa, also miles to kilometres and vice versa.

How can this be 160 lines?

**Iconoklast** · 07-20-2003

I just realised there is an attach file thing at the bottom here, hehe, go blind me! so I have attached it in this post

**Iconoklast** · 07-20-2003

Originally posted by ygfperson
[code]
How can this be 160 lines?

Lots of repetitions more than likely..

**confuted** · 07-20-2003

Link soup:
http://faq.cprogramming.com/cgi-bin/...&id=1043284351 14 occurances of that...

http://cboard.cprogramming.com/showt...100#post267100 Don't use goto

http://www.cprogramming.com/tutorial/lesson5.html Read up on switch case

http://www.cprogramming.com/tutorial/lesson4.html Functions!

That should keep you busy for a while. When you've read those and changed your program to be better by using them, post the new code. Or, if you get stuck, post a smart question.

That code doesn't need to be anywhere near as long as it is.

**Zach L.** · 07-20-2003

Also,

Code:

if (repeat == 'n')
    repeat = 'n';
else if (repeat == 'y')
{
    repeat = 'y';
} /* end else if */
else while (repeat != 'y' && repeat != 'n')
{
    printf("\nPerform another conversion (y/n)? \n");
    fflush(stdin);
    repeat = getchar();
} /* end else while statement */

can be simplified to:

Code:

while (repeat != 'y' && repeat != 'n')
{
    printf("\nPerform another conversion (y/n)? \n");
    repeat = getchar();
} /* end while statement */

**Iconoklast** · 07-20-2003

yeah, its far too long, I noticed.. longer than it could be, also because I took into account user input errors on all possibilities.. and I don't know at the moment if there is an easier way to go about it..
The main purpose of this post, is just to see if what is written so far, is generally justifiable for the very basics of which I know.. for any of you who have compiled the program and run it, I was wondering if there was a basic way (long or not) to go about choosing their select AFTER they have made an input error and the printf "Please enter a selection between 1-5: " comes up without using a goto..

**confuted** · 07-20-2003

The only thing with Zach L.'s example is that you have to initialize repeat to something that is neither n nor y. For example... o. Throw in a toupper in the while loop too, making the code this:

Code:

repeat='o';
while (repeat != 'Y' && repeat != 'N')
{
    printf("\nPerform another conversion (y/n)? \n");
    repeat = toupper(getchar());
} /* end while statement */

edit: You posted while I did. To answer your question, use a while loop

**Iconoklast** · 07-20-2003

Originally posted by Zach L.
Also,

Code:

if (repeat == 'n') repeat = 'n'; else if (repeat == 'y') { repeat = 'y'; } /* end else if */ else while (repeat != 'y' && repeat != 'n') { printf("\nPerform another conversion (y/n)? \n"); fflush(stdin); repeat = getchar(); } /* end else while statement */

can be simplified to:

Code:

while (repeat != 'y' && repeat != 'n') { printf("\nPerform another conversion (y/n)? \n"); repeat = getchar(); } /* end while statement */

that helped a lot, I never even looked at it that way... I tend to make things a little more complicated than they need to be.. thanks for that bit

**Zach L.** · 07-20-2003

But blackrat, repeat exists and has already been initialized.

**Zach L.** · 07-20-2003

Originally posted by Iconoklast
that helped a lot, I never even looked at it that way... I tend to make things a little more complicated than they need to be.. thanks for that bit

No problem - it was technically correct, just a little longer than it had to be.

And what blackrat was saying about toupper (or tolower for that matter) is a good point too. That'll allow users to enter y/Y or n/N without additional lines of code.

**Iconoklast** · 07-20-2003

Originally posted by Zach L.
No problem - it was technically correct, just a little longer than it had to be.

And what blackrat was saying about toupper (or tolower for that matter) is a good point too. That'll allow users to enter y/Y or n/N without additional lines of code.

Ah, excellent, was about to ask what it did, that'll save me a lot of time too

**confuted** · 07-20-2003

I had already closed the window with his code in it, and I didn't look at it very closely. It's always safe to make sure it's initialized

**Iconoklast** · 07-20-2003

to blackrat, there is a while loop there, are you suggesting I use another?

**confuted** · 07-20-2003

Yeah, a while loop starting at your label and ending at your goto.

If you check into putting the different conversions in functions, you could trim main() down to just a few lines - a while loop, within which there is the while loop which accepts input, and then a switch case that calls the conversion functions. Doing that will trim down your code a TON and make it a lot more readable (and it gives you good practice with functions, even though they aren't strictly needed here).

Thread: I could use a favour..

Thread Tools

Search Thread

Display

I could use a favour..

Similar Threads

Bug favour needed re: C++ Program