![]() |
![]() ![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 21 Pomógł: 0 Dołączył: 1.01.2016 Ostrzeżenie: (0%) ![]() ![]() |
Pytanie: czy zmierzam w dobrym kierunku ? Podam tylko najważniejsze pliki.
Jest to aplikacja czatu. Za krytykę dziękuję - to się przyda. Plik czat.php
Plik BazaDanych.php - singleton
Plik javascript odebranieWiadomosciZSerwera.js $(document).ready(function(){ //odbieranie wiadomośći z serwera co określony czas function odebranieWiadomosciZSerwera() { $.ajax({ url: "./../czat/php/czat.php", method: "POST", data: { odbierzWiadomosc: true, idWiadomosci: $("#wiadomosci p.id").last().attr('id'), }, }).done(function(odpowiedz){ $("#wiadomosci").append(odpowiedz); $('#wiadomosci').animate({ scrollTop: $('#wiadomosci')[0].scrollHeight}, "slow"); }); }; setInterval(odebranieWiadomosciZSerwera,1500); }); Plik javascript wysylanieWiadomosciNaSerwer.js $(document).ready(function(){ //wysyłanie wiadomości na serwer po kliknięciu buttona $("#wyslijWiadomosc").click(function(){ $.ajax({ url: "./../czat/php/czat.php", method: "POST", data: { zapiszWiadomosc: true, trescWiadomosci: $("#trescWiadomosci").val(), }, }).done(function(odpowiedz){ }); $("#trescWiadomosci").val(""); }); }); Liczę na konstruktywną krytyke Dziękuję i pozdrawiam |
|
|
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 6 365 Pomógł: 1114 Dołączył: 30.08.2006 Ostrzeżenie: (0%) ![]() ![]() |
A to rok 2010 mamy ? Brak composer, brak DI, brak PSR, podatne na sql injection, jakiś miks html i php.
-------------------- |
|
|
![]()
Post
#3
|
|
![]() Grupa: Moderatorzy Postów: 36 455 Pomógł: 6292 Dołączył: 27.12.2004 ![]() |
Naduzywasz tworzenie prywatnych wlasiciwosci klasy. Uzywasz ich jako zmiennej lokalne a i tak robisz z nich wlasciwosci klasy. TO nie ma zadnego sensu. Uzywajac klas nie znaczy, ze nie mozesz uzywac zmiennych lokalnych
ID wiadomosci to liczba wiec traktuj to jako liczbe anie jako tekst. No i to co juz napisano: takie kody pisalo sie w sredniowieczu ![]() -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
![]()
Post
#4
|
|
Grupa: Zarejestrowani Postów: 44 Pomógł: 5 Dołączył: 20.05.2019 Ostrzeżenie: (0%) ![]() ![]() |
Faktycznie ten kod wygląda jak z jakiegoś poradnika Pana Zenka z przed ~20 lat.
Najlepiej zainteresuj się jakimś frameworkiem PHP'a, szybciej zrozumiesz jak to powinno wyglądać i zostaw jQuery (szkoda na to czasu), jest masa dobrych frameworków JS (vue, react, angular), które robią robotę czyściej. |
|
|
![]() ![]() |
![]() |
Wersja Lo-Fi | Aktualny czas: 24.04.2024 - 09:25 |