Forum PHP.pl

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

Ok dzięki za wskazówki... poprawię

Ogólnie skrypt się powinien wykonać?

Jeszcze jedno pytanie:
w bazie będzie kilka tabel czy każdą którą tworzę muszę robić w osobna w znaczniku <?php .... ?> czy można wszystko ze sobą połączyć

ktos odpowie (IMG:style_emoticons/default/questionmark.gif)

Jeśli wprowadziłeś poprawki: tak, wykona się.

Możesz nawet wszystkie wsadzić w jeden string i zainstalować, a osobno tylko kopiujesz kod.

dzięki za pomoc (IMG:style_emoticons/default/smile.gif)

Przenoszę do Przedszkola.

Zainstaluj coś pokroju "MySQL Workbench", stwórz tabelę i spróbuj wyeksportować. Zobaczysz jak wyglądają poprawne zapytania do bazy danych tworzące tąże bazę. Twój kod uruchomiony 2x poinformuje że nie udało mu się w zasadzie nic co miał zrobić.

Wskazówki
- Sprawdzaj czy SCHEMA bazy danych już istnieje,
- Sprawdzaj czy istnieje tabela którą chcesz utworzyć,
- mysql_* jest oznaczone jako przestażałe używaj mysqli lub \PDO,

[PHP] pobierz, plaintext 
$lacz = mysql_connect( $serwer, $login, $haslo );
$zapytanie = 'CREATE DATABASE $Dziekanat';
 
$wykonaj = mysql_query ($zapytanie);
 
if($wykonaj==true)
{
	echo 'Baza została utworzona';
}
else
{
	echo 'Baza nie została utworzona';
}
?>
 
<?php
 
	$lacz = mysql_select_db ( 'WirtualnyDziekanat' );
 
	$zapytanie1 = 'CREATE TABLE Users (
	user_id INT () not null AUTO_INCREMENT,
	login VARCHAR (11) not null,
	password VARCHAR (30) not null,
	forname CHAR(20) not null,
	surname CHAR(40) not null,
	group_id INT() not null,
	pesel VARCHAR(11) not null,
	PRIMARY KEY (user_id))';
 
	$wykonaj1 = mysql_query ($zapytanie1);
	if($wykonaj1==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
	$zapytanie2 = 'CREATE TABLE Group (
	group_id INT() not null AUTO_INCREMENT,
	name CHAR(20) not null,
	PRIMARY KEY (group_id)';
 
	$wykonaj2 = mysql_query ($zapytanie2);
	if($wykonaj2==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
	$zapytanie3 = 'CREATE TABLE GroupDeanery (
	group_deanery_id INT() not null AUTO_INCREMENT,
	user_id INT() not null,
	PRIMARY KEY (group_deanery_id)';
 
	$wykonaj3 = mysql_query ($zapytanie3);
	if($wykonaj3==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie4 = 'CREATE TABLE GroupAdmin (
	group_admin_id INT() not null AUTO_INCREMENT,
	user_id INT() not null,
	PRIMARY KEY (group_admin_id))';
 
	$wykonaj4 = mysql_query ($zapytanie4);
	if($wykonaj4==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie5 = 'CREATE TABLE StudentSubject (
	student_subject_id INT() not null AUTO_INCREMENT,
	subject_semestr_id INT() not null,
	group_student_id INT() not null,
	mark_1 INT() null,
	mark_2 INT() null,
	PRIMARY KEY (student_subject_id))';
 
	$wykonaj5 = mysql_query ($zapytanie5);
	if($wykonaj5==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
	$zapytanie6 = 'CREATE TABLE GroupStudent (
	group_student_id INT() not null AUTO_INCREMENT,
	user_id INT() not null,
	index_no CHAR(10) not null,
	semestr_no INT() not null,
	activity_status_id INT() not null,
	PRIMARY KEY (group_student_id))';
 
	$wykonaj6 = mysql_query ($zapytanie6);
	if($wykonaj6==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
	$zapytanie7 = 'CREATE TABLE ActivityStatus (
	activity_status_id INT() not null AUTO_INCREMENT,
	status_name CHAR(20) not null,
	PRIMARY KEY (activity_status_id))';
 
	$wykonaj7 = mysql_query ($zapytanie7);
	if($wykonaj7==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie8 = 'CREATE TABLE SubjectSemestr (
	subject_semestr_id INT() not null AUTO_INCREMENT,
	subject_id INT() not null,
	semestr_no INT() not null,
	year INT() not null,
	PRIMARY KEY (subject_semestr_id))';
 
	$wykonaj8 = mysql_query ($zapytanie8);
	if($wykonaj8==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie9 = 'CREATE TABLE Subject (
	subject_id INT() not null AUTO_INCREMENT,
	name NCHAR(10) not null,
	subject_type_id INT() not null,
	hours_number INT() not null,
	ects INT() not null,
	PRIMARY KEY (subject_id))';
 
	$wykonaj9 = mysql_query ($zapytanie9);
	if($wykonaj9==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie10 = 'CREATE TABLE SubjectType (
	subject_type_id INT() not null AUTO_INCREMENT,
	type_name CHAR(20) not null,
	PRIMARY KEY (subject_type_id))';
 
	$wykonaj10 = mysql_query ($zapytanie10);
	if($wykonaj10==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie11 = 'CREATE TABLE LecturerSubject (
	lecturer_subject_id INT() not null AUTO_INCREMENT,
	subject_semestr_id INT() null,
	group_lecturer_id INT() not null,
	PRIMARY KEY (lecturer_subject_id))';
 
	$wykonaj11 = mysql_query ($zapytanie11);
	if($wykonaj11==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie12 = 'CREATE TABLE GroupLecturer (
	group_lecturer_id INT() not null AUTO_INCREMENT,
	user_id INT() not null,
	user_title_id INT() not null,
	PRIMARY KEY (group_lecturer_id))';
 
	$wykonaj12 = mysql_query ($zapytanie12);
	if($wykonaj12==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie13 = 'CREATE TABLE UserTitle (
	user_title_id INT() not null AUTO_INCREMENT,
	user_id INT() not null,
	title_id INT() not null,
	PRIMARY KEY (user_title_id))';
 
	$wykonaj13 = mysql_query ($zapytanie13);
	if($wykonaj13==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
 
 
	$zapytanie14 = 'CREATE TABLE Title (
	title_id INT() not null AUTO_INCREMENT,
	full_name CHAR(20) not null,
	title CHAR(10) not null,
	PRIMARY KEY (title_id))';
 
	$wykonaj14 = mysql_query ($zapytanie14);
	if($wykonaj14==true)
	{
		echo 'Tabela została utworzona';
	}
	else
	{
		echo 'Tabela nie została utworzona';
	}
?>
[PHP] pobierz, plaintext 

tworzy 14 tabele tylko nie wiem czy kazda ma miec oddzielne zapytanie?

po pierwsze napisz sobie do tego funkcje do której tylko wrzucisz stringa z strukturą bazy ,bo pewnie koło obiektówki nie stałeś, ale powielanie tego samego kodu 14 razy jest bez sensu (IMG:style_emoticons/default/smile.gif) A poza tym to musisz napisać ten skrypt tak, że tylko raz go wywołasz, bo przecież tabele tworzysz tylko raz w czasie działania, potem tylko ja uzupełniasz danymi (IMG:style_emoticons/default/wink.gif)

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...