0

このコードの最後に向かって、「到達不能ステートメント」エラーが発生します。いくつか試してみましたが、なぜこれが起こっているのかわかりません。エラーはコードの一番下にあります (エラーがある場所に // を付けてコメントしました) 正しい方向に向けて助けてください。困惑しています!

/**
* Describes a certain model.
* 
* @author (Joshua Baker) 
* @version (1.0)
*/
public class Model
{
public static final int IN_PER_FOOT = 12;
public static final int BASE_RATE = 60;
public static final int TALL_INCHES = 67;
public static final double THIN_POUNDS = 140.0;
public static final int TALL_THIN_BONUS = 5;
public static final int TRAVEL_BONUS = 4;
public static final int SMOKER_DEDUCTION = 10;

private String firstName;
private String lastName;
private int heightInInches;
private double weightInPounds;
private boolean travel;
private boolean smokes;
private String newHeight;
private int perHourRate;

/**
 * Default constructor
 */
public Model()
{
    setFirstName ("");
    setLastName  ("");
    setHeightInInches  (0);
    setWeightInPounds  (0.0);
    setTravel  (false);
    setSmokes  (false);
}

/**
 * 
 */
public Model (String whatIsFirstName, String whatIsLastName, int whatIsHeight, double whatIsWeight,
boolean canTravel, boolean smoker)
{
    setFirstName  (whatIsFirstName);
    setLastName  (whatIsLastName);
    setHeightInInches  (whatIsHeight);
    setWeightInPounds  (whatIsWeight);
    setTravel  (canTravel);
    setSmokes  (smoker);
}

/**
 *@return first name
 */
public String getFirstName()
{
    return firstName;
}

/**
 *@return last name
 */
public String getLastName()
{
    return lastName;
}

/**
 *@return height in inches
 */
public int getHeightInInches()
{
    return heightInInches;
}

/**
 *@return the converted height 
 */
public String getNewHeight()
{
    return newHeight;
}

/**
 *@return weight in pounds
 */
public double getWeightInPounds()
{
    return weightInPounds;
}

/**
 *@return models pay per hour rate
 */
public int getPerHourRate()
{
    return perHourRate;
}


/**
 *@return travel
 */
public boolean getTravel()
{
    return travel;
}

/**
 *@return smokes
 */
public boolean getSmokes()
{
    return smokes;
}

/**
 * models first name
 */
public void setFirstName(String whatIsFirstName)
{
    firstName = whatIsFirstName;
}

 /**
 * models last name
 */
public void setLastName(String whatIsLastName)
{
    lastName = whatIsLastName;
}

 /**
 * models height in inches
 */
public void setHeightInInches(int whatIsHeight)
{
    if (whatIsHeight >0){
    heightInInches = whatIsHeight;
    }

}

 /**
 * models weight in pounds
 */
public void setWeightInPounds(double whatIsWeight)
{
    if (whatIsWeight >0){
    weightInPounds = whatIsWeight;
    }
}

 /**
 * can model travel
 */
public void setTravel(boolean canTravel)
{
    travel = canTravel;
}

 /**
 * does model smoke
 */
public void setSmokes(boolean smoker)
{
    smokes = smoker;
}

/**
 * Converts to feet and inches
 */
public String convertheightToFeetInches() 
{
int leftOver = (heightInInches %= IN_PER_FOOT);
int newHeight = (heightInInches % IN_PER_FOOT);
return newHeight + "Foot" + leftOver + "Inches";
}

/**
 * 
 */
public int calculatePayPerHour(){
    if (heightInInches  >= TALL_INCHES && (weightInPounds <= THIN_POUNDS)) {
        perHourRate = BASE_RATE + TALL_THIN_BONUS;
        return perHourRate;
    }
    else
    {
        perHourRate = BASE_RATE;
        return perHourRate;
    }

    if (travel)  {          //unreachable statement


        perHourRate = BASE_RATE + TRAVEL_BONUS;   
        return perHourRate;
    }
    else
    {
        perHourRate = BASE_RATE;
        return perHourRate;

    }

    if (smokes) {             //unreachable statement
        perHourRate = BASE_RATE - SMOKER_DEDUCTION;
        return perHourRate;
    }

    else {}

    }






/**
 * Displays details
 */
public  void displayInfo()
{
    System.out.print("Name : " + getFirstName() + " ");
    System.out.println(getLastName());
    System.out.println("Height : " + getNewHeight() + "inches");
    System.out.println("Weight : " + getWeightInPounds() + "pounds");
    System.out.print("Travel : " + getTravel() + " " );
    System.out.print("Smokes : " + getSmokes() );
    System.out.println("Hourly rate : " + getPerHourRate() );
}

}

4

5 に答える 5

11

これは、プログラムが最初のブロックif blockまたは対応するelseブロックから戻るためです。

if (heightInInches  >= TALL_INCHES && (weightInPounds <= THIN_POUNDS)) {
    perHourRate = BASE_RATE + TALL_THIN_BONUS;
    return perHourRate;
}
else
{
    perHourRate = BASE_RATE;
    return perHourRate;
}
System.out.println("This will never get printed. And will show compiler error");

したがって、2 つの return ステートメントのいずれかが実行されます。したがって、それ以上のコードには到達できません。


service rates最終的な を取得するには、すべての累積合計が必要なようです。そのために、各ブロックからperHourRateを削除できます。そして、最初のブロックの後のすべてのブロックに対して、 を に割り当てる代わりに、 を実行します。return statementif-elseif-elsecurrent priceperHourRatecompound addition +=

また、インスタンス フィールド -perHourRateで作業しているため、それを返す必要はまったくありません。行った変更は、perHourRateを使用して取得できますgetPerHourRate()。したがって、戻り値の型を に変更しvoidます。

calculatePayPerHourメソッドを以下のように更新してみてください。

public void calculatePayPerHour(){
    if (heightInInches  >= TALL_INCHES && (weightInPounds <= THIN_POUNDS)) {
        perHourRate = BASE_RATE + TALL_THIN_BONUS;  // Initial assignment

    } else {
        perHourRate = BASE_RATE;  // Initial assignment
    }

    /** Rest of the assignment will be compound assignment, since you
        are now updating the `perHourRate` **/

    if (travel)  {      
        perHourRate += TRAVEL_BONUS;   
    } // You don't need an else now. Since BASE_RATE is already added

    if (smokes) {    
        perHourRate -= SMOKER_DEDUCTION;
    }
}
于 2013-02-08T08:19:33.783 に答える
3

メソッド内にcalculatePayPerHourif/else があり、どちらの場合も結果を返すため、ステートメントに到達することはありません。

if (heightInInches  >= TALL_INCHES && (weightInPounds <= THIN_POUNDS)) {
    perHourRate = BASE_RATE + TALL_THIN_BONUS;
    return perHourRate; // you return in the if
}
else
{
    perHourRate = BASE_RATE;
    return perHourRate; // you return in the else
}

... the execution will never reach here
于 2013-02-08T08:20:02.100 に答える
3
if (heightInInches  >= TALL_INCHES && (weightInPounds <= THIN_POUNDS)) {
    perHourRate = BASE_RATE + TALL_THIN_BONUS;
    return perHourRate; // <------------
}
else
{
    perHourRate = BASE_RATE;
    return perHourRate;  // <------------
}

身長や体重に関係なく、これら 2 つの戻り値のいずれかがトリガーされるため、それ以降のステートメントは実行されません。これは、以下のコードと同じです。

if (heightInInches  >= TALL_INCHES && (weightInPounds <= THIN_POUNDS)) {
    perHourRate = BASE_RATE + TALL_THIN_BONUS;
}
else
{
    perHourRate = BASE_RATE;
}

return perHourRate;  
//unreachable
于 2013-02-08T08:20:44.217 に答える
2

コードの上のif/elseには、すでに2つのreturnステートメントがあります...これは、この下のコードに決して到達しないことを意味します..

于 2013-02-08T08:21:29.350 に答える
2
  • あなたのメソッドpublic int calculatePayPerHour()では、最初の if else ステートメントは、どのような場合でも (if ブロックと else ブロックの両方で) 値を返しています。
  • プログラムをドライランすると、制御フローがこのブロックの下のステートメントに到達しないため、例外が発生することがわかります。
于 2013-02-08T08:22:41.277 に答える