Bład return , zwracanie wartości

Question

Program wydaje się ok, chociaż ma błąd związany z return , sam nie potrafię zdiagnozować problemu .

Proszę o wskazówkę. (chodzi mi o uzupełnienie klas bo to sam robiłem)

przypisywałem wartość do return  i chyba to złe zrobiłem bo ja napisałem  

System.out.println(wynik);

wszytko działało  poprawnie.

 

public class Solution {
    public static List getListaDlaGet() {
        
        ArrayList<Integer> arr = new ArrayList<Integer>();
        arr.add(10); 
        arr.add(20);
        
        int wynik = arr.get(0);
        return wynik;
    }

    public static List getListaDlaSet() {
        
        ArrayList<Integer> arr = new ArrayList<Integer>();
        arr.add(10); 
        arr.add(20);
        
        int wynik = arr.set(0, 15);
        return wynik;
    }

    public static List getListaDlaDodajLubWstaw() {
        ArrayList<Integer> arr = new ArrayList<Integer>();
        arr.add(10); 
        arr.add(20);
        
        int wynik = arr.add(15);
        return wynik;
            
    }

    public static List getListaDlaUsun() {
        LinkedList<String> arr = new LinkedList<String>();
        arr.add(10); 
        arr.add(20);
        int wynik = arr.remove(20);
        
        return wynik;
    }

    public static void main(String[] args) {
        
    }
}

 

Answer 1

Przykładowo tutaj mamy tutaj błędnie napisane:

---

public static List getListaDlaUsun() {

int List wynik = arr.get(0) arr;

return wynik;

}

//wynik powinien być tak jak to zdefiniowano na górze czyli List 

---

...List a ty zwracasz int (wynik)
musisz zmienić typy, aby były takie same. np. na List

Prosty przykład z int

public static int getInt() {

int z = 1;

return z;

}

Rozwiązanie:

import java.util.ArrayList;

public class Solution {
    
    public static ArrayList<Integer> get1(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        return arr;
    }
    public static ArrayList<Integer> get2(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        arr.set(0, 15);
        return arr;
        
    }
    public static ArrayList<Integer> get3(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        arr.add(15);
        return arr;
    }
    public static ArrayList<Integer> get4(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        arr.remove(1);
        return arr;
    }

 
    public static void main(String[] args) {
        ArrayList<Integer> arrMain = new ArrayList<Integer>();
        arrMain.add(10); 
        arrMain.add(20);

        System.out.println(get1(arrMain)); //wyciąganie całej listy
        System.out.println(get2(arrMain));
        System.out.println(get3(arrMain));
        System.out.println(get4(arrMain));

        System.out.println(get1(arrMain).get(0)); //wyciąganie wartości z indexu listy nr1
        System.out.println(get1(arrMain).get(1)+"\n");

        System.out.println(get3(arrMain).get(0)); //wyciąganie wartości z indexu listy nr3
        System.out.println(get3(arrMain).get(1));
        System.out.println(get3(arrMain).get(2));
    }
}

 

Comments

robisz z int to zwracasz w return int.
tak samo z List

---

@Rollczi,

czyli chodzi o to ?

ArrayList<Integer> arr = new ArrayList<Integer>();
        arr.add(10); 
        arr.add(20);
        
        List wynik = arr.get(1);
        return wynik;

 

---

Taki zapis będzie nie poprawny i Przydało by się wiedzieć co chcesz ty zwrócić...

---

ogólnie nie wiem dokładnie jaki miałeś cel w np.
int wynik = arr.remove(20);

---

chce zwrócić 2 element listy arr.

---

Faktycznie to nie ma sensu ten przypisanie wartość do remove

Dobra mam, i rozumiem

po prostu bez sensu zrobiłem te przypisania 

Dzięki za wskazówki! dzięki pana pomocy siej udało !

poprawiony kod wyglada tak

LinkedList<Integer> arr = new LinkedList<Integer>();
        arr.add(10); 
        arr.add(20);
        arr.remove(20);
        
        return arr;

 

---

czekaj zrobię wersję która zwróci Listę

---

@danielo665, o świetnie właśnie o to chodzi :) ale arr.remove(20); tyczy się INDEXU

np.

arr.add(10);

arr.add(20); 

daje: arr[10, 20]
10 jest w [index 0]   i    20 jest w [index 1]

po:

arr.remove(1);

Zostanie usunięty [index 1] czyli wartość 20

---

@danielo665, zobacz jak to zrobiłem; Zapomniałem dodać, że typ powinien być  ArrayList<Integer> a nie List (nie powinno spowodować błędów)

import java.util.ArrayList;

public class Solution {
    
    public static ArrayList<Integer> get1(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        return arr;
    }
    public static ArrayList<Integer> get2(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        arr.set(0, 15);
        return arr;
        
    }
    public static ArrayList<Integer> get3(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        arr.add(15);
        return arr;
    }
    public static ArrayList<Integer> get4(ArrayList<Integer> arrM){
        ArrayList<Integer> arr = new ArrayList<Integer>(arrM);
        arr.remove(1);
        return arr;
    }

 
    public static void main(String[] args) {
        ArrayList<Integer> arrMain = new ArrayList<Integer>();
        arrMain.add(10); 
        arrMain.add(20);

        System.out.println(get1(arrMain)); //wyciąganie całej listy
        System.out.println(get2(arrMain));
        System.out.println(get3(arrMain));
        System.out.println(get4(arrMain));

        System.out.println(get1(arrMain).get(0)); //wyciąganie wartości z indexu listy nr1
        System.out.println(get1(arrMain).get(1)+"\n");

        System.out.println(get3(arrMain).get(0)); //wyciąganie wartości z indexu listy nr3
        System.out.println(get3(arrMain).get(1));
        System.out.println(get3(arrMain).get(2));
    }
}

PS liczę na zatwierdzenie (zaliczenie odpowiedzi)