我想知道我是否正确地编写了这个等于方法。即使两个对象不相等Item
,我的程序也会不断打印出“相等”。otherObject
我有三个 equals 方法,每次运行一个似乎都不能正常工作。三个 equals 方法紧挨着放置以供参考。我的主线在最后。
import static java.lang.System.*;
import java.text.DecimalFormat;
import java.util.Arrays;
import java.util.Scanner;
import java.math.*;
public class Item {
DecimalFormat df = new DecimalFormat("#.00");
//the properties of an Item
static double cash=59.00;
static double sum=0.00;
private int priority;
private String name;
private double price;
static boolean value= false;
//default constructer
public Item() {
priority = -1; //fill with default values
price = 0.00;
name = "No name yet";
}
public Item(int priority, String name, double price) {//constructor with all 3 arguments
this.priority = priority;
this.name = name;
this.price = price;
}
public int getPriority() {
return priority;
}
public void setPriority(int priority) {
//priority must be between 1 and 7
if (priority > 0 && priority <= 7) {
this.priority = priority;
} else {
System.err.println("Error, enter 1 through 7");
}
}
public String getName() {
return name;
}
public void setName(String name) {
this.name = name;
}
public double getPrice() {
return price;
}
public void setPrice(double price) {
//price between 0 and 100 inclusive
if (price >= 0.00) {
if (price <= 100.00) {
this.price = price;
cash = cash-=price;
sum=sum+=price;
} else {
System.err.println("Error: price to high");
}
} else {
System.err.println("Error: price to low");
}
}
public boolean equals(Item otherObject) {
return this.name.equals(otherObject.name);
}
/*public boolean equals(Item otherObject) {
if(this.getPriority()==(otherObject.getPriority()));
return true;
} */
@Override
public int hashCode() {
final int prime = 31;
int result = 1;
result = prime * result + ((name == null) ? 0 : name.hashCode());
result = prime * result + priority;
return result;
}
/*@Override
public boolean equals(Object obj) {
if (this == obj)
return true;
if (obj == null)
return false;
if (!(obj instanceof Item))
return false;
Item other = (Item) obj;
if (name == null) {
if (other.name != null)
return false;
} else if (!name.equals(other.name))
return false;
if (priority != other.priority)
return false;
return true;
}*/
@Override
public String toString() {
StringBuilder builder = new StringBuilder();
builder.append("Item [Price= ").append(getPrice()).append(", ");
if (getName() != null) {
builder.append("Name= ").append(getName()).append(", ");
}
builder.append("Priority= ").append(getPriority()).append("]");
return builder.toString();
}
public static void main (String[] args) {
Item[] list = new Item[2];
Scanner keyboard = new Scanner(System.in);
for (int i = 1; i <= list.length; i++) {
if(cash==59)
{
System.out.println("You have 59 dollars");
}
Item otherObject=new Item();
Item anItem = new Item(); // new item object created 7 times
System.out.println("Enter an item you want to add to your list " + i);
anItem.setName(keyboard.next());
System.out.println("Enter a price " + i);
anItem.setPrice(keyboard.nextDouble());
System.out.println("Enter the priority of the item " + i);
anItem.setPriority(keyboard.nextInt());
list[i-1] = anItem;
System.out.println("Cash left "+cash);
System.out.println("Sum of Items "+sum);
System.out.println(Arrays.toString(list));
if (anItem.equals(otherObject)); //--------------- This is printing out each time. Is it comparing default constructors?
{System.out.println("\nequal");}
}
if(sum>59)
{System.err.println("Error, you ran out of money\t\t");
}
// int a;
//int b;
//a=list[0].getPriority();
// b=list[1].getPriority();
//System.out.println(a +" here");
// System.out.println(b +" here");
//final int[] arraySort = { a, b,};
Item temp;
for (int i = 0; i < list.length; i++) {
//min = i;
for (int j = 1; j < (list.length - i); j++) {
if (list[j-1].getPriority() > list[j].getPriority()) {
temp = list[j - 1];
list[j - 1] = list[j];
list[j] = temp;
}
} //min = j;
System.out.println(list[i]);
}
} //main
}// class Item