aspose file tools*
The moose likes Beginning Java and the fly likes Please check my code Big Moose Saloon
  Search | Java FAQ | Recent Topics | Flagged Topics | Hot Topics | Zero Replies
Register / Login
JavaRanch » Java Forums » Java » Beginning Java
Bookmark "Please check my code" Watch "Please check my code" New topic
Author

Please check my code

Emanuel Mensa
Ranch Hand

Joined: Feb 22, 2013
Posts: 31
i'm starting to complete java exercises i find online (starting from the basics).

the task is: Exercise SumAndAverage (Loop): Write a program called SumAndAverage to produce the sum of 1, 2, 3, ..., to an upperbound (e.g., 100). Also compute and display the average. The output shall look like:

but in my program, when i enter an upperlimit of say 10 which totals to 55. i expect the average to be 5.5. but my program returns 5.0.

can you please tell me how to fix this?

Jesper de Jong
Java Cowboy
Saloon Keeper

Joined: Aug 16, 2005
Posts: 14144
    
  18

Emanuel Mensa wrote:but in my program, when i enter an upperlimit of say 10 which totals to 55. i expect the average to be 5.5. but my program returns 5.0.

That's because number is an int, which can only hold integer values (not fractional values such as 5.5).

Lines 34 and 35 are wrong. Don't store the result in number. Remove those lines, just return number divided by upperBound there. Since upperBound is a double, the result will be a double and the calculation will be correct.

You're also doing things wrong with static. When a variable is static, it means that only one copy of it exists which is shared by all instances of a class (see Understanding Instance and Class Members).

You're first setting upperBound in line 46, then you're passing it to the constructor of SumAndAverage in line 47, where you're setting it again (line 17), as if it is an instance variable instead of a class variable. Then you're again passing it to the calc method (line 49). The calc method doesn't need to get upperBound as a parameter - it can access the member variable (line 5) directly.

Java Beginners FAQ - JavaRanch SCJP FAQ - The Java Tutorial - Java SE 7 API documentation
Scala Notes - My blog about Scala
 
I agree. Here's the link: http://aspose.com/file-tools
 
subject: Please check my code