• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

question-closed Projekt To do list w PHP

VPS Starter Arubacloud
+1 głos
900 wizyt
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

+4 głosów
odpowiedź 2 kwietnia 2016 przez efiku Szeryf (75,160 p.)
wybrane 4 kwietnia 2016 przez event15
 
Najlepsza
Baw się dobrze.

https://github.com/kreex/tld/issues/1

Pozdrawiam.
+1 głos
odpowiedź 2 kwietnia 2016 przez Mateusz11 Pasjonat (22,910 p.)

1.

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

PDO!

2.

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

2
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.
+1 głos
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
0 głosów
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.
0 głosów
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 (605,680 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.

Podobne pytania

0 głosów
0 odpowiedzi 92 wizyt
pytanie zadane 24 sierpnia 2016 w PHP przez Alterwar Dyskutant (7,650 p.)
+4 głosów
3 odpowiedzi 650 wizyt
pytanie zadane 8 maja 2018 w Nasze projekty przez Danrox Bywalec (2,370 p.)
0 głosów
1 odpowiedź 385 wizyt

93,079 zapytań

142,043 odpowiedzi

321,452 komentarzy

62,424 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj

Wprowadzenie do ITsec, tom 2

Można już zamawiać tom 2 książki "Wprowadzenie do bezpieczeństwa IT" - będzie to około 650 stron wiedzy o ITsec (17 rozdziałów, 14 autorów, kolorowy druk).

Planowana premiera: 30.09.2024, zaś planowana wysyłka nastąpi w drugim tygodniu października 2024.

Warto preorderować, tym bardziej, iż mamy dla Was kod: pasja (użyjcie go w koszyku), dzięki któremu uzyskamy dodatkowe 15% zniżki! Dziękujemy zaprzyjaźnionej ekipie Sekuraka za kod dla naszej Społeczności!

...