2

I have a Point class which implements Comparable. I want to do Point f = arr[first];

Now , I have read that using raw type Comparable etc is bad. So , If i make arr[] of type Comparable<Point>[] instead of Comparable[] What will i be doing wrong ?

This is stolen code , i loved it and i stole it.

private static void sort(Comparable[] a, Point compPoint, int lo, int hi) {
    if (hi <= lo)
        return;
    int lt = lo;
    int gt = hi;
    int i = lo;
    int count = 0;
    Comparator comp = compPoint.SLOPE_ORDER;
    Comparable v = a[lo];
    ArrayList<Point> line = new ArrayList<Point>();
    line.add(compPoint);
    while (i <= gt) {
        int cmp = comp.compare(a[i], v);
        if (cmp < 0)
            exch(a, lt++, i++);
        else if (cmp > 0)
            exch(a, i, gt--);
        else {
            count++;
            line.add((Point) a[i]);
            i++;
        }
    }
    if (count >= 3) {
        Collections.sort(line, new Comparator<Point>() {
            public int compare(Point v, Point w) {
                return v.compareTo(w);
            }
        });
        for (int j = 0; j < line.size(); j++) {
            if (j == line.size() - 1)
                StdOut.println(line.get(j).toString());
            else
                StdOut.print(line.get(j).toString()
                    + " -> ");
        }

        line.get(0).drawTo(line.get(line.size() - 1));
    }

    sort(a, compPoint, lo, lt - 1);
    sort(a, compPoint, gt + 1, hi);
}



private static void exch(Comparable[] a, int v, int w) {
    Comparable tmp = a[v];
    a[v] = a[w];
    a[w] = tmp;
}

I want to know if there is a better way than having raw type Comparable.

4

3 に答える 3

3

現在のように、次のように呼び出すことができます。

String[] array = {"a", "b"};
sort(array, point, 1, 2);

明らかにこれはばかげています。文字列をポイントと比較するのは無意味です。

メソッド内のコードは、同等の型に対して機能するようです。署名を次のように変更することを検討してください。

private static <T extends Comparable<T>> void sort(T[] a, T comp, int lo, int hi) {
于 2013-09-22T22:43:18.360 に答える
0

ここにあなたがすべきことの束があります：

  • class Point implements Comparableなるべきclass Point implements Comparable<Point>
  • ComparatorなるべきComparator<Point>
  • に置き換えます - 要素を s にキャストするComparable[]場合は、それらをポイントとして宣言することもできますPoint[]Point

  • 交換

    Collections.sort(line, new Comparator<Point>() {
    public int compare(Point v, Point w) {
        return v.compareTo(w);
    }
});

    冗長性が少ない

    Collections.sort(line);
于 2013-09-22T22:43:24.513 に答える
0

ジェネリックを使用するようにコードをリファクタリングすることに問題はありません。このメソッドはPoints にのみ使用されるようですので、Point[]代わりにComparable<Point>[]. 私が見たところ、クラスPointとそのcompare中の Comaprator のメソッドもリファクタリングする必要がありますSLOPE_ORDER。

許可されている場合は、すべてのコードをダウンロードしてリファクタリングしてください。すべてがうまくコンパイルされれば、これが Java ジェネリックのポイントです。実行時には、タイプeraser により、とにかくすべて生です。

于 2013-09-22T22:21:44.170 に答える