om Posted December 2, 2008 Share Posted December 2, 2008 hELLO EVERY ONE, I am just a begginer, and this site is and will be my only source of income, so Kindly don't spoil my site but try to find flaws and report to me at oomdotin@gmail.com www.ucy.in/cmb/ Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/ Share on other sites More sharing options...
darkfreaks Posted December 2, 2008 Share Posted December 2, 2008 on your register page please filter your variables with this function <?php function clean($text) { $text=strip_tags(trim(htmlspecialchars(mysql_real_escape_string($text)))); return htmlspecialchars($text);}?> Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-703705 Share on other sites More sharing options...
newb Posted December 3, 2008 Share Posted December 3, 2008 disgusting design Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705309 Share on other sites More sharing options...
om Posted December 3, 2008 Author Share Posted December 3, 2008 on your register page please filter your variables with this function <?php function clean($text) { $text=strip_tags(trim(htmlspecialchars(mysql_real_escape_string($text)))); return htmlspecialchars($text);}?> Thank you, 1. You mean to say apply clean() on all input values of the form??? 2. Were you able login with values on begin page or register page?? I am still trying to market my site, Income is yet to start Once again Thank you very much Mr. Darkfreaks for your Good fair help!!! Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705428 Share on other sites More sharing options...
om Posted December 3, 2008 Author Share Posted December 3, 2008 disgusting design Thanks, Can you Elaborate????? Dark Green back ground consumes less power I wanted my site to be fair, hence I expressed facts right on first page? Thanks Once again Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705433 Share on other sites More sharing options...
darkfreaks Posted December 4, 2008 Share Posted December 4, 2008 1.) i am much more of a coder than a designer but i think it looks horrid. 2.) when you call the function to each variable it will clean it. Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705567 Share on other sites More sharing options...
ardyandkari Posted December 4, 2008 Share Posted December 4, 2008 Quote from: newb on Today at 03:36:09 PM disgusting design Thanks, Can you Elaborate????? Dark Green back ground consumes less power I wanted my site to be fair, hence I expressed facts right on first page? Thanks Once again what he means is actually a list of things that could/should be changed. 1.) i dont care if green consumes less power, it's just ugly...but in the end it is your site, you can do what you want. 2.) the layout sucks. when i was looking at the page, my eye didnt know where to look for anything. i was unable to locate/understand what the site was about easily. suggestion: make the site multiple pages. if you break up the content, your readers/members can more easily read and understand what they are looking at. 3.) that form for registration is terrible. like i said in #2, if you were to break up the one page into many, that would give you the room to have a dedicated "Register" page. again, definitely sanitize your user inputs. if you dont, people can really destroy your db, and if this is your only source of income...game over. Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705664 Share on other sites More sharing options...
darkfreaks Posted December 4, 2008 Share Posted December 4, 2008 2.) the layout sucks. when i was looking at the page, my eye didnt know where to look for anything. i was unable to locate/understand what the site was about easily. suggestion: make the site multiple pages. if you break up the content, your readers/members can more easily read and understand what they are looking at. Exactly what i was thinking 4.) Remember to sanitize your inputs with the clean() function darkfreaks provided indeed Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705665 Share on other sites More sharing options...
dclamp Posted December 4, 2008 Share Posted December 4, 2008 disgusting design Thanks, Can you Elaborate????? Dark Green back ground consumes less power I wanted my site to be fair, hence I expressed facts right on first page? Thanks Once again 1. Too much Green 2. Too cluttered 3. Marquee is bad 4. iFrames are bad 5. Mixed colors 6. You didnt use CSS 7. You used Hex, RGB, Name for colors, pick one. 8. Failed HTML Traditional http://validator.w3.org/check?uri=http%3A%2F%2Fwww.ucy.in%2Fcmb%2F&charset=(detect+automatically)&doctype=Inline&group=0 Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705668 Share on other sites More sharing options...
Adam Posted December 4, 2008 Share Posted December 4, 2008 To be brutally honest, it wouldn't be a surprise if I saw this on some kind of 'worlds worst websites' site.. It's everything that's wrong with web design! It's google's nightmare too! Try learning some design techniques.. http://www.google.co.uk/search?q=web+design+techniques As ardyandkari said, this will not make you money! Sorry to be harsh.. Adam Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-705765 Share on other sites More sharing options...
dropfaith Posted December 4, 2008 Share Posted December 4, 2008 agreed i have no idea what your page does becuase the clutter drove me insane check out lawrenceguide the link in my sig it has tons of content and huge room to grow by user input and such and theres hardly any clutter at all making navigation easy and finding the info needed easier Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-706070 Share on other sites More sharing options...
Mchl Posted December 4, 2008 Share Posted December 4, 2008 Dark Green back ground consumes less power This is true only for CRT monitors. For LCDs the element that consumes most power is the backlight, and it doesn't care what colour your background is. Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-706098 Share on other sites More sharing options...
petersro Posted December 5, 2008 Share Posted December 5, 2008 IMO, the layout and color scheme, is well, interesting (to put it nicely), if this site is as you say your only line of income, try and make it welcoming, it honnestly hurt my eyes when it loaded, my suggestion, go back to the drawing bord with layout and color. Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-706558 Share on other sites More sharing options...
dclamp Posted December 6, 2008 Share Posted December 6, 2008 There are a lot of websites that offer free template downloads. You may want to check on out. Your website looks like it came out of the 90's Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-707408 Share on other sites More sharing options...
om Posted December 8, 2008 Author Share Posted December 8, 2008 1.) i am much more of a coder than a designer but i think it looks horrid. 2.) when you call the function to each variable it will clean it. God bless U, MY LOGIC CODIFIES EVEN ENTIRE PASSWORD LIKE DIGESTS DO AND SOME MORE LOGIC, SO THAT An example SQL Injection Attack LIKE THE ONE CASE EXPLAINED IN PHP5 MANUAL IS RULED OUT SELECT * FROM users WHERE user='aidan' AND password='' OR ''='' SO IS IT SAFE i WILL SHORTLY LOAD AT WWW.UCY.IN/TCMB HENCE YOU CAN TEST THIS BUT DONT SPOIL THE DATABASE VALUES REGARDING LAYOUT i WILL CERTAINLY REDESIGN IT TO SOME EXTENT. tHANKS FOR ALL THE THIRTEEN RESPONSERS GOD WILL CERTAINLY BLESS YOU ALL Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709225 Share on other sites More sharing options...
om Posted December 8, 2008 Author Share Posted December 8, 2008 There are a lot of websites that offer free template downloads. You may want to check on out. Your website looks like it came out of the 90's Thank you very much Mr. Dclamp, I have downloaded some but I will try to code it my self It has my own signature / light wt => loads quickly. Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709236 Share on other sites More sharing options...
dropfaith Posted December 8, 2008 Share Posted December 8, 2008 as light weight as it is its also hard to navigate for content that you want to find on the page. you can build light weight fast loading sites that are still very easy to get the content needed and only the content needed without looking over 100 sections like i felt i needed to do on your page Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709268 Share on other sites More sharing options...
om Posted December 8, 2008 Author Share Posted December 8, 2008 Quote from: newb on Today at 03:36:09 PM disgusting design Thanks, Can you Elaborate????? Dark Green back ground consumes less power I wanted my site to be fair, hence I expressed facts right on first page? Thanks Once again what he means is actually a list of things that could/should be changed. 1.) i dont care if green consumes less power, it's just ugly...but in the end it is your site, you can do what you want. 2.) the layout sucks. when i was looking at the page, my eye didnt know where to look for anything. i was unable to locate/understand what the site was about easily. suggestion: make the site multiple pages. if you break up the content, your readers/members can more easily read and understand what they are looking at. 3.) that form for registration is terrible. like i said in #2, if you were to break up the one page into many, that would give you the room to have a dedicated "Register" page. again, definitely sanitize your user inputs. if you dont, people can really destroy your db, and if this is your only source of income...game over. Great thank + God Bless you, Sunday in india my sites were down I thought, some hackers have really spoiled my site. But today they are up again I dont dont know really what happened my service providers didnt comment to me. May it was usual maintainer's procedure. Well if really some caused because of my site they would have contacted me. Any how I once again appeal, dont spoil but just try to login and post you message in the MESSAGE BOARD LIKE THIS ONE. I wanted Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709272 Share on other sites More sharing options...
ardyandkari Posted December 8, 2008 Share Posted December 8, 2008 it doesnt look like you have made any changes at all... this is what i suggest: __________ layout-> | header | ------------ |m| c | |e | o | |n | n | |u | t | | | e | | | n | | | t | ------------- | footer | ------------- this can all be done w/ css to keep a fast loading page. you can also have the menu right under the header... i was going to try to help you split up the one page into a few, just for an idea...but i still cant decide what you are doing with your page...what is the purpose? you DEFINITELY want to split that one page up into many because more than likely you WILL NOT make any money with the layout (or lack thereof) that you have now. a page not only has to have content, but also needs to be easy on the eye. http://www.freecsstemplates.org/ http://www.free-css-templates.com/ http://www.free-css.com/ <--this one even has just layouts...not templates. Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709302 Share on other sites More sharing options...
om Posted December 8, 2008 Author Share Posted December 8, 2008 it doesnt look like you have made any changes at all... this is what i suggest: __________ layout-> | header | ------------ |m| c | |e | o | |n | n | |u | t | | | e | | | n | | | t | ------------- | footer | ------------- this can all be done w/ css to keep a fast loading page. you can also have the menu right under the header... i was going to try to help you split up the one page into a few, just for an idea...but i still cant decide what you are doing with your page...what is the purpose? you DEFINITELY want to split that one page up into many because more than likely you WILL NOT make any money with the layout (or lack thereof) that you have now. a page not only has to have content, but also needs to be easy on the eye. http://www.freecsstemplates.org/ http://www.free-css-templates.com/ http://www.free-css.com/ <--this one even has just layouts...not templates. thanks I will it may take 2 days. Please for gods sake dont spoil any thing try to login and see in www.ucy.in Apost a message. I wanted to know whether some body can do it without values? Please for gods sake dont spoil any thing try to login and see in www.ucy.in/tcmb I want to can some on login without actual values Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709314 Share on other sites More sharing options...
om Posted December 8, 2008 Author Share Posted December 8, 2008 disgusting design Thank you very much Thanks, Can you Elaborate????? Dark Green back ground consumes less power I wanted my site to be fair, hence I expressed facts right on first page? Thanks Once again 1. Too much Green 2. Too cluttered 3. Marquee is bad 4. iFrames are bad 5. Mixed colors 6. You didnt use CSS 7. You used Hex, RGB, Name for colors, pick one. 8. Failed HTML Traditional http://validator.w3.org/check?uri=http%3A%2F%2Fwww.ucy.in%2Fcmb%2F&charset=(detect+automatically)&doctype=Inline&group=0 Many Thanks for Validator GOD BLESS YOU Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709321 Share on other sites More sharing options...
darkfreaks Posted December 8, 2008 Share Posted December 8, 2008 Value Error : top Parse Error 1. Too cluttered 2. Marquee is bad 3. iFrames are bad 4. You didnt use CSS Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709363 Share on other sites More sharing options...
darkfreaks Posted December 8, 2008 Share Posted December 8, 2008 Value Error : top Parse Error take out : <body style="color: rgb(51, 255, 51); background-color: rgb(0, 51, 0);" onload="sp_resetcbe()" text="#00ff00" vlink="#ffcc00" alink="#ff0000" bgcolor="#004300" link="#ffff00"> replace with: <style type="text/css"> BODY { background-color:#030; color:#3F3; vlink:#fc0; alink:#fc0; text:#0f0; link:#ff0; } </style> 1. Too cluttered 2. Marquee is bad 3. iFrames are bad Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709391 Share on other sites More sharing options...
darkfreaks Posted December 8, 2008 Share Posted December 8, 2008 remove input style type and replace with: <style type="text/css"> input[type="text"] { background-color:#FFFFA0; } </style> Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709410 Share on other sites More sharing options...
Maq Posted December 8, 2008 Share Posted December 8, 2008 Your website looks like it came out of the 90's lol. Sorry to sound so harsh but you need to redo the whole site. Everything I want to say has been said again and again. Seems like you haven't tried much yet. Come back when you make a significant change. You can't possibly think that's good, do you? I am just a begginer, and this site is and will be my only source of income, so I feel bad for you... Link to comment https://forums.phpfreaks.com/topic/135094-kindly-dont-spoil-my-site-but-try-to-find-flaws-and-report-to-me/#findComment-709424 Share on other sites More sharing options...
Recommended Posts