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

[Brick]

Dzięki za odpowiedzi.

W książce chodziło trochę o inny przypadek, gdy argument funkcji nazywa się inputValue i potem w ciele funkcji robi się na nim różne działania i na końcu jest zwracana zmienna, dalej nazywająca się inputValue. To jest jasny przypadek, że tak nie powinno się robić bo wprowadza w błąd. Ale zastanawiałem się czy ma sens pójście krok dalej czyli trochę sztuczne dodatkowe nazywanie zmiennych. W moim przykładzie "doSomething" jest oczywiście bardziej złożoną procedurą, więc użycie zmiennej robi się konieczne.

No ale chyba macie rację, takie kombinowanie sztuczne z nazwami nie ma sensu.