dannyluked Posted December 5, 2009 Share Posted December 5, 2009 Hi, I have been running this website for about six months now. It has no purpose yet apart form helping me practise my PHP coding skills! It is hosted from a free hosting company and I'd just like a few people to take a look and tell me how I could make any improvements http://dannyluked.comze.com PS. I only really use the site to create PHP stuff so I've not yet moved onto more complicated CSS stuff (planning that soon) Quote Link to comment Share on other sites More sharing options...
ayok Posted December 6, 2009 Share Posted December 6, 2009 It's okay for the first time. Only i see <head> inside a <td>?? Quote Link to comment Share on other sites More sharing options...
canadabeeau Posted December 7, 2009 Share Posted December 7, 2009 This is a bit all over the place, parts of the source code are wrong, stuff that should be in the <head> is before it (see lines #3 #4) Not that bad for a first site Quote Link to comment Share on other sites More sharing options...
dannyluked Posted December 7, 2009 Author Share Posted December 7, 2009 Thanks guys! The main reason it is so badly coded is that I made the basic HTML in good code a while ago. Since then Ive started learning PHP so codes have been messed with! I am going to finish the forum and then try to tidy it up. PS. Does anyone know why including header/navigation is slowing down my site and what is the best way to do it, Thanks Quote Link to comment Share on other sites More sharing options...
mikesta707 Posted December 10, 2009 Share Posted December 10, 2009 It looks pretty good. Your CSS/Javascript/HTML is all over the place, and you should think about organizing it a little better so the page isn't as cluttered. You need to check verify stuff with PHP too. You're register javascript validation works (to make sure they put in all the correct stuff) but if I turn javascript off, I can register with empty data. I can also log in with empty data too, and the login works after I created an account with empty data. You do use mysql_real_escape_string() on your data, which is good. That flash calender is kind of annoying. You also probably want to apply html entities to your comments. I can inject html in there, and can do harmful things to your site Quote Link to comment Share on other sites More sharing options...
sudoku-solver Posted December 11, 2009 Share Posted December 11, 2009 Id say also move the advertisers boxes to the LEFT menu I know this oposite of what most do but you wil maximize click throughs by doing so, Quote Link to comment Share on other sites More sharing options...
dannyluked Posted December 13, 2009 Author Share Posted December 13, 2009 Thanks for the replies. I have stoped people injecting html and got rid of the flash calendar. I am now going to lighten images as they slow the site. Quote Link to comment Share on other sites More sharing options...
448191 Posted December 13, 2009 Share Posted December 13, 2009 IMPORPANT? Quote Link to comment Share on other sites More sharing options...
roopurt18 Posted December 13, 2009 Share Posted December 13, 2009 "Designed for FireFox." Is it 1996 again? Quote Link to comment Share on other sites More sharing options...
dannyluked Posted December 14, 2009 Author Share Posted December 14, 2009 Thought it was an easy way to link to firefox... Quote Link to comment Share on other sites More sharing options...
roopurt18 Posted December 14, 2009 Share Posted December 14, 2009 You'll have to find a way show your support for FireFox without igniting memories of the early days of the web, when every site was "designed" for one browser and version only. Quote Link to comment Share on other sites More sharing options...
lynxus Posted December 17, 2009 Share Posted December 17, 2009 Looks very pretty and clean. Loads quite fast. Not sure what else to say here. Welldone. -G Quote Link to comment Share on other sites More sharing options...
FaT3oYCG Posted December 17, 2009 Share Posted December 17, 2009 IMPORPANT? lol Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.