Thao tác không an toàn hoặc không được kiểm tra cho ArrayList

Question

Tôi được chỉ định tạo một chương trình có 100 số nguyên ngẫu nhiên trong khoảng 0-25 và lưu chúng trong một mảng. Sau đó, tôi phải gọi 2 phương pháp để tách các phần mềm và tỷ lệ cược (rất điển hình). Vì vậy, tôi đã cố gắng điều ArrayList (Tôi Jút học nó) và nó có vẻ tốt (Tôi đã làm theo hướng dẫn và những thứ trên mạng) cho đến khi tôi chạy vào này: Unit8.java sử dụng không được kiểm soát hoặc không an toàn hoạt động

Mã của tôi là này:

import java.util.*; 
    import java.awt.*; 

    public class Unit8 
    { 
public static void main (String [] args) 
{ 
    //create an array for original 100 integers 
    //create a 2D array for evens and odds 
    //split them up using 2 methods 

    int[] originalArray = new int[100]; 
    ArrayList even = new ArrayList(1); 
    ArrayList odd = new ArrayList(1); 

    for (int x = 0; x < originalArray.length; x++) 
    { 
     originalArray[x] = (int)(Math.random() * 25); 
    } 

    evensDivider(originalArray, even); 
    oddsDivider(originalArray, odd); 
} 

public static void evensDivider (int[] e, ArrayList even) 
{ 


    for (int y = 0; y < e.length; y++) 
    { 
     if (e[y]%2 == 0) 
      even.add(e[y]); 
    } 

    System.out.println("The evens are: " + even); 
} 

public static void oddsDivider (int[] o, ArrayList odd) 
{ 


    for (int z = 0; z < o.length; z++) 
    { 
     if (o[z]%2 == 1) 
      odd.add(o[z]); 
    } 
} 

}

với các lỗi xảy ra đặc biệt tại địa chỉ: even.add(e[y]); và odd.add(o[z]);

Hãy giúp tôi với điều này, tôi đã cố gắng hết sức để làm cho nó rõ ràng và dễ hiểu.

Answer 1

Điều này là do bạn đang sử dụng ArrayList với raw type. Và bạn đang thêm một loại cụ thể vào nó.

Loại thô ArrayList sẽ mong đợi yếu tố của loại Đối tượng. Nếu bạn thêm bất kỳ loại nào khác, thì Trình biên dịch sẽ không biết chính xác loại bạn đang lưu trữ. Vì vậy, nó cung cấp cho bạn unchecked or unsafe operations để cảnh báo bạn rằng bạn có thể đang làm điều gì đó sai.

Bạn nên tạo tốt hơn một ArrayList Generic: -

List<Integer> evenNumbers = new ArrayList<Integer>(); 

Ngoài ra, thay đổi nó trong chữ ký method của bạn: -

public static void evensDivider (int[] e, List<Integer> evenNumbers) 

PS: - Bạn nên luôn luôn có một tham chiếu của interface type nếu bạn có.Ý tôi là sử dụng List ở vị trí của ArrayList

Answer 2

Nó complaing về vấn đề này

ArrayList even = new ArrayList(1); 

bạn có phiên bản không-chung của ArrayList đó. Thay đổi nó thành

List<Integer> even = new ArrayList<Integer>(1); 

Đó là loại chung và an toàn.

Ngoài ra (dưới dạng ghi chú bên), giá trị 1 trong new ArrayList(1); chỉ định dung lượng ban đầu của danh sách. Tính năng đó là có cho các tình huống mà bạn biết bạn sẽ thêm một số lượng lớn các yếu tố vào danh sách, trong trường hợp đó bạn sẽ cung cấp một số lớn hơn, điều này sẽ giúp tránh được phí thay đổi kích thước. Việc chuyển một giá trị nhỏ (như 1) không có ý nghĩa.

Answer 3

Thay vì điều này:

ArrayList even = new ArrayList(1); 

thử điều này:

List<Integer> evens = new ArrayList<Integer>(50); 

Dưới đây là những lợi thế:

• List này chỉ có thể giữ số (Số nguyên, thực ra, còn được gọi là int), Tôi giả định là cảnh báo đến từ đâu.
• tên này có tên số nhiều, phản ánh tốt hơn tên đó, danh sách
• Nó tự khởi tạo với kích thước 50 (thay vì 1), tức là bạn nghĩ nó lớn tới mức nào.

Answer 4

Thay thế này ArrayList even = new ArrayList(1); với ArrayList<Integer> even = new ArrayList<Integer>(1);

Hoặc thêm @SuppressWarnings("rawtypes") trên đỉnh của thuộc tính/phương pháp/lớp.