Obliczanie silni Java

Question

Witam, mam napisać program, który obliczy silnie, jednak cos nie działa, może ktoś mi wytłumaczyć, dlaczego tak się dzieje

import java.util.*;
import java.lang.Math;

public class JavaApplication {
    private static Scanner input = new Scanner(System.in);

    public static void main(String[] args) {
        int liczba, wynik, x;
        
        System.out.println("Podaj silnie: ");
        liczba = input.nextInt();
        if (liczba <= 1) {
            System.out.println("Wynik: " + 1);
        } else {
            for (x = 1; x <= liczba; x++) {
                wynik = wynik * x;
            }
            System.out.println("Silnia: " + wynik);
        }
    }
}

 

Answer 1

Wynik nie jest zainicjowany. Zainicjuj go 1. If jest niepotrzebny.

Comments

Jak mam to zrobić?

---

wynik=1

---

Nadal nie rozumiem...

Napisałem coś takiego

import java.util.*;
import java.lang.Math;

public class JavaApplication {
    private static Scanner input = new Scanner(System.in);

    public static void main(String[] args) {
        int liczba, wynik, x;
        
        System.out.println("Podaj silnie: ");
        liczba = input.nextInt();
        if (liczba <= 1) {
            System.out.println("Wynik: " + 1);
        } else {
            for (x = 1; x <= liczba; x++) {
                wynik = liczba * (liczba - 1);
            }
            System.out.println("Silnia: " + wynik);
        }
    }
}

Jednak mam problem z petlą

---

Bo to: 

 wynik = liczba * (liczba - 1);

jest kompletnie bezsensu.

Po pierwsze: nie deklaruj zmiennych "na zapas". Deklaruj je w chwili, kiedy potrzebujesz i jeśli możesz to od razu je inicjuj. GC dostaje w pewnym stopniu przez to po dupie (przenoszenie obiektów między heapami). Pogubisz się potem też w zmiennych. 

Zapis:

int liczba, wynik, x;

też nie powinien mieć miejsca. Lepiej zapisywać jedna pod drugą. W programie wystarczy zmienić w pierwszej wersji na:

int liczba;
int wynik = 1; 
int x;

żeby zadziałało. 

Zmienne sterujące pętlą, jeżeli nie potrzebujesz ich nigdzie indziej tylko w pętli/do sterowania pętli, deklaruj w nagłówku pętli.