Unchecked-Warnungen für Arraylist

Warum bin ich immer diese 4 Warnungen aus -Xlint und was sollte ich tun Sie? Ich bin gerade erst in Java, also bin wahrscheinlich etwas fehlt offensichtlich.

import java.util.*;

class CompareGerbils implements Comparator {
    public int compare(Object o1, Object o2) {
       return ((Gerbil)o2).number() - ((Gerbil)o1).number();
    }
}

class Gerbil {
int gerbilNumber;

Gerbil(int gN) {
    gerbilNumber = gN;
    }

int number() {
    return gerbilNumber;
    }
}

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

    //I'd like to be able to add both ints and strings to list
    ArrayList list = new ArrayList();

    //unchecked call warning:
    list.add(1);  

    //unchecked call warning:
    list.add("b");  

    ArrayList<Gerbil> gerbillist = new ArrayList<Gerbil>();

    for(int i = 0; i < 5; i++) {
        gerbillist.add(new Gerbil(i));
    }

    //unchecked conversion warning
    //unchecked method invocation
    Collections.sort(gerbillist, new CompareGerbils());
  }
}

EDIT: Antworten bisher beantwortet haben, Arraylist-Deklaration. Wie über die Art der Warnungen an der Unterseite des Codes? Dank

  • Jemand anderes antwortete, während ich getippt, aber ich würde gerne eine Stil-note. Ihre Komparator impl benannt ist falsch und sollte mit einem Großbuchstaben beginnen. Nicht in dem Sinne zu sein, pedantisch, aber du wirst verrückt irgendwann die Aufrechterhaltung dieser code Fragen, warum das argument Typ für eine variable, Erklärung scheint zu sein, eine variable.
  • Bitte seien Sie pedantisch. Wie ich schon erwähnt habe, ich bin nur zu lernen. Ich werde die änderung vornehmen.
  • // I'd like to be able to add both ints and strings to list - Das ist in der Regel ein Designfehler. Warum sind Sie versuchen, erstellen Sie eine Sammlung mit grundlegend verschiedenen Arten der Dinge?
  • die Art Warnung, weil Sie definiert haben, der Komparator keine Daten geben, d.h. new Comparator() statt new Comparator<Gerbil>(). Ich habe aktualisiert, meine Antwort entsprechend den richtigen Gebrauch.
  • Peters: ich bin nur zu lernen, im moment. Alle Beispiele, die ich gesehen habe Stimmen, um eine einzige Art, so dass ich wollte sehen, ob die Sprache hatte diese Flexibilität.
InformationsquelleAutor foosion | 2011-02-28
  1. 7

    Sie bekommen das, weil Sie nicht definiert haben, einen Datentyp für die ArrayList list. Der einzige Weg, um beide Zeichenfolgen und Ganzzahlen in list ohne Warnungen ist definiert als ArrayList<Object> list - das ist das, was hier geschieht implizit (Linie list.add(1); implizite Konvertierung 1 zu new Integer(1) - das nennt man autoboxing). Beachten Sie auch, dass, wenn Sie wollen, um beide Zeichenfolgen und Ganzzahlen in lists, die Sortier-Methode nicht wirklich sinnvoll - wie werden Sie erwarten, dass alles sortiert man alphabetisch oder numerisch?

    Edit: Außerdem, es wird nicht als gute Praxis zu erklären, einen konkreten Typ (d.h. ArrayList<Object> list), es sei denn, Sie haben sehr gute Gründe, dies zu tun. Es wird empfohlen, dass Sie die Initialisierung mit Hilfe einer Schnittstelle, d.h. List<Object> list.

    So, würde der code so sein (Hinweis: das Teil Comparator<Gerbil> die behebt die Warnung in Collections.sort):

    //I'd like to be able to add both ints and strings to list
List<Object> list = new ArrayList<Object>();

list.add(new Integer(1));  

list.add(new String("b"));  

List<Gerbil> gerbillist = new ArrayList<Gerbil>();

for(int i = 0; i < 5; i++) {
    gerbillist.add(new Gerbil(i));
}

