Problem z "duplikacją" danych na stronie www

Question

Dzień dobry

Ostatnio aplikowałem do pewnej firmy tworzącej strony internetowe na praktyki studenckie. Na rozmowach kwalifikacyjnych otrzymałem zadanie stworzenia strony, która będzie zawierać strukturę "drzewa". Po stworzeniu takowej strony, odesłałem ją do (mam nadzieję) przyszłego "praktykodawcy" i otrzymałem odpowiedź zwrotną, że na stronie znajduję się błąd. I w rzeczy samej, strona (a raczej struktura drzewa) została wysypana poprzez wprowadzenie duplikatu wartości (np. wprowadzenie drugiego obiektu o wartości 3 na tym samym "poziomie"). 

Niestety nie mam pojęcia jak osoba sprawdzająca tą stronę tego dokonała, przez co nie wiem gdzie i co muszę poprawić w kodzie.

Prosiłbym Was o przetestowanie strony i wgląd do kodu. Może Wy coś znajdziecie.

 

Oto link do strony: http://piotr.bobusia.niedoszkolenie.pl/

Kod strony na GitHubie: https://github.com/piotrek490/Praktyki-Zadanie.git

Pliki odpowiedzialne za dodawanie wartości do drzewa są w folderze Hash2/Ajax

 

Z góry bardzo dziękuję za wszelką pomoc.

 

Answer 1

I w rzeczy samej, strona (a raczej struktura drzewa) została wysypana poprzez wprowadzenie duplikatu wartości (np. wprowadzenie drugiego obiektu o wartości 3 na tym samym "poziomie"). 

Zakładam że przez "została wysypana" masz na myśli "nie dodaje się do drzewa".

Może czegoś nie widzę, ale na moje oko... explicite obsługujesz ten przypadek i na to nie pozwalasz, nie?

// Sprawdzam czy jest już w tabeli taka wartość
$zapytanie="select * from ".$_GET['key']." where val=".$_GET['value'].";";
$wynik = mysqli_query($polaczenie, $zapytanie);
$switch = mysqli_fetch_array($wynik, MYSQLI_ASSOC);
if($switch) exit(1); //Jeśli jest już taka wartość -> "wyłączam" ten plik

A boczne uwagi, tak szczerze, gdybym był rekruterem to raczej byłbym przerażony. Dynamiczne tworzenie tabel per ojciec drzewa; surowe stringowe sklejanie zapytań SQLowych; ręcznie napisany kod sortujący; obsługa ActiveXObject, jakby to miało chodzić na IE6; synchroniczne zapytania XHR (na które sama przeglądarka krzyczy w konsoli)... D:

Comments

Dziękuję za uwagi :) Co do obsłużonego przypadku jednak jakoś jest on jakoś pomijany przez Pana sprawdzającego. NA chwilę obecną zauważyłem, że dodanie wartości NULL do MainTable i usunięcie wszystkich rekordów skutkuję właśnie "wysypaniem" drzewa.

Przez "wysypanie" mam na myśli sytuację, w której obiekty nie są wyświetlane (mimo tego że istnieją).

---

Pewnie dodali kod do strony zdążając ją przekazać przed zakończeniem sesji. Niektóre biblioteki mają mechanizmy reagowania na takie sytuacje, czasem trzeba samemu to zrobić.

---

(z tego co widzę, tu w ogóle nie ma sesji.)