ruby​​ method：ifステートメント内のステートメント

Question

私のプログラムで、私のプログラムに「終了」マーカーが多すぎる/していないというエラーが多かったです。私は前にこのコードをテストしていますが、動作しますが、if文またはtoo manyに十分な "end"があるかどうかを誰かが教えてくれるかどうかは疑問です。 ！おかげ

はデフホップ（d）の自動字下げをサポートしているテキストエディタで

if d== 0 
    if @current_location.addpoint(0,1) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(0,1) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(0,1) 
    puts "all good" 
    end 
    end 
elsif d == 1 
if @current_location.addpoint(0,-1) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(0,-1) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(0,-1) 
    puts "all good" 
    end 
    end 
elsif d== 2 
if @current_location.addpoint(1,0) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(1,0) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(1,0) 
    puts "all good" 
    end 
    end 

else d= 3 
if @current_location.addpoint(-1,0) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(-1,0) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(-1,0) 
    puts "all good" 
    end 
    end 

end 

Answer 1

、あなたのコードは次のようになります。

は

if d== 0 
    if @current_location.addpoint(0,1) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(0,1) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(0,1) 
    puts "all good" 
    end 
end 
elsif d == 1 
    if @current_location.addpoint(0,-1) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(0,-1) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(0,-1) 
    puts "all good" 
    end 
end 
elsif d== 2 
    if @current_location.addpoint(1,0) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(1,0) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(1,0) 
    puts "all good" 
    end 
end 
else d= 3 
    if @current_location.addpoint(-1,0) < @boundary1 
    puts "error" 
    elsif if @current_location.addpoint(-1,0) > @boundary2 
    puts "error2" 
    else 
    @current_location= @current_location.addpoint(-1,0) 
    puts "all good" 
    end 
end 

end 

別にendの数が正しくありませんから、それをよあなたがelsifで苦労しているようですが、私はどんな状況も考えることができません。elsif if [...]は書くのに良いことです

Answer 2

あなたがコードを正しくインデントしているかどうかは明らかです。次のコードでは、適切にインデントされるとend sの問題とelsif ifが修正されています

def hop!(d) 
    if d == 0 
    if @current_location.addpoint(0,1) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(0,1) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(0,1) 
     puts "all good" 
    end 
    elsif d == 1 
    if @current_location.addpoint(0,-1) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(0,-1) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(0,-1) 
     puts "all good" 
    end 
    elsif d == 2 
    if @current_location.addpoint(1,0) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(1,0) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(1,0) 
     puts "all good" 
    end 
    elsif d == 3 
    if @current_location.addpoint(-1,0) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(-1,0) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(-1,0) 
     puts "all good" 
    end 
    end 
end 

また、この場合にはcaseを使用すると、よりエレガントになります：

def hop!(d) 
    case d 
    when 0 
    if @current_location.addpoint(0,1) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(0,1) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(0,1) 
     puts "all good" 
    end 
    when 1 
    if @current_location.addpoint(0,-1) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(0,-1) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(0,-1) 
     puts "all good" 
    end 
    when 2 
    if @current_location.addpoint(1,0) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(1,0) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(1,0) 
     puts "all good" 
    end 
    when 3 
    if @current_location.addpoint(-1,0) < @boundary1 
     puts "error" 
    elsif @current_location.addpoint(-1,0) > @boundary2 
     puts "error2" 
    else 
     @current_location = @current_location.addpoint(-1,0) 
     puts "all good" 
    end 
    end 
end 

Answer 3

case d 
when 1: 
    puts "error" and return if @current_location.addpoint(0,1) < @boundary1 
    puts "error2" and return if @current_location.addpoint(0,-1) > @boundary2 
    puts "all good" 
    break 
when 2: 
    puts "error" and return if @current_location.addpoint(1,0) < @boundary1 
    puts "error2" and return if @current_location.addpoint(1,0) > @boundary2 
    puts "all good" 
    break 
when 3: 
    ... 

この私は思っているだけでなく、トリックを行う必要があります。

Answer 4

case