0

だから私は文字列配列を持っています 

private  String[] transmission = {"Drive", "Park", "Reverse"};

これが私の set メソッドです。あまり使用しないため、switch ステートメントを練習しています。

public void setTransmission(String[] transmission) {
    System.out.println("To change the transmission, enter D for Drive,P for Park or R for reverse");
    switch (input.nextLine()) {
        case "D":
        case "d":
            System.out.println("The Car is currently in Drive.");
            transmission[0] = this.transmission[0];
            break;
        case "P":
        case "p":
            System.out.println("The Car is currently in Park.");
            transmission[1] = this.transmission[1];
            break;
        case "R":
        case "r":
            System.out.println("The Car is currently in Reverse");
            transmission[2] = this.transmission[2];
            break;

    }

ここに本当の問題があります。私の getMethod では、配列の最初のインデックスのみを出力します。

 public String getTransmission()
    {
        return String.format("The car is currently in %s",transmission);
    }

ユーザーが入力した内容を印刷するにはどうすればよいですか? 文字列変数を使用できることはわかっていますが、配列を使用することをお勧めします。

4

1 に答える 1

1

これが同じクラスで発生すると仮定すると、内部状態としてインデックスが必要になります。

可能な実装は次のとおりです。

public class Car
{
    private final String[] transmission = { "Drive", "Park", "Reverse" };

    private final Scanner  input;

    private int            index;

    public Car( Scanner input )
    {
        this.input = input;
    }

    public void setTransmission()
    {
        System.out.println( "To change the transmission, enter D for Drive,P for Park or R for reverse" );
        switch ( input.nextLine() )
        {
        case "D":
        case "d":
            System.out.println( "The Car is currently in Drive." );
            index = 0;
            break;
        case "P":
        case "p":
            System.out.println( "The Car is currently in Park." );
            index = 1;
            break;
        case "R":
        case "r":
            System.out.println( "The Car is currently in Reverse" );
            index = 2;
            break;
        }
    }

    public String getTransmission()
    {
        return String.format( "The car is currently in %s", transmission[index] );
    }
}

問題にもう少しコンテキストを追加できると助かります。特に、メソッドの呼び出し方法とメソッドが存在する場所。

より良い設計は、おそらく送信用の列挙型を作成し、解析をデータから分離することです。

これにより、車はロジックのないデータ コンテナーに縮小されます。

public class Car
{
    public enum Transmission
    {
        Drive, Park, Reverse
    }

    private Transmission transmission;

    public void setTransmission( Transmission transmission )
    {
        this.transmission = transmission;
    }

    public String getTransmission()
    {
        return String.format( "The car is currently in %s", transmission );
    }
}

この Car クラスを使用するアプリケーションは、トランスミッションを解析し、型指定されたトランスミッションを自動車に設定します。

public class CarApplication
{
    public static void main( String[] args )
    {
        try ( Scanner input = new Scanner( System.in ) )
        {
            System.out.println( "To change the transmission, enter D for Drive,P for Park or R for reverse" );
            String answer = input.nextLine();
            Transmission transmission = parseTransmission( answer );

            Car car = new Car();

            car.setTransmission( transmission );

            System.out.println( car.getTransmission() );
        }
    }

    private static Transmission parseTransmission( String input )
    {
        switch ( input )
        {
        case "D":
        case "d":
            return Transmission.Drive;
        case "P":
        case "p":
            return Transmission.Park;
        case "R":
        case "r":
            return Transmission.Reverse;
        default:
            throw new IllegalArgumentException( "expected an input D,P,R but got: " + input );
        }
    }
}
于 2014-02-17T20:37:22.700 に答える