Witam,

Mam taki skrypt, który wyświetla w tabeli pliki graficzne. I chciałbym, żeby następna strona galerii już była generowana przez skrypt, z niewielkimi, acz istotnymi zmianami. Chodzi o tablicę, która jest odpowiedzialna za nazwy plików oraz ich (count) ilość. Poniżej zamieszczam kod wraz z komentarzami (będzie łatwiej wytłumaczyć).

[PHP] pobierz, plaintext 
<?
 
//lista plików na srwerze w katalogu images
 
$dir=opendir('./images'); 
 
 
$file_list=" "; 
while($file_name=readdir($dir)) 
{ 
if(($file_name!=".")&&($file_name!="..")) 
{ 
$file_list.=" $file_name"; 
} 
} 
$file_list.=""; closedir($dir); 
 
//umieszczenie w tablicy nazw plików z kat. images
 
$dupa = explode(" ", $file_list);
 
 
// tutaj zaczyna się generowanie tabeli wypełnianej plikami graficznymi
 
echo "<center><table border='0'><p class='paragraph'>";
 
  for ($index = 2; $index < count($dupa); $index++) 
	   {
		  if ($index > 17) {
 
 
 
/* tutaj natomiast rozpoczyna się tworzenie pliku (kopiowanie) i chciałbym żeby w nowym pliku pętla wyglądała nieco inaczej, mianowicie ż
eby zaczynała się od pliku 17 i konczyła na 32, a poza tym żeby do tych poniższyc
h warunków odnośnie wstawiania znacznika tr również było dodawane 16. Jak to zrob
ić?*/
 
 
	$aCurBasePath = dirname( $PATH_TRANSLATED );
	$aNewName = "gallery".$index.".php" . $thefile;
	$thefile = "gallery.php";
 
				if  (file_exists($aNewName)) 
		{
			   echo "<a href='".$aNewName."'><center>Następna strona</center></a><br><br>"; 
		}
						   else
		{
 
				copy ($thefile, $aNewName );
						   echo "<a href='".$aNewName."'><center>Następna strona</center></a><br><br>";
 
		}
 
	break;
 
		  }
 
 
 
 
else if ($index == 2 || $index == 6 || $index == 10 || $index == 14)
	 {
 
					echo "<tr>";
echo "<td align='center'>$dupa[$index]<br><a href = 'images/".$dupa[$index]."'><img border = '0' src = 'images/".$dupa[$index]."' width = '100' height = '75'></a> <form method='post' action = 'del.php'><input type = 'submit' name = '".$dupa[$index]."'></form> <br></td>";
 
				 }
 
 
else if ($index == 6 || $index == 10 || $index == 14 || $index == 18)
 
	 {
 
 
echo "<td align='center'>$dupa[$index]<br><a href = 'images/".$dupa[$index]."'><img border = '0' src = 'images/".$dupa[$index]."' width = '100' height = '75'></a> <form method='post' action = 'del.php'><input type = 'submit' name = '".$dupa[$index]."'></form> <br></td>";
 
echo "</tr>";
 
	 }
 
 
else if ($index != 2 || $index != 6 || $index != 10 || $index != 14)
 
	 {
 
 
echo "<td align='center'>$dupa[$index]<br><a href = 'images/".$dupa[$index]."'><img border = '0' src = 'images/".$dupa[$index]."' width = '100' height = '75'></a> <form method='post' action = 'del.php'><input type = 'submit' name = '".$dupa[$index]."'></form> <br></td>";
 
 
	 }
 
 
 
} 
 
//koniec petli
 
?>
[PHP] pobierz, plaintext 

poprawiam
---
nospor




A ponadto może jak już proszę o pomoc, to powiedzcie mi jak zrobić link lub zwykły input, który byłby odpowiedzialny za usuwanie konkretnego zdjęcia (tego pod którym input się znajduje)? To też mi nie działa tak jak trzeba.


Dzięki z góry za odpowiedzi.

Jak coś to cały kod (skrypt znajduje się tutaj)


http://www.dnawrot.webas.pl/php/gallery.php

Pozdrawiam
dnawrot

Dodawanie kolejnych nazw plików do stringa oddzielonych spacją to nie jest najlepszy pomysł... Najlepiej od razu je do tabeli dodaj.

[PHP] pobierz, plaintext 
<?php
$files[] = $file_name;
?>
[PHP] pobierz, plaintext 

Usuwanie to tylko znajomość htmla...
Link:

[HTML] pobierz, plaintext 
<a href="usun.php?id=x">link</a>
[HTML] pobierz, plaintext 

Przycisk:

[HTML] pobierz, plaintext 
<form action="usun.php" method="get">
<input type="hidden" name="id" value="x" />
</form>
[HTML] pobierz, plaintext 

W obu przypadkach x jest identyfikatorem lub nazwą zdjęcia, które chcesz usunąć. W skrypcie usun.php będziesz miał zmienną $_GET['id']. Druga metoda pozwala na przesłanie POST-em. Uważaj na te zmienne, bo to częsty błąd nie sprawdzać wejścia... Zawsze trzeba je sprawdzić...

dobra dzięki, usuwanie działa jak należy, ale nie wiem za bardzo o co chodzi z tym tworzeniem pliku, prosiłbym trochę jaśniej, bo ciemny jestem z php

pozdrawiam

Chodziło mi o to, że trochę przekombinowałeś z wydobywaniem nazw plików. Ty robisz to tak:

[PHP] pobierz, plaintext 
<?php
$file_list=" "; 
while($file_name=readdir($dir)) 
{ 
if(($file_name!=".")&&($file_name!="..")) 
{ 
$file_list.=" $file_name"; 
} 
} 
$file_list.=""; closedir($dir); 
 
//umieszczenie w tablicy nazw plików z kat. images
 
$dupa = explode(" ", $file_list);
?>
[PHP] pobierz, plaintext 

Pomijając nazwę tablicy z nazwami plików i to, że explode wypluje tablicę z dwoma pierwszymi elementami pustymi, oraz to, że nazwy plików zawierające spacje rozwalą tablicę, to łatwiej i szybciej od razu nazwy umieszczać w tablicy.

[PHP] pobierz, plaintext 
<?php
$fileNames = array()
while ($fileName = readdir($dir)) {
	if ($fileName != '.' && $fileName != '..') {
		$fileNames[] = $fileName;
	}
}
?>
[PHP] pobierz, plaintext 

W rezultacie masz tablicę $fileNames zawierającą te same dane, ale kod jest pozbawiony błędów, szybszy i czytelniejszy.

Dzięki za zwrócenie uwagi, poprawię to. Ale czy możesz mi powiedzieć jak rozwiązać problem z tworzeniem nowych plików, a w zasadzie kopiowaniem plików php? A poza tym jeszcze jedna rzecz mi przyszła na myśl. W momencie przechodzenia do następnej strony galerii nie tylko musi być tworzona następna strona .php, ale jeśli okaże się że nie ma więcej plików to następna, która została stworzona przy ostatnim wywoływaniu skryptu musi być usunięta. Nie wiem czy dobrze mnie zrozumiałeś. Ponadto, przydałoby się aby przy ostatniej stronie wyświetlany był tylko link "poprzednia strona", a nie "poprzednia || następna". Dzięki z góry za odpowiedź. Pozdrawiam
dnawrot