Collections.sort(gerbillist, new Comparator<Gerbil>() {
    public int compare(Gerbil o1, Gerbil o2) {
        int diff = o1.getNumber() - o2.getNumber();
        if (diff > 0)
           return 1;
        else if (diff <0)
           return -1;
        else
           return 0;
    }
});

    Bezug auf die Wüstenrennmaus Klasse, ich schlage vor, Sie verwenden das Formular getNumber als den Namen einer Methode anstatt number - es ist ein de-facto-standard für die Methoden-Namen zum abrufen des Werts einer member-variable (bzw. setNumber(int value) für Einstellung):

    class Gerbil {
int gerbilNumber;

Gerbil(int gN) {
    gerbilNumber = gN;
    }

int getNumber() {
    return gerbilNumber;
    }
}
    • Ich denke, du meinst, es ist keine gute Praxis, um declare einen konkreten Typ. Die Initialisierung wird immer nach unten kommen, um einen konkreten Typ.
    • Peters - du hast Recht, Feste, bekommen eine Tasse Kaffee.
    • zurück g2.Anzahl() - g1.Anzahl(); arbeiten, die für den Vergleich-Logik. Ich habe es in anderen Sprachen. Ist es nicht eine gute form in java?
    • Ich sehe Peter Lowrey hat, antwortete der Vergleich Frage.
    • abgesehen von Peter sehr guter Kommentar, auch ich schlage vor, Sie verwenden getNumber eher als number bei der Programmierung in Java.
    InformationsquelleAutor Lucas Zamboulis
  2. 3

    Die Warnung tritt auf, wenn Sie eine nicht-generischen Typs in einem Kontext, in dem eine generische erwartet. Das kompilieren ist zu sagen, Sie könnten Recht haben, aber ich kann nicht überprüfen Sie den Typ für Sie.

    Sie cna entweder;

    • Die Art der richtigen generischen.
    • Schalten Sie die Warnung mit @SuppressWarnings
    • Die Warnung ignorieren.

    EDIT: In diesem Beispiel, müssen Sie wählen Sie eine super-Klasse/interface der Elemente in der Liste.

    //I'd like to be able to add both ints and strings to list
List<Object> list = new ArrayList<Object>();

list.add(1);  
list.add("b");

    Anstelle von Objekt, das Sie auswählen könnte Seralizable oder Comparable jedoch eneither wahrscheinlich ist, nützlich zu sein. In der Tat ein List<Object> ist selten sinnvoll, außer in den übungen.

    Den richtigen Weg zur Umsetzung der Comparator ist nutzen-Vergleich. Mit - ist nur gültig, wenn Sie wissen das vielleicht nicht überlaufen. z.B. 2000000000 - -2000000000 < 0 in der Erwägung, dass Sie vielleicht erwarten 2000000000 - -2000000000 > 0

    class CompareGerbils implements Comparator<Gerbil> {
    public int compare(Gerbil a, Gerbil b) {
       return a.number() > b.number() ? +1 
            : a.number() < b.number() ? -1 : 0;
    }
}

    Für Testzwecke ich schlage vor, versucht zu Sortieren, eine Liste, die nicht sortiert sind. Ein einfacher Weg, dies zu tun ist, verwenden Sie die shuffle() Methode. Dies könnte noch sortiert werden, aber die große der Liste, desto weniger wahrscheinlich ist der Fall.

    Collections.shuffle(gerbillis);
    • Ich würde gerne verstehen, die Warnungen anstatt Sie zu ignorieren oder Sie zu unterdrücken. Wie mache ich das in der richtigen generischen?
    • Guter Punkt auf den Vergleich, obwohl ich ' ve wurde mit 'a - b' in anderen Sprachen, die seit Jahren ohne ein Problem (und dachte, dass es eine übliche Redewendung). Bei der Reflexion, eine > b ist klarer, das ist eine Tugend.
    InformationsquelleAutor Peter Lawrey
  3. 2
    public static void main(String[] args) {
    final List<Object> list = new ArrayList<Object>();
    list.add(1);
    list.add("b");

    final List<Gerbil> gerbillist = new ArrayList<Gerbil>();
    for (int i = 0; i < 5; i++) {
        gerbillist.add(new Gerbil(i));
    }

   Collections.sort(gerbillist, new CompareGerbils());
}
    InformationsquelleAutor Paul McKenzie
  4. 1

    1. müssen Sie wissen, Generika http://download.oracle.com/javase/1.5.0/docs/guide/language/generics.html

    2. die Sie hinzufügen möchten primitive und eine Zeichenfolge in arraylist.
      entweder u tun, etwas zu nehmen

    List<Object> list = new ArrayList<Object>();

    oder wenn Sie möchten, dass nur strings

     List<String> list = new ArrayList<String>();
    InformationsquelleAutor Dead Programmer
Schreibe einen Kommentar