Help on a Basic Java Problem

[The Penguins]

I'm thoroughly stuck on this one problem in a Basic Java lesson. Can't figure out how to get the factorial calculator right as the answer is always = (Actual factorial answer * user input).

Please help 

[Khaos]

The prpblem is you multiplied 4 with 4 in the beginning

[Axle Change]

It should be int i = factorial-1

In factorials we don't need to multiply  by the number it is, in your code it would start at the same number, i.e. 5*5*4*3*2*1, it should start with the number-1 so we get 5*4*3*2*1

[A Delicious Cashew]

The easiest way to debug simpler programs is to write it out - you can see the steps being taken to reach the output

 

Let's say factorial is 4. That means product is initially 4. i goes from 4 down to 1. Product will equal product * 4 * 3 * 2 * 1. Since product was 4 originally, you're multiplying the first number twice effectively. You would want to set product = 1 initially or use the above solution. 

[The Penguins]

Thanks for help guys, got it right now