We prefer 'if' conditional block over a trailing 'if' for multiline code.
For code with single line adding if condition at the end is definitely preferred but for multiline code it makes it more readable and the developer does not have to read the whole block of code only to realize that its conditional.
Condition on multiline Code:
#bad
def update
return render json: {errors: "You can't to edit this directory"},
status: :forbidden if @directory.master?
end
#preferred
def update
if @directory.master?
return render json: {errors: "You can't to edit this directory"},
status: :forbidden
end
end
Condition on block of code:
#bad
10.times do
# multi-line body omitted
end if some_condition
#preferred
if some_condition
10.times do
# multi-line body omitted
end
end