Witam,

Pisząc swój skrypt natknąłem się na 'moralny' dylemat. Mianowicie starając się jak najbardziej skrócić kod, stworzyłem kod który w wyniku skracania ma.. dwie instrukcje warunkowe więcej niż w pierwotnym rozwiązaniu. Obecnie wygląda on tak:

[PHP] pobierz, plaintext 
				try {
					$pdo = new PDO('mysql:host=localhost;dbname=baza', 'xxx', 'yyy', array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
					$pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
					$imie = $pdo -> prepare ("UPDATE users_act SET ".($_POST['imie'] !== "" ? "imie=:imie" : "")." ".($_POST['nazwisko'] !== "" && $_POST['imie'] !== "" ? "," : "")." ".($_POST['nazwisko'] !== "" ? "nazwisko=:nazwisko" : "")." WHERE login=:login");
 
					$_POST['imie'] !== "" ? $imie-> bindValue(':imie', trim($_POST['imie']), PDO::PARAM_STR) : "";
					$_POST['nazwisko'] !== "" ? $imie -> bindValue(':nazwisko', trim($_POST['nazwisko']), PDO::PARAM_STR) : "";
					$imie -> bindValue(':login', $login, PDO::PARAM_STR);
 
					$ile = $imie -> execute();
 
					$imie -> closeCursor();
				}
 
				catch(PDOException $e){
					echo 'Błąd bazy danych';
				}	
[PHP] pobierz, plaintext 

Mogłem to zrobić tworząc 3 warunki: albo 'imie' i 'nazwisko' wypełnione, albo samo 'imie', albo samo 'nazwisko' - a następnie do każdego warunku inne zapytanie. W wyżej zamieszczonym rozwiązaniu zmieściłem niejako wszystko w jednym zapytaniu ale utworzyło się... 5 instrukcji warunkowych. Które rozwiązanie będzie korzystniejsze jeśli chodzi o wydajność skryptu?

Pozdrawiam.

Chłopie, wyluzuj. Dwie instrukcje warunkowe więcej? Kilka linii kodu więcej? A to wszystko w obliczu interakcji z bazą danych, którą masz dostępną zapewne po sieci... O czym tu mówić...

Pamietaj też o tym, że niepotrzebnie zaciemniasz sobie kod.
Czasem wolę napisać coś trochę nadmiarowo (co i tak nie zrobi różnicy), ale przynajmniej za jakiś czas nie bedę miał problemów z szybkim przypomnieniem sobie o co chodziło w danej funkcji.