Czy można poddawać recyklingowi lub ponownie używać zmiennych?

Question

Szukałem wszędzie i nie mogę znaleźć odpowiedzi w taki czy inny sposób. Czy jest to do przyjęcia (dobra lub zła praktyka) do ponownego użycia lub recyklingu zmiennej? Działa, i użyłem tej metody kilka razy, ale nie wiem, czy powinienem to robić, czy nie. Próbuję uciec od używania metod statycznych i przejść do zastrzyku zależności.

w tym przykładzie nazwa $ table_name jest ustawiona gdzie indziej.

class DbObject { 
    private $db = NULL; 
    protected $table_name; 

    public function __construct($dbh, $item) { 
     $this->db = $dbh; 
     $this->$table_name = $item; 
    } 

    // counts items in database // 
    public function count_all() { 
     try { 
      $sql = 'SELECT COUNT(*) FROM ' . $this->table_name; 

      $stmt = $this->db->query($sql); 
      $stmt->setFetchMode(pdo::FETCH_COLUMN, 0); 
      $result = $stmt->fetchColumn(); 
      return $result; 
     } catch (PDOException $e) { 
      echo $e->getMessage()); 
     } 
} 

Aby użyć tego chciałbym użyć go w ten sposób:

$total_count = new DbObject(new Database(), 'items'); 
$total_count = $total_count->count_all(); 

Czy to akceptowalnym sposobem na kod? Dzięki za pomoc.

Answer 1

Ponowne użycie zmiennych do różnych celów to błąd konserwacyjny, który może się wydarzyć.

To zła praktyka. Dobrze określona zmienna może zdziałać cuda, pomagając w zrozumieniu kodu.

Ponowne użycie zmiennych jest szczególnie delikatne w słabo napisanych językach dynamicznych, takich jak PHP.

[W przyćmionym przeszłości mam natknąć błędów w kodzie produkcyjnym (odczekać, myślę, że to było moje!), Gdzie ponowne użycie pętli lokalnej vars jak i oraz j prowadzić do błędów ...]

Answer 2

Ludzie mają tendencję do ponownego użycia zmiennych „jednorazowe” jak i i j, ale jest to zwykle zła forma do ponownego użycia innych zmiennych lokalnych w innym celu. Prowadzi to do nieporozumień czytelnika na temat tego, gdzie zainicjowano zmienną i czy aktualizacje w jednej części kodu wpłyną na inną część kodu. Mogą również wystąpić złe efekty w pakiecie optymalizacyjnym kompilatora (być może sugerując, że zmienna powinna zostać zapisana, gdy nie jest to konieczne).

Answer 3

co kiedy potrzebujesz zadzwonić inną metodą z DbObject?

Wolę dać nazwę zmiennej, co to jest:

$dbo = new DbObject(new Database(), 'items'); 
$total_count = $dbo->count_all(); 

//so you call still do things next 
$result = $dbo->get_all(); 

Answer 4

Jeśli podoba Ci się używając nazwy pospolite często przechodzące skryptu (np $i do zliczania iteracji w pętli) nie powinno być żadnych problemów, jeśli zrobić przyzwyczajenie się do wywoływania unset(), gdy skończysz używać zmiennej w konkretnym przypadku.

Answer 5

Głównym powodem uniknięcia ponownego użycia zmiennych jest to, że jeśli ponownie wykorzystasz zmienną bez prawidłowego jej ponownego zainicjowania, stara wartość "wycieknie", powodując nieprzewidywalne efekty, a nawet luki w zabezpieczeniach. Na przykład:

$foo = $_GET['input']; 
# use $foo 

if ($a == $b) { 
    $foo = 1; 
} else { 
    # $foo = 2; # Commented out for some reason 
} 
# Value $foo supplied in URL leaks through to here 

Generalnie, ponowne użycie zmiennych nie uszkodzić wydajność jeśli kompilator używa pojedynczej statyczny formularz przypisania (SSA) jako formę pośrednią podczas optymalizacji (część co SSA nie daje odrębne nazwy do takiego ponownego wykorzystania zmienne). Nie martw się o wydajność - martw się o łatwość konserwacji.

Answer 6

Czasami zmienne mogą powodować konflikt lub można je wymieszać. Tak więc ponownie użyłbym tylko $i i $row, chyba że po zakończeniu będziesz unset() zmienną.