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

[viking]

1. zależy od kontekstu. Ja używam zazwyczaj tylko jak potrzebuję oryginalną zmienną gdzieś jeszcze. I kod też od razu zamiast return $string, return some();
2. Wiesz że jest PHP 7 i to co opisałeś załatwia się bezpośrednio? https://seld.be/notes/php-versions-stats-2017-2-edition Według statystyk packagist PHP 7 to już prawie 70% wszystkich wersji. Może pora przestać udawać że nie istnieje?