Beginner excersice #1

[C_beginner]

Am I did the following program corectly according to the question?

Question:

/* 1:You have just been employed by MacroMuscle, Inc.
(Software for Hard Bodies). The company is entering the European
market and wants a program that converts inches to
centimeters (1 inch = 2.54 cm). The company wants the
program set up so that it prompts the user to enter an inch
value. Your assignment is to define the program objectives and
to design the program (steps 1 and 2 of the programming process).
*/

solution:

#include<stdio. h>

#define centmeter 2.54
int main(void)
{
int inches_to_input ;
float inches_to_outpu t;
printf("please enter a inches\n");
scanf("%d",&inc hes_to_input);
inches_to_outpu t = inches_to_input * centmeter;
printf("inches converted is %f\n",inches_to _output);
return 0;
}

[Kies Lee]

I think your program is correct except the int type of inches_to_input .
Why don't you set the type of inches_to_input to float?

[Flash Gordon]

Kies Lee wrote:

I think your program is correct except the int type of inches_to_input .
Why don't you set the type of inches_to_input to float?

Please provide context when replying, there is no guarantee that others
have (or ever will) see the post you are replying to. See
http://cfaj.freeshell.org/google/ for details on how to provide proper
context and other useful information.
You may also find this URL useful http://clc-wiki.net/wiki/Intro_to_clc
--
Flash Gordon
Living in interesting times.
Although my email address says spam, it is real and I read it.

[Flash Gordon]

C_beginner wrote:

Am I did the following program corectly according to the question?
It looks to me like it should basically work, although there are some
issues and style matters you should consider.
Question:

/* 1:You have just been employed by MacroMuscle, Inc.
(Software for Hard Bodies). The company is entering the European
market and wants a program that converts inches to
centimeters (1 inch = 2.54 cm). The company wants the
program set up so that it prompts the user to enter an inch
value. Your assignment is to define the program objectives and
to design the program (steps 1 and 2 of the programming process).
*/

solution:

#include<stdio. h>
This would be easier for a human to read with an extra space.

#include <stdio.h>
#define centmeter 2.54
int main(void)
{
int inches_to_input ;
Why use an int? It is common for people to deal with fractional inches.
Obviously if you change it you will have to change the scanf format
specifier.
float inches_to_outpu t;
Why use a float rather than a double? Most of the time when you use
floats they get immediately promoted to double for the calculation
anyway. In this case, of course, you could also not use the variable at
all and print the result directly.
printf("please enter a inches\n");
scanf("%d",&inc hes_to_input);
You should check the return value of scanf to find out if it succeeded.
If it fails because the user entered "one" then inches_to_input would
not be initialised so the program could well output some random value.
inches_to_outpu t = inches_to_input * centmeter;
printf("inches converted is %f\n",inches_to _output);
return 0;
}

Generally a better first attempt than many we see here from beginners.

In the spirit of being helpful, I would also like to point out the
comp.lang.c FAQ to you which contains a lot of useful information and
help with questions you are likely to ask as you progress http://c-faq.com/
--
Flash Gordon
Living in interesting times.
Although my email address says spam, it is real and I read it.

[thomas.mertes]

"C_beginner " <[e-mail]> wrote:

#define centmeter 2.54
int main(void)
{
int inches_to_input ;
float inches_to_outpu t;
printf("please enter a inches\n");
scanf("%d",&inc hes_to_input);
inches_to_outpu t = inches_to_input * centmeter;
printf("inches converted is %f\n",inches_to _output);
return 0;

I would suggest using better names like:

#define CM_PER_INCH 2.54

int main(void)
{
float inches;
float centimeters;

printf("please enter inches ");
scanf("%f", &inches);
centimeters = inches * CM_PER_INCH;
printf("%1.2f inches converted are %f1.2 centimeters\n",
inches, centimeters);
return 0;
}

Greetings Thomas Mertes

Seed7 Homepage: http://seed7.sourceforge.net
Wikipedia: http://en.wikipedia.org/wiki/Seed7
Project page: http://sourceforge.net/projects/seed7

[nobody]

On Sat, 21 Jan 2006 13:51:23 +0530, C_beginner wrote:

Am I did the following program corectly according to the question? int inches_to_input ;
float inches_to_outpu t;
inches_to_outpu t = inches_to_input * centmeter;

inches_to_input is an integer while inches to output is float. While
the integer will be automatically become a float during the
multiplication with a float(centmeter ) I would write this code
differently.

Option1 : Why not float inches_to_input ?
Option2 : If you want inches_to_input to be an integer you could
do -> inches_to_outpu t = (float) inches_to_input * centmeter;
This is called casting

[Malcolm]

"C_beginner " <[e-mail]> wrote

Am I did the following program corectly according to the question?

Question:

/* 1:You have just been employed by MacroMuscle, Inc.
(Software for Hard Bodies). The company is entering the European
market and wants a program that converts inches to
centimeters (1 inch = 2.54 cm). The company wants the
program set up so that it prompts the user to enter an inch
value. Your assignment is to define the program objectives and
to design the program (steps 1 and 2 of the programming process).
*/

solution:

#include<stdio. h>

#define centmeter 2.54
int main(void)
{
int inches_to_input ;
float inches_to_outpu t;
Give these better names, like "inches" and "centimeter s".
printf("please enter a inches\n");
scanf("%d",&inc hes_to_input);
check the return from scanf(). If it returns 1, the user has entered an
integer correctly. If it doesn't, something has gone wrong, so print out an
error message.
inches_to_outpu t = inches_to_input * centmeter;
printf("inches converted is %f\n",inches_to _output);
return 0;
}

Program seems otherwise OK to me

[C_beginner]

Thanks for all the help. I learned little bit about promotion rank,
conversion operator.

[C_beginner]

[help sniped...]

You should check the return value of scanf to find out if it succeeded.
If it fails because the user entered "one" then inches_to_input would
not be initialised so the program could well output some random value.

Also told by Malcolm, is this the way that it has to be done?

if(!scanf("%d", &inches_to_inpu t))
{
operations and calcuations
}

[William J. Leary Jr.]

"C_beginner " <[e-mail]> wrote in message
news:43******** *************** @news.sunsite.d k...

Am I did the following program corectly according to the question?

Question:

/* 1:You have just been employed by MacroMuscle, Inc.
(Software for Hard Bodies). The company is entering the European
market and wants a program that converts inches to
centimeters (1 inch = 2.54 cm). The company wants the
program set up so that it prompts the user to enter an inch
value. Your assignment is to define the program objectives and
to design the program (steps 1 and 2 of the programming process).
*/

I'm probably being overly literal here, but I note that the "question" doesn't
actually tell you to write or implement anything.

It asks you to

1. Define the program objectives
2. Design the program

In the business world (at least the several I'm familiar with) neither of these
involves actually writing any code. That would usually be:

3. Implement the design

- Bill