0

我要做的是总结危险等级中的影响值

例如，它将通过一个居住者列表，找到一个危险并从中获取影响量。然后将所有危险的总影响相加并将该值返回给我。

下面我有Cave 类、Hazard 类和Abstract Occupant 类。

当向洞穴添加危险时，它会成为 HashSet 中的居住者。尝试使用 getImpact() 方法获取能量级别时，无法访问该方法，因为它在 Hazard 而非 Occupant 中。

我还有另外两个课程也扩展了 Occupant。播放器和物品。

在添加到 HashSet 以便可以使用 getImpact() 方法时，我找不到将危险保留为 Hazard 类的方法。

这也需要在添加到 HashSet 时满足其他类 Player 和 Item。

public class Cave {

HashSet<Occupant> occupants;
private double impact;

/**
 * Creat a new Cave instance with no occupants.
 */
public Cave() 
{
    occupants = new HashSet<Occupant>();
}

/**
 * Adds an occupant to a Cave if the occupant is not already there and
 * if cave currently has fewer than the maximum number of occupants.
 * @param occupant, the occupant to add
 * @return  true if successfully added
*/
public boolean addOccupant(Occupant occupant) {
    boolean validNewOccupant = occupant != null;
    boolean enoughRoom = occupants.size() < MAX_OCCUPANTS;
    if (validNewOccupant && enoughRoom) {
        validNewOccupant = occupants.add(occupant);
    }

    return validNewOccupant && enoughRoom;
}

/**
 * Gets the sum of the impact from all hazards in the cave
 * @returns hazardEnergyImpact
 */
public double getHazardEnergyImpacts(){
    double energyImpact = 0.0;
    for( Occupant occupant : occupants ){
        if(occupant.toString() == "!"){
            energyImpact += occupant.getImpact();
        }
    }
    return energyImpact;
}
}

public abstract class Occupant {

private Address address;
private String name;

/**
 * Construct an occupant for a known address & name.
 * @ param row, row of address  
 * @ param column, row of address.
 * @ param name, occupant's name
 */
public Occupant(Address address, String name) {
    this.address = address;
    this.name = name;
}

@Override
public String toString(){
    return "";
}
}

public class Hazard extends Occupant  {

private String longDescription;
private double impact;

/**
 * Construct a hazard with know attributes
 * @param row
 * @param column
 * @param name
 * @param longDescription
 * @param impact
 */
public Hazard(Address address, String name, String longDescription, double impact) {
    super(address, name);
    this.longDescription = longDescription;
    this.impact = impact;
}

@Override
public String toString(){
    return "!";
}

/**
 * gets impact amount
 * @returns impact
 */
public double getImpact(){
    return this.impact;
}
}
4

4 回答 4

3

另一种选择是将getImpact()方法添加到 Occupant，例如，

public double getImpact() {
    return 0.0;
}

而Hazard' 的@Override实现getImpact()只会返回它的impact实例变量，因为你已经设置了它。然后，您的循环被简化为：

public double getHazardEnergyImpacts() {
    double energyImpact = 0.0;
    for( Occupant occupant : occupants ) {
        energyImpact += occupant.getImpact();
    }
    return energyImpact;
}

如果您以后需要提取到适当的接口抽象，现代 IDE 使这变得容易是一件好事。

于 2011-04-28T03:02:49.147 回答
1

当迭代你的时候，occupants你可以检查每个项目是否是Hazard这样的：

for(Occupant occupant : occupants){
    if(occupant instanceof Hazard){
        Hazard hazard = (Hazard) occupant; // now it's safe to cast
        double impact = hazard.getImpact();
        // do what you want with impact
    }
}
于 2011-04-28T02:35:14.113 回答
0

我会在这里使用访客模式。

public interface Occupant {
  void interact(Player p);
}

public class Player {
  public void handleInteraction(Hazard hazard) {
    // add code here
  }
  public void handleInteraction(Person person) {
    // add code here
  }
}

public class Hazard implements Occupant {
  public void interact(Player p) {
    p.handleInteraction(this);
  }

  public double getImpact(){
    return this.impact;
  }
}
于 2011-04-28T02:59:51.790 回答
0

杰里米打败了我。

但是，instanceof 并不总是最好的解决方案。但在这种情况下，这是一个修复。

我实际上建议在这里使用接口来处理行为，而不是使用抽象类。但是，如果必须使用抽象类，更有效的方法是简单地创建要在子类中使用的抽象方法。您必须在每个孩子中都覆盖它们，但您不必在每种情况下都实现它们。

于 2011-04-28T02:48:10.970 回答