Cześć,

czy tworzenie zmiennych przechowujących zapytania do bazy w taki sposób jest poprawne pod względem optymalizacji?

[PHP] pobierz, plaintext 
$sql = "INSERT INTO contacts (email,telefon) VALUES ('$email','$telefon')";
$connection->query($sql);
 
 
$sql2 = "INSERT INTO adresses (ulica,numer_domu,numer_mieszkania,kod_pocztowy,miasto) VALUES ('$ulica','$numer_domu','$numer_mieszkania','$kod_pocztowy','$miasto')";
$connection->query($sql2);
 
[PHP] pobierz, plaintext 

Czy prędzej robi się to tak, żeby nie tworzyć miliona zmiennych?

[PHP] pobierz, plaintext 
$sql = "INSERT INTO contacts (email,telefon) VALUES ('$email','$telefon')";
$connection->query($sql);
 
 
$sql = "INSERT INTO adresses (ulica,numer_domu,numer_mieszkania,kod_pocztowy,miasto) VALUES ('$ulica','$numer_domu','$numer_mieszkania','$kod_pocztowy','$miasto')";
$connection->query($sql);
 
[PHP] pobierz, plaintext 

Zapewne nie robi się obu tylko PDO i prepared statements albo zewnętrzny ORM
Jeżeli z jakiegoś powodu potrzebujesz zmiennej jeszcze gdzieś to wiadomo że nie ma sensu jej nadpisywać więc tu nie ma reguły.
Twoja metoda jest też słaba bo musisz coś wcześniej pod zmienne podstawić. Już wolałbym sprintf dać do values(). Jest to bardziej czytelne i chociaż łatwo można rzutować typy czy dać mysqli::real_escape_string.

Z optymalizacją ma to niewiele wspólnego, zyskasz tyle co nic. Ja osobiście jednak stosuję nadpisywanie zmiennych (czyli przykład 2) ale z racji czytelności - raziłyby mnie w oczy zmienne $sql2, $sql3, ..., $sql123.