Projekt To do list w PHP

pytanie zadane 2 kwietnia 2016 w Nasze projekty przez Karol Wiczling Początkujący (430 p.)
zamknięte 4 kwietnia 2016 przez event15

Witam,

Chciałbym przedstawić wam mój projekt w PHP. Jest to prosta aplikacja w typu "to do list". Link do repo na githubie.

Aplikacja pozwala na dodawanie list zadań (wyświetlone są na stronie głównej index.php) oraz w ramach tych list (list.php) dodawać konkretne zadania do zrobienia

Można też:

usuwać listy (odbywa się po wejściu do listy)
zmienic nazwe listy (tak jak wyżej )
usuwać pojedyncze zadania lub kilka na raz
oraz oznaczać zadania jako zrobione/niezrobione także masowo

Użyłem tu ajax-a żeby wszystko było "nowocześniejsze" ale chyba więcej z tym zamieszania zrobiłem.

Jako serwera użyłem XAMPP-a.

Zdaję sobie sprawę ,że jest w tym projekcie masa błędów najróżniejszej natury. Dlatego właśnie umieszczam moją pracę pod waszą ocenę. Najbardziej interesują mnie informacje związane z PHP , trochę mniej JS.

Z góry dzięki za każdą odpowiedź.

komentarz zamknięcia: rozwiązane

5 odpowiedzi

odpowiedź 2 kwietnia 2016 przez efiku Szeryf (75,160 p.)
wybrane 4 kwietnia 2016 przez event15

odpowiedź 2 kwietnia 2016 przez Mateusz11 Pasjonat (22,910 p.)

$connection = @new mysqli($host, $db_user, $db_password, $db_name);

PDO!

require_once("../server/server.php");

Nawiasy nie są już potrzebne

komentarz 2 kwietnia 2016 przez jpacanowski VIP (101,940 p.)

A czemu nie mysqli() ?

2	komentarz 2 kwietnia 2016 przez efiku Szeryf (75,160 p.) Bo nie żyjemy w erze dinozaurów, temu.

komentarz 2 kwietnia 2016 przez Mateusz11 Pasjonat (22,910 p.)

@efik, w erze dinozaurów, używało się mysql, my nie jesteśmy w erze dinozaurów, tylko 100 lat przed egipcjanami ;)

komentarz 2 kwietnia 2016 przez jpacanowski VIP (101,940 p.)

A jakiś argument albo link do przekonywującego artykułu MySQLi vs. PDO ?

1	komentarz 2 kwietnia 2016 przez event15 Szeryf (93,790 p.) http://php.net/manual/en/mysqlinfo.api.choosing.php Dokumentacja prawdę ci powie.

komentarz 2 kwietnia 2016 przez jpacanowski VIP (101,940 p.)

It is recommended to use either the mysqli or PDO_MySQL extensions.

Tak jak myślałem... Argumentów brak. Sam używam PDO, ale nie lubię po prostu jak ktoś pisze, żeby używać tylko TO i koniec, kropka...

komentarz 2 kwietnia 2016 przez jpacanowski VIP (101,940 p.)

Nawiasy nie są już potrzebne

Nie są potrzebne, ale można używać... Więc w czym problem?

komentarz 2 kwietnia 2016 przez efiku Szeryf (75,160 p.)
edycja 2 kwietnia 2016 przez efiku

Tak jak myślałem... Argumentów brak. Sam używam PDO, ale nie lubię po prostu jak ktoś pisze, żeby używać tylko TO i koniec, kropka...

@jpacanowski, zapoznaj się może pierwsze jakie API dostarcza mysqli, a jakie PDO. Później sobie podyskutujemy, okej? Bo na razie widzę, że nie raczyłeś zajrzeć do manuala ;)

http://code.tutsplus.com/tutorials/pdo-vs-mysqli-which-should-you-use--net-24059 (dlatego mówimy PDO, nie mysqli^^, a no i czemu niby doctrine jest abstrakcją na PDO?)

Edit: Rzeczywiście, mówiłeś o mysqli, przeoczyłem. Tak więc przyda się ten obrazek:

EOT

komentarz 2 kwietnia 2016 przez jpacanowski VIP (101,940 p.)

Ale Ty pytałeś o MYSQL_*

