Java- problem z nauką pętli for

Question

Witam !

Od niedawna podjąłem się nauki Javy, lecz przez dłuższy czas nie mogę zrozumieć działania pętli for w ćwiczeniu opisanym poniżej:

Metoda zwraca sumę kolejnych liczb naturalnych począwszy od liczby przekazanej przez parametr int firstNumber skończywszy na liczbie przekazanej przez parametr int lastNumber. Uzupełnij metodę aby zwracała poprawny wynik.

Mój kod:

public class Tools
{
    public static int sum ( int firstNumber, int lastNumber ) {
        int y=0;
        for ( int x = firstNumber ; x <= lastNumber ; x++ )
        {
        if (y > x)
        {return y= y +x;}
        }
        return y;
    }
}

Testy pokazują wartość y=0

Bardzo proszę o pomoc w zrozumieniu działania pętli w tym zadaniu

Comments

Tak tylko dodam że jeżeli chodzi o naukę pętli to wszystko jest OK, natomiast z punktu widzenia optymalizacji to rozwiązanie jest fatalne. Dużo szybszym i lepszym rozwiązaniem jest kod bez użycia pętli:

public static int sum ( int firstNumber, int lastNumber ) {
        int answer;
        answer = (firstNumber + lastNumber) * (lastNumber - firstNumber + 1) / 2;
        return answer;
}

 

Answer 1

I dobrze Ci pokazuje :-)

public static int sum ( int firstNumber, int lastNumber ) {
        int y=0;
        for ( int x = firstNumber ; x <= lastNumber ; x++)
        {
             if (y < x) {
                  y = y +x;
            }
        }
        return y;
}

Co chcesz uzyskać tą pętlą?

Comments

Tą pętlą chcę uzyskać sumy kolejnych liczb naturalnych począwszy od liczby przekazanej przez parametr int firstNumber do int lastNumber.

Przykładowo

firstNumber = 3;

lastNumber=  5

y= 3+ 4 + 5 = 12;

Nie wiem tylko w jaki sposób wyciągnąć taki wynik z pętli

---

To ten if w środku pętli nie jest Ci potrzebny.

---

Sama instrukcja  - return y= y+x; w ciele pętli ostatecznie doprowadza w testach:

FAILEDsum( 333, 969 ) { return 333; } == 414687

FAILEDsum( 103, 903 ) { return 103; } == 402903

FAILEDsum( 722, 1674 ) { return 722; } == 1141694

FAILEDsum( 889, 1334 ) { return 889; } == 495729

FAILEDsum( 3, 311 ) { return 3; } == 48513

FAILEDsum( 16, 590 ) { return 16; } == 174225

FAILEDsum( 594, 791 ) { return 594; } == 137115

FAILEDsum( 790, 1144 ) { return 790; } == 343285

FAILEDsum( 664, 1597 ) { return 664; } == 1055887

FAILEDsum( 768, 1295 ) { return 768; } == 544632

W wielkim skrócie funkcja zwraca tylko wartość firstNumber.

---

Ale dając return w ciele funkcji chcesz z niej wyskoczyć dlatego Ci krzyczy błędami.

Spróbuj tak:

public static int sum ( int firstNumber, int lastNumber ) {
        int y=0;
        for ( int x = firstNumber ; x <= lastNumber ; x++)
        {
                  y += x;
        }
        return y;
}

 

---

Zadziałało idealnie !

Nie pomyślałem nad wydaniem polecenia "+=" gdyż uważałem to za działanie dziwaczne.

Musze się jeszcze wiele nauczyć ale bardzo dziękuję za pomoc :D ;)

Zamykam post :)