Forum PHP.pl > [www] Strona szkoły

Pomoc - Szukaj - Użytkownicy - Kalendarz

WebSee

1.07.2010, 20:44:09

Witam,

proszę o ocenę wykonanej przeze mnie strony szkoły: http://group.websee.com.pl/

Pozdrawiam

l0ud

1.07.2010, 20:54:24

Strona dość monotonna i prosta graficznie ale poprawna. Jak na większość stron szkolnych to OK. Główne zastrzeżenie mam takie, że zdjęcia powinny się powiększać. No i nie widać nigdzie adresu do szkoły ani kontaktu - a to powinno być przecież wyeksponowane.

erix

1.07.2010, 21:18:07

A jak pod względem kodu:

[HTML] pobierz, plaintext 
<body onload="viewphotos();">
[HTML] pobierz, plaintext

- WTF?

[HTML] pobierz, plaintext 
<div id="header">
					<a href="./"><img src="themes/default_images/logo.png" id="logo" alt="Zespół szkół technicznych i ogólnokształcących im. Jana Pawła II w Limanowej" /></a>
					<img src="media/photos/right_1.png" id="photo-right" alt="" />
					<img src="media/photos/right_2.png" id="photo-none" alt="" />
					<img src="media/photos/right_3.png" id="photo-none" alt="" />
				</div>
[HTML] pobierz, plaintext

-> primo: po co IMG tutaj, jak można dać jako tło, secundo: zapisywanie zdjęć w PNG, to raczej zły pomysł

SHiP

1.07.2010, 21:52:09

Nie chcę wyjść na złego hakira z małej wioski ale usunąłem Ci newsa o id=35

Zrzut backupa z panelu admina

Cytat

<div id="p-cont">
<h1>Pomysł na biznes w powiecie limanowskim - 14 stycznia 2010 <small class="edinfo">[ <a href="news.php?go=edit&id=35" title="Edytuj newsa">Edytuj</a> - <a href="news.php?go=delete&id=35">Usuń</a> ]</small></h1>
Byliśmy gospodarzami uroczystości wręczenia nagród laureatom V Konkursu "Pomysł
na biznes w powiecie limanowskim" organizowanego przez Powiat Limanowski oraz
Radę Przedsiebiorczości Powiatu Limanowskiego. W konkursie wzięły udział 43 projekty.
Wśród czołówki młodych biznesowych talentów znaleźli sie również nasi uczniowie z klasy 3TE
- Michał Śliwa i Krystian Tokarz - zajęli III miejsce za projekt za projekt "GAB - LIM".
Gratulujemy!
<br /> <br />

<div style="text-align: center; "><img hspace="10" align="center" src="http://zstio.edu.pl/img/2009_10/biznes1.jpg" /> <img hspace="10" align="center" src="http://zstio.edu.pl/img/2009_10/biznes2.jpg" />
</div>
</div>

Poczytaj o atakach CSRF ponieważ linki typu news.php?go=delete&id=35 są kretyńskie ponieważ nie wymagają żadnego potwierdzenia. Co więcej aby taki link zadziałał nie potrzeba nawet uprawnień administratora. Jak zobaczyłem, że to działało bez uprawnień admina to aż pomyślałem WFT?

nmts

1.07.2010, 22:27:34

Cytat(SHiP @ 1.07.2010, 22:52:09 )

Poczytaj o atakach CSRF ponieważ linki typu news.php?go=delete&id=35 są kretyńskie ponieważ nie wymagają żadnego potwierdzenia.

Głównym problemem jest to, że jedyną ochroną panelu jest javascript, bez tego byś musiał zgadywać, co prawda nie trudne zmienne ale zawsze. ^^

Kildyt

1.07.2010, 22:27:57

Co do designu nie ma co się rozpisywać: szału nie ma, ale wielkiego wstydu też nie. Zgadzam się z @l0ud.

Co do kodu mam kilka uwag:
- niepoprawne użycie nagłówków (kilka razy h1 itp.),
- puste alt-y,
- po co jako przedrostek do nazwy plików dodawać "zobacz-"?
- znaczniki hspace i align w IMG nie są już używane,
- nie widzę sensu wstawiania linka do panelu admina w stopce - kto będzie chciał się tam dostać dostanie się, a tak tylko niepotrzebnie kusisz hakerów.

Jak już wspomniał @l0ud nigdzie nie widzę kontaktu, a to podstawa, szczególnie na tego typu stronach.

WebSee

2.07.2010, 09:44:32

Dziękuję za informacje i sugestie

Strona była pisana jakiś czas temu, pewnie stąd tyle błędów i dziur.

Pozdrawiam

Cypherq

2.07.2010, 13:13:32

Świetne wytłumaczenie. Już sobie wyobrażam taką sytuację:

- Spójrzcie, fajne buty mam?
- No, niezłe, wygląd poprawny, kolor w miarę.
- Tylko sznurówki Ci się prują.
- I masz dziurę z przodu.
- W ogóle to z tyłu odczepia Ci się także podeszwa.
- ...
- Eeee... buty kupione jakiś czas temu stąd tyle dziur.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.