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

[nospor]

No dobrze, ale niewazne czy masz ValueObject czy poprostu tekst, to przy bardzo zawilym kodzie, odpowiednie nazewnictwo zmiennych ma znaczenie tak czy siak

takze zmierzam do tego, ze uzywanie ValueObject nie wymusza na nas zmieniania nazw zmiennych. Zmiana nazw zmiennych jest potrzebna dla czytelnosci tylko, prosty przyklad

[PHP] pobierz, plaintext 
class valueObjectClass
{
    private $v1;
 
    public function __construct($v1)
    {
        $this->v1 = $v1;
    }
 
    public function display()
    {
        echo $this->v1;
    }
 
    public function modify($v1)
    {
        return new self($v1);
    }
}
 
$vo1 = new valueObjectClass(1);
echo 'After creation: ';
$vo1->display();
 
$vo2 = $vo1->modify(2);
echo 'After modification: ';
$vo2->display();
echo 'And again original: ';
$vo1->display();
 
echo 'Ok, now we will not change name of the variable';
$keepOriginal = $vo1;
$vo1 = $vo1->modify(22);
echo 'After modification: ';
$vo1->display();
echo 'And original not changed: ';
$keepOriginal->display();
 
[PHP] pobierz, plaintext