twierdzenie pitagorasa

    public static void trojkatProstokatny (int a, int b, int c){
        a=a*a;
        b=b*b;
        c=c*c;
        if (a=b+c || b=a+c || c=a+b){
            System.out.println("Trojkat prostokatny");
        }
        else {
            System.out.println("----");
        }
        }

co jest zle w if'ie??

w porównaniu powinien być podwójny znak równości

Dzieki
/close

To jak dziękujesz, to dodam.

tego typu funkcje nie robi się void a bool i zwraca wynik oceny (tutaj: spełnienie kryterium Pitagorasa). Obecnie jest NIEUŻYTECZNA w większym kontekście, projekcie.
A drukuje się zwykle na zewnątrz funkcji "oceniającej" (bo niekiedy wcale się nie drukuje)

Ogólnie staraj się oddzielać dane(np. wynik sprawdzenia, czy dany trójkąt jest prostokątny) od tego, co robisz z tymi danymi, bo teraz możesz chcieć wyświetlić na ekranie (System.out.println), ale innym razem możesz chcieć np. zapisać do pliku, zachować w zmiennej itp. Więc dobrze, żeby funkcja po prostu liczyła, co ma liczyć i zwracała wynik (tak jak przedmówca wspomniał).