Forum PHP.pl

Mam taki kod:

[PHP] pobierz, plaintext 
$nr_rejestracji=$_POST['nr_rejestracji'];
$typ=$_POST['typ'];
$data_dodania=$_POST['data_dodania'];
$przebieg_przy_dodaniu=$_POST['przebieg_przy_dodaniu'];
 
mysql_connect("localhost",$username,$password);
@mysql_select_db($database) or die("Nie znaleziono bazy danych");
 
 
$query= "CREATE TABLE $nr_rejestracji (id int(6) NOT NULL auto_increment, opis_naprawy varchar(300) NOT NULL, cena_naprawy varchar(30) NOT NULL, miejsce_wykonania varchar(30) NOT NULL, data_wykonania varchar(15) NOT NULL, przebieg_przy_naprawie varchar(20) NOT NULL,PRIMARY KEY (id))";
mysql_query($query);
[PHP] pobierz, plaintext 

Problem tkwi w tym, że kiedy w "CREATE TABLE $nr_rejestracji" jest zmienna skrypt nie tworzy nowej tabeli w mojej bazie danych. Lecz kiedy $nr_rejestracji zmienię w stałą np. nr_rejestracji (bez znaku dolara), skrypt tworzy nową tabele.

Jestem początkujący w php jak i mysql, więc proszę o wyrozumiałość w moich pewnie podstawowych błędach.

Ten post edytował przemo191 1.08.2011, 10:53:09

Zwraca:

[PHP] pobierz, plaintext 
string 'FZ 2671' (length=7)
[PHP] pobierz, plaintext 

Nowa tabela na nowy numer rejestracji jest mi potrzebny, ale dużo tych rejestracji nie będzie, więc nie przeszkadza mi to.

/edit

Ponieważ później do każdej tabeli w inna rejestracją bedę musiał wpisywać sporo rekordów.

Ten post edytował przemo191 1.08.2011, 10:57:58

Tworzenie tabeli dla każdego pojazdu to bardzo zły pomysł. Powinieneś utworzyć tabelę pojazdy, do której dodajesz kolejne rekordy.
Poza tym nigdy nie używaj @ w kodzie. Błędów się nie ukrywa, błędy się eliminuje lub im zapobiega. W dodatku wiele mówią i gwarantuję ci, że błędy to nie jest czyste zło, wręcz odwrotnie.
Jeżeli jednak upierasz się na tym tworzeniu tabeli to:
- dodaj do kodu, przed wywołaniem mysql_query(), var_dump($_POST) i napisz co ci wyświetla.
- dostajesz jakieś błędy lub cokolwiek takiego, czy zero reakcji?

@up trochę się spóźniłem:)
W nazwie tabeli nie możesz mieć spacji, zamień ją np. na _

Ten post edytował bastard13 1.08.2011, 10:59:19

Zwraca:

[PHP] pobierz, plaintext 
array
  'nr_rejestracji' => string 'FZ 2671' (length=7)
  'typ' => string 'ciągnik siodłowy' (length=18)
  'data_dodania' => string '1.08.2011r.' (length=11)
  'przebieg_przy_dodaniu' => string '182198km' (length=8)
[PHP] pobierz, plaintext 

Co do błędów, to zero.



Niestety nie działa ;(

FZ 2671 zmieniłem na FZ_2671, niestety dalej nic.

Znalazłem chochlika, zamiast `$nr_rejestracji` wpisałem $nr_rejestracji`. Dzięki za tak błyskawiczną pomoc, oczywiście + będzie.

Być może, ale człowiek uczy się na błędach, jak się pogubię to wymyślę nową strukture (IMG:style_emoticons/default/wink.gif)
Dzięki za podpowiedzi zaraz to pozmieniam.

Człowiek uczy się też na wskazówkach. Skoro ktoś bardziej doświadczony ci doradza to lepiej posłuchaj już teraz. No ale Twoja sprawa (IMG:style_emoticons/default/smile.gif)

Nie zaprzeczę, bo jest to prawdą. A ty jaką strukturę byś proponował?

Musiałbym poznać większy obraz sytuacji. Opisz co to będzie, co tam będzie. Pisałeś coś o dużej liczbe rekordów dla danej rejestracji

Ja widzę to miej więcej tak, muszą być opcje:
1. dodawania nowych samochodów do bazy,
2. do każdego samochodu trzeba będzie pododawać 5 różnych danych (opis naprawy, cena naprawy, miejsce wykonania, data wykonania, oraz przebieg samochodu przy naprawie)
3. aktualizacja tych 5 różnych pól z danymi w razie błędnego wpisania,
3. usuwanie rekordów z wpisami,
4. zliczanie kosztów wszystkich napraw dla każdego samochodu z osobna
5. zliczanie kosztów wszystkich napraw przy wszystkich samochodach

Wszystkie dane będą wprowadzane z odpowiedniego formularza.

Ten post edytował przemo191 1.08.2011, 11:44:16

tabela SAMOCHOD
ID
REJESTRACJA
inne dane dla samochodu

tabela NAPRAWA
ID
ID_SAMOCHODU - klucz obcy z tabeli SAMOCHOD wskazujący na pole ID
OPIS
CENA
MIEJSCE
DATA

I już. Na takiej strukturze zrealizujesz wszystko bez problemu. Na tym co ty wymyśliłeś, miałbyś już duże problemy.

I to jest chyba najlepsze rozwiązanie, naprawdę dzięki Ci za pomoc.



Jeszcze raz ja, zrobiłem wszystko co chciałem ale nie mogę zrobić tego co napisałeś. Po prostu nie wiem jak ;( szukałem w google, ale nic ciekawego mi nie pokazał.

Utworzyłeś pole o nazwie ID_SAMOCHODU?

Jeśli nie to utwórz i przejdź do punktu niżej

Jeśli tak to:
- wersja LITE:
i już masz to co napisałem
- wersja HARDCOR dla super hiper wyjadaczy*:
Jeśli chcesz mieć super hiper baze, to wypadałoby zrobić ją typu InnoDB i zakładać relacje między tabelami, by baza sama dbała o porządek.
Np. jak skasujesz samochód, to wszystkie naprawy dla tego samochodu się skasują z automatu. Albo np. na odwrót, że jeśli masz jakieś naprawy dla samochodu X to baza nie pozwili ci usunąć samochodu X.

* bez przesady z tymi super hiper wyjadaczami, ale jeśli dopiero zaczynasz, to można narazie olać etap super hiper kluczy obcych (IMG:style_emoticons/default/wink.gif)

Może być wersja lite (IMG:style_emoticons/default/happy.gif)
pole id_samochodu stworzyłem już na początku, ale wartość tego pola wynosi 0, a powinno odpowiednio do id w tabeli samochody, czy jak?