Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [skrypt] Galeria w php
number0
post
Post #1





Grupa: Zarejestrowani
Postów: 14
Pomógł: 0
Dołączył: 12.03.2004
Skąd: Warszawa

Ostrzeżenie: (0%)
-----

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 
  1. <?
  2. /* Galeria v. 0.5
  3. ----------------------------------------*/
  4.  
  5.  
  6.  
  7. /* sprwdzanie poprawności $pic
  8. ----------------------------------------*/
  9. $pic = $_GET[&#092;"pic\"]; 
  10. if (!$pic or $pic=='-1' or !is_numeric($pic)) { $pic = 1; }
  11.  
  12.  
  13. /* Tworzenie tablicy z plikami .jpg
  14. ----------------------------------------*/
  15. @$d = dir('.');
  16. if ($d) { 
  17.   while($entry=$d->read()) {  
  18. 	$pos = strpos (strtolower($entry), &#092;".jpg\");
  19. 	if (!($pos === false)) {  
  20. 		  $arr_pic[] = $entry; 
  21. 	}
  22.   }
  23. $d->close(); 
  24. }
  25. @sort ($arr_pic);	 
  26. $total  = sizeof($arr_pic);
  27.  
  28. if ($pic > $total){ $pic = 1;}
  29.  
  30.  
  31. /* Nazwa obrazka
  32. ----------------------------------------*/
  33. $img = $arr_pic[$pic-1]; 
  34.  
  35.  
  36. /* Tworznie nawigacji poprzedni/następny
  37. ----------------------------------------*/
  38. if ( $pic<2) { 
  39. 	$back_url=&#092;"?&pic=1\"; 
  40. 	} else { 
  41. 	$back_url='?pic='.($pic-1); }
  42.  
  43. if ( $pic+1>$total) { 
  44. 	$next_url=&#092;"?pic=\".$pic; 
  45. 	}else { 
  46. 	$next_url='?pic='.($pic+1); }
  47.  
  48.  
  49. /*----------------------------------------
  50. <?=$img?>			 ;nazwa obrazka
  51. <?=$back_url?>		;link do poprzedniego obrazka w galeri
  52. <?=$next_url?>		;link do następnego obrazka w galeri
  53. <?=$total?>			;liczba obrazków w katalogu
  54. <?=$pic?>			;numer bierzącego obrazka
  55.  
  56. ----------------------------------------*/
  57. ?><html>
  58. <head>
  59. <title>Galeria</title>
  60. <style>
  61.   body, td { background-color:#999; font:10px verdana; color: #000; }
  62.   a { color:#000; text-decoration: none;}
  63.   img {	background:#000;padding:1px;display:block;border:5px solid #fff;}
  64.   table { border:0;}
  65. </style>
  66. </head>
  67. <body>
  68. <table width=\"100%\" height=\"100%\">
  69. <tr><td align=center>
  70. <p><img src=\"<?=$img?>\"></p>
  71. <p><a href=\"<?=$back_url?>\"><b>&laquo;</b> Poprzednie</a> (<?=$pic?> z <?=$total?>) <a href=\"<?=$next_url?>\">Nastepne <b>&raquo;<b/></a></p>
  72. </td></tr>
  73. </table>
  74. </body>
  75. </html>
[PHP] pobierz, plaintext


Ten post edytował number0 14.09.2004, 16:23:04
Go to the top of the page
+Quote Post
hwao
post
Post #2


Developer


Grupa: Moderatorzy
Postów: 2 844
Pomógł: 20
Dołączył: 25.11.2003
Skąd: Olkusz
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
Go to the top of the page
+Quote Post
Vertical
post
Post #3





Grupa: Zarejestrowani
Postów: 848
Pomógł: 0
Dołączył: 7.07.2004
Skąd: Wrocław

Ostrzeżenie: (0%)
-----

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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
Noctovizor
post
Post #4





Grupa: Zarejestrowani
Postów: 8
Pomógł: 0
Dołączył: 2.08.2005

Ostrzeżenie: (0%)
-----

a z jakiego katalogu on pobiera te obrazki?
Go to the top of the page
+Quote Post
Bakus
post
Post #5


Administrator serwera


Grupa: Przyjaciele php.pl
Postów: 909
Pomógł: 0
Dołączył: 12.08.2003
Skąd: /var/www/wroclaw.php

Ostrzeżenie: (0%)
-----

@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 (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
Go to the top of the page
+Quote Post
Hacker
post
Post #6





Grupa: Zarejestrowani
Postów: 225
Pomógł: 0
Dołączył: 1.11.2005

Ostrzeżenie: (0%)
-----

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)
Go to the top of the page
+Quote Post
« Następny starszy · Oceny · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 23.08.2025 - 09:24
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn