Rubyの1行でreturn文

Question

がある場合は、Rubyでこの行を短縮する方法はありますか？ P場合bla_permission_invalidの内容は

def bla_permission_invalid 
    return invalid_address_report_func if invalid_address? 
    return permission_error_report_func if @user.not_one_of? [ :group1, :group2 ] 
    return nil 
end 

invalid_adress_report_funcとpermission_error_report_func返す文字列のようなもの

Answer 1

ある

def something # many things that like this 
    if (res = bla_permission_invalid).is_a? String then return res end 
    # do something else 
    return true 
end 

上

if (res = bla_permission_invalid).is_a? String then return res end 

ossible値はStringとNilClassあり、その後、コードはこのように単純化することができます。

def something 
    res = bla_permission_invalid() 
    return res if res # strings are truthy, so they'll be returned but nil will proceed 

    # do something else 
    true 
end 

Answer 2

def something 
    bla_permission_invalid || (
    # do something else 
    true) 
end 

Answer 3

は楽しみのために、人はこのようなあなたのsomething方法を書き換えることができます：

def something 
    true.tap do 
    bla_permission_invalid.tap { |res| return res if res.is_a? String } 
    # do something else (thx Sergio) 
    end 
end 

しかし、もっと重要なのは、マーク・トーマスは、信用に値します彼の観察のために、手作業での問題は、カスタム例外を使用して解決する必要があります。

例外を持たない言語では、エラーコードの手法が優れています。 Rubyにはそれらがあります。

Answer 4

マークトーマスはすでにhis commentに記載されているように、何らかの文字列識別子を使用して自分でエラーを処理しようとしているようです。あなたが代わりに例外を使用することができます：bla_permission_invalidを呼び出すsomething上記のコードで

class AddressError < StandardError; end 
class PermissionError < StandardError; end 

def something 
    bla_permission_invalid 
    # do something 
    true 
end 

def bla_permission_invalid 
    raise AddressError if invalid_address? 
    raise PermissionError if @user.not_one_of? [ :group1, :group2 ] 
end 

を、その作業を実行し、trueを返します。 bla_permission_invalidで例外が発生した場合は、自動的に呼び出しスタックを伝播します。somethingから明示的に返す必要はありません。例外処理するには

：

begin 
    something 
rescue AddressError 
    # handle address error 
rescue PermissionError 
    # handle permission error 
end