Zmienne lokalne i sprawdzanie argumentów Opcje

[Brick]

W sumie dwa pytania:
1. Czy używacie specjalnych zmiennych lokalnych wewnątrz funkcji żeby zwiększyć czytelność kodu?
Zamiast:

[PHP] pobierz, plaintext 
function convertString($string)
{
    $string = doSomething($string);
    return $string;
}
[PHP] pobierz, plaintext 

Coś takiego:

[PHP] pobierz, plaintext 
function convertString($input_string)
{
    $converted_string = $input_string;
    $converted_string = doSomething($converted_string);
    return $converted_string;
}
[PHP] pobierz, plaintext 

W książce "Kod doskonały" autor pisze, że jest to dobry zwyczaj bo jest jasne co jest wartością zadaną a co wyjściową. Ale zastanawiam się jak to jest w praktyce.

2. Czy sprawdzanie argumentów funkcji ma zawsze sens?
Przykładowo

[PHP] pobierz, plaintext 
function convertString($string)
{
    if ($string)
    {
         $string = doSomething($string);
    }
    return $string;
}
[PHP] pobierz, plaintext 

Na ogół tak robię zakładając, że nie ma sensu wykonywać jakiś operacji jeżeli nie ma na czym tego wykonać (wartość równa null). Ale może to niepotrzebne komplikowanie kodu gdyż (jak zakładam) PHP nie będzie wykonywać operacji na pustych danych?
Cały czas mam na myśli właśnie takie małe, dość proste funkcje a nie bardziej złożone gdzie kontrola argumentów jest po prostu konieczna.

Ten post edytował Brick 14.11.2017, 13:18:23

[Pyton_000]

beberlei/assert normalnie na prod. właśnie żeby nie robić ifów, po prostu pakujesz wartość i jak coś nie gra to rzuca wyjątkiem.

Co do sprawdzania powiem jeszcze raz, sprawdzaj zawsze bo jesli oczekujesz string jako input to tak ma być, a każda inna powoduje jakąś akcję albo exception, albo zwrot domyślnego return.