simply Array: Syntax trouble vs Desired Output

ATS16805

I've got a script working real well for reading the .jpg's within a folder, and printing them to screen (thumbs) via foreach() loop

my ultimate goal is to come up w/ a way to do the following:

User selects a thumbnail from my gallery, which itself is displayed via a glob() script function
desired larger-size image is displayed on its own HTML "slide" page (it's currently linking direct to .jpg, w/out the HTML slide container file)
to enhance user experience using a pop-up for the image display, need to enable a "previous / next" style navigation from w/in the pop-up (if the user decides to use that method-- she could go back to the thumbs if desired)

my plan is to use put the consecutive thumbnail $items, read by the glob() function, into a string array, so i can come back later and make html files from the strings in that array-- thinking that i can use that info also for making my "previous -- next" navigation-- the point being of course never to have to hard-code any of the html side-files, or the links to the thumbs, etc.

here's what i'm working with-- i thiink i'm on the right track, but my string array is fouled up.

$filelist = "";

$folder = 'images/stadiumjump';
$pics = count(alpharead3($folder));

$pn = "";

foreach(alpharead3($folder) as $item) {
	if ($item != "thumbs") {
	$pn = $pn++;
	$photonum = array($pn => $item);

 $filelist .= '<div class="float"><a href="'.$folder.'/'.$item.'"><img src="'.$folder.'/thumbs/'.$item.'" alt=" "'.$item.'" /></a><br /></div>';

} elseif ($item == "thumbs") {
break;
}
}	

print_r ($photonum); /* $photonum is my array TEST -- which proves i'm not there yet. the $filelist is okay */
echo $filelist;

which outputs to the screen:

Array ( [] => stadium.jpg )

stadium.jpg being the last file of the images folder

what have i done wrong w/ my array, causing it to output ([] => stadium.jpb) ? really-- that's what i want to know-- if i can figure out how to get the code below i think i'll be set for the second part-- the html slides.

$photonum[0] = imageone.jpg
$photonum[1] = imagetwo.jpg

thanks.

frikikip

You are creating a new array each time you loop, overwriting the old one.

So instead of this:

$pn = "";

foreach(alpharead3($folder) as $item) {
    if ($item != "thumbs") {
    $pn = $pn++;
    $photonum = array($pn => $item);

try this

$photonum = array();

foreach(alpharead3($folder) as $item) {
    if ($item != "thumbs") {
    $photonum[] = $item;

Greets,

Marvin.

Performing a ++ operation on a string isn't going to work. It should be:
$pn = 0;
$pn++; // Not $pn = $pn++;

ATS16805

frikikip wrote:
You are creating a new array each time you loop, overwriting the old one.

So instead... try this
$photonum = array();

foreach(alpharead3($folder) as $item) {
    if ($item != "thumbs") {
    $photonum[] = $item; 

hey, thank you, frikikip!!

it works beautifully-- output is now:

Array ( [0] => 1.jpg [1] => 6.jpg [2] => 7.jpg [3] => 9.jpg [4] => 12.jpg [5] => 13.jpg [6] => 16.jpg [7] => 17.jpg [8] => drophigh.jpg [9] => stadium.jpg )

which is precisely what i was trying to get, of course. 🙂
for the record, the "AlphaRead3" thing is a script from Admiral at NuclearPixel.com (i can't recall where i found it, but i'm happy to share if anyone would like-- quite useful in that it reads the file names and makes 'em into links to those files. i simply modded it to turn them into thumbnail links to the larger images, and used CSS floats so they cascade down the page w/out the need for a table:
now, all i gotta do is figure out the next part-- which will be to take that dynamically generated array data, and produce my hyperlinks out of it... which i'll then want to send that info over into "slide" pages into which the corresponding "viewing size" image should be placed (so my entire gallery should be dynamic)... hmmm... sounds like one i'll be pullin' some hair out with... but, oh well... gotta learn somehow.
🙂
let me at least ask this before i set out to work on it-- is what i'm describing i want w/ those slides "possible" w/ just php, and what about-- is it wise to do it this way, or does anyone recommend i'll need to mix in some javascript?

frikikip wrote:
Performing a ++ operation on a string isn't going to work. It should be:
$pn = 0;
$pn++; // Not $pn = $pn++;

yeh.. total mind-lapse there,

okay-- cool, i totally see what you mean about re-writing the array too! thanks very much!

madwormer2

Performing a ++ operation on a string isn't going to work

You may wanna try this then:

$a = "a";
$a++;
echo $a;

And the output on your page has an extra " in the alt= section, causing all the images alts to... well, not work:

alt=" "12.jpg"

will not work.