Co jest nie tak?

Pytania dotyczące zadania

Granie w Javarellę

Składnia Java, Poziom 7, Lekcja 9

Dyskutowane

Odpowiedź jest poprawna lecz nie wiem co jest nie tak I próbowałem:

for(int k=0; list1.size()>k; k++){
            if(list1.get(k)%2==0){
                przez2.add(list1.get(k));
            }
            if(list1.get(k)%3==0){
                przez3.add(list1.get(k));
            }
            if(list1.get(k)%3!=0 && list1.get(k)%2!=0){
                pozostale.add(list1.get(k));
            }
        }

oraz robiłem:

for(int k=0; list1.size()>k; k++){
            if(list1.get(k)%2==0){
                przez2.add(list1.get(k));
            }else if(list1.get(k)%3==0){
                przez3.add(list1.get(k));
            }else{
                pozostale.add(list1.get(k));
            }
        }

Oba sposoby nie zaliczają zadania a odpowiedź wydaje mi się poprawna

1. Wprowadź 20 liczb z klawiatury, zapisz je w liście, a następnie sortuj do trzech różnych list:
Liczby podzielne przez 3 (x%3==0), liczby podzielne 2 (x%2==0) oraz pozostałe.
Liczby podzielne zarówno przez 3, jak i przez 2 (na przykład 6) umieść w obu listach.
Kolejność deklarowania list jest bardzo ważna.
2. Metoda printLista powinna wyświetlać każdą pozycję z listy w nowej linii.
3. Używając metody printLista, wyświetl te trzy instancje listy. Najpierw listę liczb x%3, następnie listę liczb x%2, a na końcu listę ostatnią.

Wymagania:

Zadeklaruj i natychmiast zainicjalizuj 4 zmienne ArrayList<Integer>. Pierwsza lista będzie tą główną. Pozostałe listy będą listami dodatkowymi.
Wczytaj 20 liczb z klawiatury i dodaj je do głównej listy.
Do pierwszej dodatkowej listy dodaj wszystkie liczby z listy głównej, które są podzielne przez 3.
Do drugiej dodatkowej listy dodaj wszystkie liczby z listy głównej, które są podzielne przez 2.
Do trzeciej dodatkowej listy dodaj pozostałe liczby z listy głównej.
Metoda printLista powinna wyświetlać każdy element z przekazywanej listy w nowej linii.
Program powinien wyświetlać trzy dodatkowe listy przy użyciu metody printLista.

package pl.codegym.task.task07.task0713; import java.io.BufferedReader; import java.io.InputStreamReader; import java.util.ArrayList; import java.util.List; /* Granie w Javarellę */ public class Solution { public static void main(String[] args) throws Exception { ArrayList<Integer> list1 = new ArrayList<Integer>(); ArrayList<Integer> przez2 = new ArrayList<Integer>(); ArrayList<Integer> przez3 = new ArrayList<Integer>(); ArrayList<Integer> pozostale = new ArrayList<Integer>(); BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); for(int i=0; i<20; i++){ String a2 = reader.readLine(); int a = Integer.parseInt(a2); list1.add(a); } for(int k=0; list1.size()>k; k++){ if(list1.get(k)%2==0){ przez2.add(list1.get(k)); } if(list1.get(k)%3==0){ przez3.add(list1.get(k)); } if(list1.get(k)%3!=0 && list1.get(k)%2!=0){ pozostale.add(list1.get(k)); } } printLista(przez2); printLista(przez3); printLista(pozostale); } public static void printLista(List<Integer> lista) { for(int k=0; lista.size()>k; k++){ System.out.println(lista.get(k)); } } }

Komentarze (5)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Paweł Kwieciński

Poziom 10 , Mysliborz

17 lutego 2022, 17:29

Miałem ten sam błąd, problem jest w tej lini

ArrayList<Integer> przez2 = new ArrayList<Integer>();
ArrayList<Integer> przez3 = new ArrayList<Integer>();

a powinno być:

ArrayList<Integer> przez3 = new ArrayList<Integer>();
ArrayList<Integer> przez2 = new ArrayList<Integer>();

kicus

Poziom 20 , Warsaw, Poland

17 maja 2020, 19:36

Cześć. Popieram DeZiliusa. Chodzi tu tylko o odwrotne wyświetlanie list. Logika umieszczania liczb wygląda ok. for(int i = 0; i < main.size(); i++) { if(main.get(i) % 3 == 0) ByThree.add(main.get(i)); if(main.get(i) % 2 == 0) ByTwo.add(main.get(i)); if(main.get(i) % 3 != 0 && main.get(i) % 2 != 0) Rest.add(main.get(i)); } printLista(ByThree); printLista(ByTwo); printLista(Rest);

Michał Skiba

Poziom 16 , Warsaw, Poland

17 maja 2020, 18:39

for (Integer li: list) { if(li%3==0 && li%2==0){ przez3.add(li); przez2.add(li); }else if (li%3==0 ){ przez3.add(li); }else if (li%2==0){ przez2.add(li); }else { reszta.add(li); } }

Krystian C

Poziom 27 , San Francisco, United States

2 lutego 2020, 23:19

Liczby podzielne zarówno przez 3, jak i przez 2 (na przykład 6) umieść w obu listach. Jesteś pewien, że masz ten warunek w kodzie?

DeZilius

Poziom 27 , Warsaw, Poland

2 lutego 2020, 17:46

Cześć, podejrzewam że chodzi o tą uwagę -> Kolejność deklarowania list jest bardzo ważna. W zadaniu jest napisane, zadeklaruj najpierw listę % 3 a potem % 2 i potem resztę. Wyświetlasz dwie pierwsze listy zamienione %2 potem %3 a potem reszte, a powinno być %3 potem % 2 a na końcu reszta, która się zgadza u Ciebie :)