Od początku pisałem o MySQLi (z "i" na końcu, od improved)! Więc nie rób mi żadnych screenów z uwagami, że MySQL_* jest deprecated - bo to oczywiste dla mnie.

Dzięki za artykuł, biorę się za czytanie.

EOT

komentarz 2 kwietnia 2016 przez efiku Szeryf (75,160 p.)

Jasne, przepraszam, zrobiłem edit.

Dodaj go do zakładek, jakby ktoś pisał kiedyś dlaczego zalecamy PDO. Na pewno się przyda... jeśli umie czytać ze zrozumieniem i myśli przyszłościowo o swojej aplikacji.

odpowiedź 2 kwietnia 2016 przez Boshi VIP (100,240 p.)

1 Małpa

2 brak PDO

3 powielanie kodu łączenia z bazą

4 brak chociaż by funkcji- choć lepiej obiektowo by to wyglądało

5 brak prepared station- masz sql injection

6 for zamiast foreach

7 mieszanie htmla z php

8 po co Request?

9 brak filtracji

to tak na szybko.

komentarz 2 kwietnia 2016 przez event15 Szeryf (93,790 p.)

A co mają funkcje do obiektowości?

A z jakiej paki for a nie foreach?

komentarz 2 kwietnia 2016 przez Boshi VIP (100,240 p.)

foreach jest stworzony do tablic..

obiektowo by to lepiej wygladało a ten kod to mieszanka kilku funkcji i braku funkcji. Bez sens. Jeżeli już funkcyjnie to wszystko prawie a nie raz tak raz nie.

komentarz 2 kwietnia 2016 przez event15 Szeryf (93,790 p.)

Dobra - nie skumałem :P

komentarz 2 kwietnia 2016 przez event15 Szeryf (93,790 p.)

W sensie nie skumałem, że radzisz foreach zamiast fora. Odwrotnie się wydało.

komentarz 2 kwietnia 2016 przez Boshi VIP (100,240 p.)

w ogóle nie rozumiem po co wyciągać ilość rekordów num_rows, skoro nie potrzebuje tego do nieczego innego jak ilości obiegu pętli to fetchall od razu i foreach. Chyba, że mysqli nie ma fetchall to lipa :p

odpowiedź 2 kwietnia 2016 przez event15 Szeryf (93,790 p.)

https://github.com/kreex/tld/blob/master/server/addlist.php#L6

Jestem pewien z której książki się uczyłeś takiego czegoś. Tego się nie robi.

https://github.com/kreex/tld/blob/master/server/addlist.php#L8-L11

Brak jakiejkolwiek filtracji danych.

https://github.com/kreex/tld/blob/master/server/addlist.php#L23-L48

To mi trudno skomentować bo pierwszy raz od bardzo dawna się z czymś takim spotkałem. Generalnie nie mieszamy PHP i innych języków. Używa się PDO, lub Doctrine2.

https://github.com/kreex/tld/blob/master/server/addtask.php

Dokładnie te same uwagi. Dochodzi jeszcze to, że w wielu miejscach uruchamiasz connect do bazy danych. Tak się nie robi. Powinieneś zadbać o to żeby w jednym miejscu uruchamiać jedną instancję bazy danych i ją zamykać. W innym wypadku strona padnie przy większym ruchu.

PS. Znów nie filtrujesz zmiennnych.

odpowiedź 2 kwietnia 2016 przez maly Nałogowiec (37,190 p.)

nagłówki dla kolumn w tabelach to <th>.
<br> służy wyłącznie do łamania tekstu.
<center> wyleciał ze standardu już dawno temu.
<h1> <b>TO DO LIST</b> </h1> A cóż to za kwiatek?

co do <b> to już sam niewiem do czego jest przeznaczony.

1	komentarz 3 kwietnia 2016 przez Comandeer Guru (601,590 p.) b jest przeznaczony do wizualnego wyróżniania pewnych fragmentów tekstu, ale bez zaznaczania ich ważności. Służy np do podkreślenia tytułu omawianego dzieła w recenzji.

komentarz 4 kwietnia 2016 przez maly Nałogowiec (37,190 p.)

Czyli tak jak myślałem. Moje wątpliwości spowodowane są tym że kilka osób twierdziło że powinienem raczej użyć <strong>, jednak frazy które oznaczałem nie mają jakiegoś szczególnego znaczenia, ot takie podświetlenie znalezionych wyrazów.