2013-10-06 48 views
-1

我有一个Arraylist温度和天。我使用插入排序方法对arraylist进行排序。当我打印出来的时候,它不会以打印出来的方式打印出来:76,84,81,88,87。难道是因为我把Collections.sort(dailytemps)这一行注释掉了吗?它示数出来,所以我评论说出来排序错误的数组打印错误

这里是我的代码:

import java.io.*; 
import java.util.Scanner; 
import java.util.Arrays; 

public class DailyTemperature 
    { 

    //variables 
    private double temperature; 
    private String day; 

    //getTemp & setTemp methods 
    public double getTemp() 
    { 
     return temperature; 
    } 

    public void setTemp(double newTemp) 
    { 
     temperature = newTemp; 
    } 

    //getDay & setDay methods 
    public String getDay() 
    { 
     return day; 
    } 

    public void setDay(String newDay) 
    { 
     day = newDay; 
    } 

    //constructor 
    public DailyTemperature(String day, double temperature) 
    { 
    this.day = day; 
    this.temperature = temperature; 
    } 

    //compareTo method used for Comparable interface 
    public int compareTo(DailyTemperature other) 
    { 
    if (temperature < other.temperature) return -1; 
    if (temperature == other.temperature) return 0; 
    return -1; 
    } 


    //toString method to print out original arraylist contents and sorted arraylist contentes 
    public String toString() 
    { 
    return("Day of Week: " + this.getDay() + 
      " - Temperature: " + this.getTemp()); 
    } 
} 



import java.io.*; 
import java.util.Scanner; 
import java.util.ArrayList; 

public class DailyTemperatureList 
{ 
    public static void main (String [] args) 
    { 
    ArrayList<DailyTemperature> dailytemps = new ArrayList<DailyTemperature>(); 

    dailytemps.add(new DailyTemperature("Mon", 87.1)); 
    dailytemps.add(new DailyTemperature("Tue", 88.3)); 
    dailytemps.add(new DailyTemperature("Wed", 81.2)); 
    dailytemps.add(new DailyTemperature("Thu", 84.0)); 
    dailytemps.add(new DailyTemperature("Fri", 76.3)); 

    //original list printout 
    System.out.println("Original List:"); 
    System.out.println(dailytemps.toString()); 
    System.out.println(" "); //empty line 

    //method to sort array list 
    insertionSort(dailytemps); 
    //sorted list printout 
    System.out.println("Sorted List:"); 
    System.out.println(dailytemps.toString()); 

    //Collections.sort(dailytemps); 
    } 

    static void insertionSort(ArrayList<DailyTemperature> dailytemps) 
    { 
    DailyTemperature temp = null; 
    int position = 0; 

    //loop from 2nd element on 
    for (int i = 1; i < dailytemps.size(); i++) 
    { 
     temp = dailytemps.get(i); 
     position = i; 

     while (0 < position && temp.compareTo(dailytemps.get(position - 1)) < 0) 
     { 
     dailytemps.set(position, dailytemps.get(position - 1)); 
     position--; 
     } 
     dailytemps.set(position,temp); 
    } 
    } 
} 

回答

2

compareTo()执行不正确。如果this.temperature大于other.temparature,则永不返回1。

public int compareTo(DailyTemperature other) 
    { 
    if (temperature < other.temperature) return -1; 
    if (temperature == other.temperature) return 0; 
    return -1; // return 1; here 
    } 
+0

谢谢你,我从我的电脑复制到我们的学校帐户,键入-1而不是1.只是一个问题。在DailyTemperature类中,我是否必须实现可比较?因为它没有它的工作。 –

+0

你最好应该实现'可比'。如果没有它,尝试'Collections.sort()',你会得到一个异常。理想情况下,该类应该准备好用于'Collections' API以及像TreeSet一样的排序集合。 –

+0

由于您正在使用自定义排序并直接访问“compareTo”方法,因此您不需要执行“Comparable”。例如,如果您想使用'Collections.sort',则必须使用'Comparable'。 –

1

您的评论与评论Collections.sort是无论如何都是打印后行。你可以简单地做Collections.sort然后打印,而不是制作一个新的方法插入排序

+0

是的。但要做到这一点,'DailyTemperature'的声明需要包含'implements Comparable '。这正是我在这个问题的早期版本上给OP的建议。 –