Ruby una riga se dichiarazione di reso

Question

c'è un modo per abbreviare questa linea su Ruby?

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

su

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 

quando il contenuto di bla_permission_invalid sono qualcosa come stringa

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 e permission_error_report_func ritorna

Answer 1

Se p I valori ossible sono String e NilClass, quindi il codice può essere semplificata a questo:

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

Per divertimento, si potrebbe riscrivere il metodo something in questo modo:

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

Ma ancora più importante, Mark Thomas merita di credito per la sua osservazione, il problema in questione dovrebbe essere risolto utilizzando eccezioni personalizzate.

L'approccio al codice di errore è valido in lingue che non hanno eccezioni. Ruby li ha.

Answer 4

Mark Thomas ha già notato in his comment che sembra che tu stia cercando di gestire gli errori da solo utilizzando una sorta di identificatore di stringa. Si potrebbe utilizzare le eccezioni invece:

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 

Nel codice precedente something chiama bla_permission_invalid, svolge il suo lavoro e restituisce true. Se viene sollevata un'eccezione in bla_permission_invalid, automaticamente si propaga sullo stack delle chiamate, non è necessario restituirlo esplicitamente da something.

per gestire l'eccezione:

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