Forum PHP.pl > [skrypt] Galeria w php

Pomoc - Szukaj - Użytkownicy - Kalendarz

number0

14.09.2004, 14:34:35

Witam

proszę o ocenę mojego skryptu Galerii
skrypt umozliwaia przegladanie obrazków *.jpg znajdujacych się w bierząceym katalogu (tam gdzie został skopiowany)

miał być łatwy w instalacji dla laika

Prosze o sprawdzenie czy nie ma błedów zagrażających bezpieczeństwu serwera i ocenę wydajności

[PHP] pobierz, plaintext 
<?
/* Galeria v. 0.5
----------------------------------------*/
 
 
 
/* sprwdzanie poprawności $pic
----------------------------------------*/
$pic = $_GET[&#092;"pic\"]; 
if (!$pic or $pic=='-1' or !is_numeric($pic)) { $pic = 1; }
 
 
/* Tworzenie tablicy z plikami .jpg
----------------------------------------*/
@$d = dir('.');
if ($d) { 
  while($entry=$d->read()) {  
	$pos = strpos (strtolower($entry), &#092;".jpg\");
	if (!($pos === false)) {  
		  $arr_pic[] = $entry; 
	}
  }
$d->close(); 
}
@sort ($arr_pic);	 
$total  = sizeof($arr_pic);
 
if ($pic > $total){ $pic = 1;}
 
 
/* Nazwa obrazka
----------------------------------------*/
$img = $arr_pic[$pic-1]; 
 
 
/* Tworznie nawigacji poprzedni/następny
----------------------------------------*/
if ( $pic<2) { 
	$back_url=&#092;"?&pic=1\"; 
	} else { 
	$back_url='?pic='.($pic-1); }
 
if ( $pic+1>$total) { 
	$next_url=&#092;"?pic=\".$pic; 
	}else { 
	$next_url='?pic='.($pic+1); }
 
 
/*----------------------------------------
<?=$img?>			 ;nazwa obrazka
<?=$back_url?>		;link do poprzedniego obrazka w galeri
<?=$next_url?>		;link do następnego obrazka w galeri
<?=$total?>			;liczba obrazków w katalogu
<?=$pic?>			;numer bierzącego obrazka
 
----------------------------------------*/
?><html>
<head>
<title>Galeria</title>
<style>
  body, td { background-color:#999; font:10px verdana; color: #000; }
  a { color:#000; text-decoration: none;}
  img {	background:#000;padding:1px;display:block;border:5px solid #fff;}
  table { border:0;}
</style>
</head>
<body>
<table width=\"100%\" height=\"100%\">
<tr><td align=center>
<p><img src=\"<?=$img?>\"></p>
<p><a href=\"<?=$back_url?>\"><b>&laquo;</b> Poprzednie</a> (<?=$pic?> z <?=$total?>) <a href=\"<?=$next_url?>\">Nastepne <b>&raquo;<b/></a></p>
</td></tr>
</table>
</body>
</html>
[PHP] pobierz, plaintext

hwao

14.09.2004, 15:36:47

Skrypt powiem bardzo biedny....
Malutko kodu tylko najprostrze rzeczy

Cytat

Prosze o sprawdzenie czy nie ma błedów zagrażających bezpieczeństwu serwera i ocenę wydajności

Do tego sluza odpowiednie programy, watpie zeby komus chcialo sie robic 'za nie';

Polecam popracowac nad sposobem pisania kodu:
http://php.pl/index.php/phppl/artyku_y/pom...dardy_kodowania

Vertical

15.09.2004, 20:04:35

77 linii z komentarzami to nie za dużo jak na Galerię obrazków. Pamiętaj, że na galerii nie tylko obrazki mają być kolorowe

Noctovizor

14.10.2005, 00:40:35

a z jakiego katalogu on pobiera te obrazki?

Bakus

14.10.2005, 01:36:18

@Noctovizor: 16 linijka kodu i treść wypowiedzi autora o tym mówią...

Co do samego pomysłu - jak dla mnie prosta galeria to 1 plik... owszem, ale:
- w locie generuje miniaturki z obrazków jakie znajdzie we wskazanym katalogu
- daje możliwość przeglądania podkatalogów (jw.)
- generuje kod zgodny z XHTML
- jest napisany czytelnie (zgodnie z jakimkolwiek standardem)
- posiada komentarze w phpDoc'u (lub podobne)

Gdy Twoja galeria spełni wszystkie w/w punkty, to z wielką chęcią z niej skorzystam

Hacker

6.11.2005, 09:55:32

No cóż trochę uboga ta "galeria". Ja moją na szybko pisałem i zajeło mi to 491 linijek. I nie jest aż taka zaawansowana. Ale za to wymaga MySQL (opis, tytuł, ocenianie zdjęć itp. nie chciało mi się robić na plikach tekstowych)

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.