메서드는 있지만 작동하지 않습니다.

Question

다음 코드가 있습니다. Java btw-에서 제대로 컴파일되지만 잘못된 매개 변수를 입력하면 오류로 인식되지 않고 그대로 받아들입니다 조건을 만났습니다. 나를 염려하는 방법은 SetMPG (int 평균)입니다. 내 질문이 애매한 경우 사과 드리며 필요한 경우 더 많은 정보를 입력하겠습니다.

public class Vehicle { 
    // instance variables - replace the example below with your own 
    private int tireCount; 
    private int mPG; 

    /** 
    * Constructor for objects of class Vehicle 
    */ 
    public Vehicle(int tCount, int mP) { 
     // initialise instance variables 
     tireCount = tCount; 
     mPG = mP; 
    } 


    public void setTire(int tire) { 
     if (tire >= 0) { 
      tireCount = tire; 

     } else/*if(tire < 0)*/ { 
      throw new IllegalArgumentException("Values must be positive"); 
     } 

    } 

    public void setMPG(int average) { 
     if (average > 0) { 
      mPG = average; 
     } else if (average < 0) { 

      throw new IllegalArgumentException("Values must be positive"); 
     } 

    } 

    public int getTire() { 
     return tireCount; 
    } 

    public int getMPG() { 
     return mPG; 
    } 

    public String toString() { 
     return String.format("There are " + tireCount + " tires and an average of " + mPG + "mpg"); 
    } 

public class VehicleTest 
{ 
// instance variables - replace the example below with your own 
public static void main(String []args) 
{ 
    Vehicle bike = new Vehicle(2,-23); // first parameter is for tires , second is for MPG 
    System.out.println(bike); 

} 
} 

Answer 1

귀하의 코드와 당신이 말하는 바에 따르면, 당신은 생성자를 통해 매개 변수를 설정하는 것이 가장 가능성이 높습니다. 생성자를 다음 형식으로 변경하십시오.

public Vehicle(int tCount , int mP) 
{ 
    // initialise instance variables 
    setTire(tCount); 
    setMPG(mP); 
} 

또한 0이 mpg에 유효한 값인지 여부도 확실하지 않습니까 ???

public void setMPG(int average) 
{ 
    if(average > 0) //should it be >= 0??? 
    { 
     mPG=average; 
    } 
    else if(average < 0) // should it be <=0 ???? 
    { 
     throw new IllegalArgumentException("Values must be positive"); 
    } 
}