Присвоение переменной на последнем ППР

Question

Допустим, у вас есть метод, который присваивает переменной на вызов метода в конце:

def this_method(x) 
    x = x + 5 
    x = some_method_like_multiply_by_two(x) 
end 

и аналогичный метод, который не имеет назначение переменной:

def this_method(x) 
    x = x + 5 
    some_method_like_multiply_by_two(x) 
end 

Какой из них лучше? Нужно ли иметь x = в последнем утверждении метода? Является ли первый яснее? Ставит ли второй риск ввести больше ошибок тем, кто читает код, потому что они могут не знать, что он что-то делает с x? Какой из них является правильным способом кодирования? Когда я хочу использовать первый стиль (несмотря на то, что переменная отображается как неиспользованная в анализе кода)?

Answer 1

Почему бы не упростить жизнь еще дальше и сделать:

def this_method(x) 
    some_method_like_multiply_by_two(x + 5) 
end 

С x чисто локальный по отношению к методе и вы возвращаете результат вызова другого метода, обновление x с промежуточными операциями ничего не выполняет. Идите прямо к цели.

Answer 2

Вы пытаетесь изменить значение «x» как переданное в this_method? Если нет, тогда «x =» будет потрачено впустую, особенно потому, что вы не используете x, если только вы не возвращаете x из метода. Если вы у вас есть два варианта:

def this_method(x) 
    x = x + 5 
    x = some_method_like_multiply_by_two(x) 
    return x 
end 

или просто:

def this_method(x) 
    x = x + 5 
    x = some_method_like_multiply_by_two(x) 
    x 
end