Image Loader

images php coding issues or problems here.

Moderators: macek, egami, gesf

Post Reply
User avatar
swirlee
Moderator
Moderator
Posts: 2257
Joined: Sat Jul 05, 2003 1:18 pm
Location: A bunk in the back
Contact:

Post by swirlee » Fri May 14, 2004 10:47 pm

The link you posted is very poorly-written. I recommend that you go to HotScripts.com and browse their PHP Image Galleries section until you find a free script which meets your requirements.

User avatar
swirlee
Moderator
Moderator
Posts: 2257
Joined: Sat Jul 05, 2003 1:18 pm
Location: A bunk in the back
Contact:

Post by swirlee » Sat May 15, 2004 9:31 am

Sorry, I didn't mean to insult, especially if you're a PHP beginner and wrote the code yourself. I was in a hurry when I replied to your post. I thought that you were using someone else's package, in which case I would have recommended something more robust. Basically I was trying to prevent you from reinventing the wheel. There are literally hundreds of image gallery scripts out there, at least one of which almost certainly meets your requirement.

Concerning your code, the trouble is that you mix data, logic, and presentation so heavily, that in the end your code will be very difficult to maintain. HTML should be kept separate from the PHP as much as possible, as should the data describing your images.

I know for newbies this is a troublesome concept, because most beginning PHP books and tutorials start out saying, "look how easy it is to mix PHP and HTML!"

Once again, I didn't mean to insult you and I hope I haven't discouraged you from using PHP or our forum. :)

Post Reply

Who is online

Users browsing this forum: No registered users and 1 guest