2011-12-23 2 views
2

Im пытается поставить arraylists в arraylist. Добавляем данные в новые массивы, а затем печатаем их по порядку. Я только получаю ошибки.Arraylist в Arraylist, unchecked error. Java

Это правильный способ создать arraylist в другом arraylist, используя цикл for?

Я также хотел бы теперь, как я могу получить данные из массива лучше, чем эти длинные выражения.

Мои ошибки

jogging.java:101: warning: [unchecked] unchecked call to add(E) as a member of the raw type java.util.ArrayList 
       res.get(iter).add(new Resultat(name,time)); 

    jogging.java:152: warning: [unchecked] unchecked conversion found : java.util.ArrayList required: java.util.List<T> Collections.sort(res.get(iter2)); 

    jogging.java:152: warning: [unchecked] unchecked method invocation: <T>sort(java.util.List<T>) in java.util.Collections is applied to (java.util.ArrayList) 
       Collections.sort(res.get(iter2)); 

 import java.util.; 
    import java.lang.;

class Resultat implements Comparable<Resultat> { String namn; double tid; public Resultat(String n, double t) { namn = n; tid = t; } public String toString() { return namn + " "+ tid; } public int compareTo(Resultat r) { if (this.tid < r.tid){ return -1; } else if (this.tid > r.tid){ return 1; } else if (this.tid == r.tid && this.namn.compareTo(r.namn) <= 0) { return -1; } else if (this.tid == r.tid && this.namn.compareTo(r.namn) >= 0){ return 1; } else {return 0;} } } public class jogging { public static void main(String[] args){ int runners = scan.nextInt(); int competitions = scan.nextInt(); //create arraylist with arraylists within ArrayList <ArrayList> res = new ArrayList<ArrayList>(); for(int i = 0; i <= competitions; ++i){ res.add(new ArrayList<Resultat>()); } for (int i = 0; i < runners; i++){ String name = scan.next(); //runs the person made int antalruns = scan.nextInt(); for(int n = 0; n <antalruns; n++){ //number of the run int compnumber = scan.nextInt(); //time for the run double time = scan.nextDouble(); for(int iter = 0; iter < res.size(); ++iter){ res.get(iter).add(new Resultat(name,time)); } } } for(int iter2 = 0; iter2 < res.size(); ++iter2) { Collections.sort(res.get(iter2)); System.out.println(iter2); for(int it = 0; it < res.get(iter2).size(); ++it) { System.out.println(res.get(iter2).get(it)); } } } }

ответ

4

непроверенным предупреждения являются, потому что вы не объявили общие типы 2-го ArrayList. Попробуйте использовать

ArrayList <ArrayList<Resultat>> res = new ArrayList<ArrayList<Resultat>>(); 

Да, это немного утомительно. :-(

Кроме того, большинство думает, что это хорошая практика, чтобы использовать интерфейс (например, List, не ArrayList) на левой стороне, на всякий случай, если вы передумаете, как к реализации в будущем. Например

List <List<Resultat>> res = new ArrayList<ArrayList<Resultat>>(); 

ДОБАВЛЕНО

Кроме того, вы можете упростить метод CompareTo() Для сравнения ПИВ, смотрите на Double.compare() Что-то вроде:..

public int compareTo(Resultat r) { 

     int compare = Double.compare(tid, r.tod); 
     if (compare != 0) 
     return compare; 
     else 
     return namn.compareTo(r.namn); 
    } 
+0

А это сработало! Чувствую себя глупо, что мне не удалось это понять :) – user1112634

+0

@ user1112634 - ошибки компиляции, которые вы получаете из-за проблем с родовыми типами, трудно понять ... если у вас еще нет хорошей ментальной модели того, как общие типы работают на Java. –

1

Каждый раз, когда вы мето te a ArrayLIst вам нужно объявить его тип данных. Поэтому, когда вы создаете Список_массивы внутри ArrayList вам нужно написать:

ArrayList<ArrayList<Resultat>> res = new ArrayList<ArrayList<Resultat>>(); 

Также компилятор должен положить меньше усилий, если вы пишете:

import java.util.ArrayList; // NOT: import java.util.*; 

Имея начало имени класса ВЕРХНИЙрегистр является Java именования. Поэтому всегда рекомендуется писать название вашего класса как:

public class Jogging { 
    /.. 
} 
+1

Нет ничего _wrong_ с 'import java.util. *;'; компилятор просто должен выполнить небольшую дополнительную работу. Нет необходимости в 'import java.lang. *;'. –

+0

-1 для фиктивных советов по импорту. –

Смежные вопросы