Dzięki tej stronie możecie wyszukiwać, odsłuchiwać oraz ściągać utwory w formacie mp3. Byłbym wdzięczny za oceny i co można ewentualnie poprawić.

Link: Ściągus.pl

"Większości" powiadasz... a więc nie wszystkie, czyli to zwykły warez. Do tego brzydki w wyglądzie

// add

I do tego kiepsko zabezpieczony. Mogę se wywoływać błędy.

Noi pomysł mało oryginalny.. w necie jest mnóstwo takich wyszukiwarek

Niewiele musiałem się natrudzić, aby rozłożyć stronę na łopatki

moglbys mi podpowiedziec jak zabezpieczyc sie przed tym? po drugie to nie warez, pliki nie sa przechowywane na moim serwerze oraz ja ich nie uploaduje. czytalem porady prawne

Co do XSS'a wystarczy używać htmlspecialchars() przy wyświetlaniu danych. Dodatkowo przyda się kontrola długości danych, bo jak dawałem jakiś długi string to wywoływał się error.

moglbys cos wiecej napisac w jakiej postaci to wstawic?

Zastosować tak jak każdą inną funkcję php, zajrzyj TUTAJ

nadal nic nie rozumiem bo jestem troche zielony w tym :/

To zapraszamy do przedszkola, a nie prowadzisz taką dyskusję w tym dziale.

Mi się wydaje, że jeżeli pośredniczysz w ściąganiu nielegalnych plików to także popełniłeś przestępstwo, ale ręki uciąć sobie nie dam.
Popieram ~pyro: to już było, kiepskie. Wyszukiwarka jest niezauważalna. Strona jest nieprzejrzysta (szczególnie informacje o utworze).

Co do kodu:
- za dużo stylów w html-u,
- brak alt-ów w img-ach,
- unikach pustych div-ów,
- masz h2, a nie ma h1.

Nie jest to nielegalne, ponieważ czytałem na forum prawnym. Link a co do tej strony postaram sie poduczyc php w lepszym stopniu bo tutaj jedynie chciałem sobie poćwiczyć. Wszystkim dziękuję za wszystkie uwagi, z pewnością pomoże to w przyszłości.7
probowalem takze uzyc tego: lecz przy probie wyszukiwania wywala mi taki blad:
ogolnie trudno jest cos edytowac, poniewaz jest to gotowy cms i trudno sie polapac.

Dałeś o jeden nawias za dużo, tam gdzie są dwa nawiasy

// edit



Nawet jak to jest legalne, to nieetyczne.

ok usuniety nawiad



teraz cos takiego

Ekhm, to nie Przedszkole/PHP...

ale, pomoc tutaj tez mozna. btw. wszystkie wyniki wyszukiwania zapisuja sie w sql. jezeli ktos bylby chetny mi pomoc to moge udostepnic kod strony w celu znalezienia dziury

No ale znajomość angielskiego to wiesz. Pomijam to.

Wygląd. Ujdzie jak poprawisz.
- Pole wyszukiwania mi ginie czyli daj na niej jakieś logiczne tło - choćby białe,
- Wyłącz pokazywanie błędów jak coś publikujesz,
- Przycisk Wyszukaj wkomponuj jakoś obok pola wyszukiwania,
- masz style w zewnętrznym css, a co tu robi znacznik style w html'u?,
- odtwarzacz gotowy, muzyka z wrzuty... no a może jakaś wzmianka o tym - mimo, że wszyscy korzystają nikt nie napisze.

Ogólnie strona rozlana. Stopka od lewej do prawej. Logo dosyć duże i byłoby fajne gdybyś go nie uciął tylko dał np zanikające przenikanie.

Zaraz zaraz a dlaczego pod linkami do piosenek tło mi jeździ w tą i z powrotem? Hę?

A w kwestii tego błędu to chyba foreach po tablicy i htmlspecialchars po stringach.

grafika niedlugo bedzie zmieniona bo jak na razie jest ona tylko tymczasowa. co do tego tla pod wyszukiwarka takze da sie zrobic, jednak nadal wystepuje blad gdy w pole wyszukiwanie wpisuje sie kod html

A jeszcze jedno - uduszę Cie!
Nie no żart, ale ściąganie piosenki to daj lepiej w osobne okno - bo jak ktoś Sobie przerwie piosenkę i nie może potem wrócić do przerwanego fragmentu. Oj źli będą ludzie.

jezdzace tlo pod piosenkami juz zmienione, z tym uduszeniem juz tez

edit: raportowanie bledow tez, znacznik style uzylem dlatego, ze cala strona nie chciala sie pokryc tlem, byc moze jakis blad z mojej strony

Ale czy jest to w jakiś sposób powiązane z oceną skryptu?

wyniklo to z tego, iz pyro mi pokazal gdzie jest dziura tylko nadal nie wiem jak ja naprawic. radzilem sie kilku osob i nadal nic.

Czym się kierowałeś robiąc tak wielki top, ograniczając jednocześnie widoczność pola na wyszukiwanie ? Widziałeś kiedyś jak wygląda najpopularniejsza wyszukiwarka świata (gogle, gugle czy coś takiego) ? Pole wyszukiwania słabo widoczne nawet jak wiesz gdzie jest, daj jakieś tło np. białe. Tak samo nie aktywne linki w stronicowaniu. Kod to tragedia, przykład:


ps. Też mam taki skrypt pobierający pliki z wrzuty, kupiłem na allegro kiedyś za parę złotych No oprócz mnie jest pełno takich stron więc wielki minus za to

tak jak wczesniej pisalem, caly wyglad jest tymczasowy. po drugie


uzylem dlatego, poniewaz uzywajac normalnie body cala strona nie chciala sie pokryc tlem. nie chcialem juz glebiej grzebac w kodzie i zrobilem tylko to. w najblizszym czasie bedzie calkowita zmiana wygladu i zapewniam ze powinna byc milsza dla oka. po trzecie u mnie top wyglada dobrze na rozdzialce 1440x900 lecz nie pomyslalem o osobach korzystajacych z mniejszych rozdzielczosci. dziekuje za wszelkie uwagi. z pewnoscia pomoga one rozwiazac wiele dotychczasowych bledow.