Przesuwanie się po tablicy - java

Question

Witam,

mam problem z zadaniem. Cały czaswyskakuje mi błąd wskazujący na brak danego indexu w tablicy. Jednak nie wime jak mogłabym o rozwiązac inaczej, wsyztsko wydaje się okej. Czy ktoś mógłby trochę naprowadzić?

TREŚĆ ZADANIA:

Zlicz ilość wszystkich potrójnych wystąpień w napisie.

Mój kod:

public class viva {
    public static void main(String[] args) {

        System.out.println(tripple("kkktokooolwieeek"));


    }

    static int tripple(String str) {

        char[] table = str.toCharArray();

        int sum = 0;
        int sumOf3 = 0;


        for (int i = 0; i < table.length; i++) {

            String value1 = str.valueOf(table[i]);
            String value2 = str.valueOf(table[i + 1]);

            if (value1.equals(value2)) {

                sum++;
            }

            sumOf3 = sum / 3;

    }
        return sumOf3;
    }
}

 

Answer 1

   String value2 = str.valueOf(table[i + 1])

W tym miejscu masz błąd - index po za zakresem.

Zobacz co się dzieje jak zmienna i przyjmie będzie miała maksymalną wartość czyli całą długość tablicy a Ty jeszcze dodasz do tej wartosci 1. Wtedy zawartość tablicy bedzie po za zakresem.

Comments

ale jak to obejść, jeżeli chce zastosowac wypisaną przezemnie funkcję, czyw ogóle muszę zmienić metodę? Bo inaczejnie będę mogła się odnieść do tego elementu...

---

 for (int i = 0; i < table.length - 1 ; i++) {

Np tak możesz obejść problem z ostatnim elementem.

Tak swoją drugą potrójne wystąpienia liter koło siebie ?

 

---

haha np np. dddlaczegggo :D

---

dziękuję! im dłuzej siedze przy kodzie tym moniej myśle widzę... dzięki jeszcze raz za pomoc :))

---

Nie ma za co :). Tak swoją drogą nie za bardzo rozumiem jak ma ta metoda działać :)

---

możesz coś innego zaproponować?

nie wiedziałam kompletnie jak to ugryźć, ale zadziała :D jeżeli 2 i 3 element  będzie taki jak 1 (tak ma być docelowo ale tablica mi się kończyła) :D to sum++;

Answer 2

Spodziewam się, że problem występuje w tej linijce - table[i + 1]

Jeżeli tablica ma 5 elementów to ostatnia wartość zmiennej i=4. Co za tym idzie, próbujesz odwołać się do 5 elementu, którego nie ma.

Answer 3

Wrzucam rozwiązanie dla tych, którzy by szukali pomocy, nagle mnie oświeciło, ze można też w 2 stronę...

 

 

public class viva {
    public static void main(String[] args) {

        String str = "Dlaaaczzzegggo";


        char[] table = str.toCharArray();

        int sum = 0;

        for (int i = 0; i < table.length; i++) {

            if (i > 1) {

                String value1 = str.valueOf(table[i]);

                String value2 = str.valueOf(table[i - 1]);

                String value3 = str.valueOf(table[i - 2]);


                if (value1.equals(value2) && value1.equals(value3)) {
                    sum++;
                }
            }
        }

        System.out.println(sum);
    }
}

 

Comments

Ale wiesz, że chary też da się porównywać i nie musisz ich konwertowac do stringow? A if w petli jest co najmniej dziwny, bo mozesz przeciez zaczac iterowac od 2.

        for (int i = 2; i < table.length; i++) {
            if(table[i-2] == table[i-1] && table[i-1] == table[i]){
                sum++;
            }
        }

 

---

dizęki za podpowiedź! wszystko mi się już miesza!