[PHP][MySQL] Sprawdzenie skryptu Opcje

[koniu90]

Witam. Jestem totalnym nowicjuszem w tej dziedzinie i dopiero się uczę, także proszę o wyrozumiałość.
Prosiłbym o sprawdzenie kodu którego napisałem. Skrypt ma instalować bazę na serwerze i dodać tabelę:

[PHP] pobierz, plaintext 
$lacz = mysql_connect( $serwer, $login, $haslo );
$zapytanie = 'CREATE DATABASE $nazwabazy';
 
$wykonaj = mysql_query ($zapytanie);
 
if($wykonaj==true)
{
	echo 'Baza została utworzona';
}
else
{
	echo 'Baza nie została utworzona';
}
	$lacz = mysql_select_db ( 'WirtualnyDziekanat' );
 
	$zapytanie = 'CREATE TABLE Users (
    user_id INT () not null AUTO_INCREMENT,
	login VARCHAR (11) not null,
    password VARCHAR (30) not null,
	forname VARCHAR(20) not null,
	surname VARCHAR(40) not null,
	group_id INT() not null,
	pesel VARCHAR(11) not null,
    PRIMARY KEY (user_id))';
 
	$wykonaj = mysql_query ($zapytanie);
	if($wykonaj==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
[PHP] pobierz, plaintext 

Ten post edytował Kshyhoo 18.03.2014, 21:10:43

Powód edycji: [Kshyhoo]: bb-code

[mmmmmmm]

1. Nie zastosowałeś się do moich rad powyżej (z post w tym wątku)
2. Wszędzie powinieneś mieć IF NOT EXISTS. Np. CREATE DATABASE IF NOT EXISTS ... Bo wcale nie obsługujesz błędów...Co to jest za obsługa, że napiszesz "Baza nie została utworzona"... To nic, że nie została utworzona - lecisz dalej... Może się uda stworzyć tabele... Bez sensu.
3. Z tego samego powodu całość powinieneś robić w transakcji.
4. Brak powiązań pomiędzy tabelami kluczami obcymi.
5. Jeśli chciałbyś je dodać, to w złej kolejności tworzysz tabele... Tabela `Group` powinna być tworzona przed tabelą `Users`.
6. Robisz:
$zapytanie = 'CREATE DATABASE $Dziekanat';
a potem:
$lacz = mysql_select_db ( 'WirtualnyDziekanat' );
Zdecyduj się - czy używasz zmiennej, czy stałej...