Help me improve my code!

Iam new to c programming and i would like anyone suggestion in how to improve this code to do the fallowing program description. If you see any mistakes feel free to tell me.

Thanks

wise_ron

Code:

---

*/
  Description: Write a program that prints your name, age and a paragraph about you.
*/

#include<stdio.h>
main()
{
      char name[15]="caleb ortega";
      int age = 21;
      printf("My name is %s and iam %d years old\n",name,age);
      printf("A paragraph about me:\n");
      printf("I like to play Sports, Guitar and listen to music\n");       
      system("pause");
return 0;
 }

---

Code:

---

*/
  Description: Write a program that prints your name, age and a paragraph about you.
*/

---

Your comment is wrong; comments start with /* and end with */.

Code:

---

char name[15]="caleb ortega";

---

Well, rather than count the number of characters in the string and risk getting it wrong, if you initialize the string to a value, you can get the compiler to do it for you:

Code:

---

char name[] = "caleb ortega";

---

This

Code:

---

main()

---

should probably be

Code:

---

int main()

---

http://faq.cprogramming.com/cgi-bin/...&id=1043284376

And your indentation could use some working on; the return statement isn't indented, and the closing brace is.

As for system("pause") . . . http://faq.cprogramming.com/cgi-bin/...&id=1043284385 . . . you might want to use a simple getchar() for now.

Here's the program with my suggested modifications:

Code:

---

/*
  Description: Write a program that prints your name, age and a paragraph about you.
*/

#include<stdio.h>
int main()
{
      char name[]="caleb ortega";
      int age = 21;
      printf("My name is %s and iam %d years old\n",name,age);
      printf("A paragraph about me:\n");
      printf("I like to play Sports, Guitar and listen to music\n");       
      getchar();
      return 0;
}

---

Thanks DWKS, I have change what you told me and thanks for your recommendations.

I have improve the code and added some more interesting stuff. Thank you all hope to receive more comments.

Code:

---

/* Description: Write a program that prints your name, age and a
              paragraph about you.
              Yout age and name should be decleared as variables. Use the age
              variable to calculate your age in months and days.
*/

#include<stdio.h>
int main()
{
      char name[]="Caleb 0rtega";
      int age = 49;
      int age_in_months = age*12;
      int age_in_days = age*365;
      printf("My name is %s and I am %d years old.\n",name,age);
      printf("My age in Months is: %d months.\n",age_in_months);
      printf("My age in Days is: %d days\n\n",age_in_days);
      printf("A paragraph about me:\n");
      printf("I like to play Sports, Guitar and listen to music\n");       
      getchar();
return 0;
}

---

Code:

---

int age_in_days = age*365;

---

What about leap years?

int age_in_days = age*365.25

Is how I have always calc'd it... since that is how long a year truly is (if I remember my science classes right).

EDIT: I know that any extra stuff after the decimal gets truncated, but most times people care more about the whole days rather than the parts of days...

Quote:

---

Originally Posted by Wraithan

int age_in_days = age*365.25

---

Wouldn't you have to do something like:
int age_in_days = int((float) age * 365.25f)
to make that work right?

Quote:

---

Originally Posted by Kennedy

Wouldn't you have to do something like:
int age_in_days = int((float) age * 365.25f)
to make that work right?

---

No. the compiler will automagically promote age into a float or double for the multiplication and then when it's assigned to the age_in_days variable the fraction will just be truncated.

It's actually closer to 365.24 days per year though ;)

According to google: http://www.google.com/search?hl=en&q=1+year+in+days

Quote:

---

Originally Posted by itsme86

No. the compiler will automagically turn age into a float or double for the multiplication and then when it's assigned to the age_in_days variable the fraction will just be truncated.

---

Strange. . . I had to do that in Dev-C++ since I wasn't getting what I was expecting. . . I'll have to go back and check that out.

Is that C99 or just the way most compilers do it?

Bah, it has been years since my last science class. For the intents and purposes of this class assignment I should think that every 4 years should have an extra day, since that is how the actual calender works. Due to that, 4*365.24 would not give you the extra day for the leap year. besides, .25 makes for cleaner numbers and we have only put on an addition .04(approx) days in the last 2006 years.

Anyway... this is getting OT

Most compilers would give you a warning to the effect that you're assigning a floating point number to an integral variable. Usually you need a cast of some kind.

If you want to round a number, add .5 to it:

Code:

---

int roundnumber(double d) {
    return (int)(d + .5);
}

---

[edit]

Quote:

---

the compiler will automagically promote age into a float or double

---

There's nothing magical about it. :D [/edit]

Quote:

---

Originally Posted by Wraithan

since that is how the actual calender works.

---

Which calendar? The leap year rule for the calendar in widespread use today is a little more complicated than that.

From wikipedia:

Quote:

---

The Gregorian calendar, the current standard calendar in most of the world, adds a 29th day to February in all years evenly divisible by 4, except for centennial years (those ending in -00), which receive the extra day only if they are evenly divisible by 400. Thus 1600, 2000 and 2400 are leap years but 1700, 1800, 1900 and 2100 are not.

---

I learn a lot about the years and how to work with them, for this program.

Everyone thanks and have a nice day