Getting "Code Review" Feedback?
-
Greetings and Salutations: I'm a junior level developer seeking to continue my career in software development. To that end I've created my own website and posted a few small applications, to demonstrate my skills. I'd like to get some [constructive] feedback on my work, but I'm not sure which thread I should post the link to my site in. Would it be OK to post here? If not, could someone suggest where the best place to post would be? Andre
-
Greetings and Salutations: I'm a junior level developer seeking to continue my career in software development. To that end I've created my own website and posted a few small applications, to demonstrate my skills. I'd like to get some [constructive] feedback on my work, but I'm not sure which thread I should post the link to my site in. Would it be OK to post here? If not, could someone suggest where the best place to post would be? Andre
-
Greetings and Salutations: I'm a junior level developer seeking to continue my career in software development. To that end I've created my own website and posted a few small applications, to demonstrate my skills. I'd like to get some [constructive] feedback on my work, but I'm not sure which thread I should post the link to my site in. Would it be OK to post here? If not, could someone suggest where the best place to post would be? Andre
This place is fine to post code review questions. You can try Quick questions as well.
Apps - Color Analyzer | Arctic | XKCD | Sound Meter | Speed Dial
-
Greetings and Salutations: I'm a junior level developer seeking to continue my career in software development. To that end I've created my own website and posted a few small applications, to demonstrate my skills. I'd like to get some [constructive] feedback on my work, but I'm not sure which thread I should post the link to my site in. Would it be OK to post here? If not, could someone suggest where the best place to post would be? Andre
This is the website I build. The link is direct to the demos page, but feel free to look around the site and offer suggestions: http://xtraordinayr.net/CVAppsDemoIntroPage.aspx[^] I appreciate any suggestions that are constructive and will help me enhance my skills. Thanks. Andre
-
This is the website I build. The link is direct to the demos page, but feel free to look around the site and offer suggestions: http://xtraordinayr.net/CVAppsDemoIntroPage.aspx[^] I appreciate any suggestions that are constructive and will help me enhance my skills. Thanks. Andre
That's pure HTML, Static - No functionalities. 1. All pages are under constructions – there is nothing to look for on them 2. Look and Feel is not nice at all – I suggest you visit website that gives free templates for ideas. You can use them or get the idea on how to structure your site. I leant a lot from them 3. Your site also doesn’t fit on my screen. Resolution not good, I dislike vertical scrolling. 4. You menu keep swapping around when I going to a different page, I recommend using controls for menus or work with Master Pages. - Do as someone above has suggested, paste your code that want to be reviewed.
I remain joe!
-
This is the website I build. The link is direct to the demos page, but feel free to look around the site and offer suggestions: http://xtraordinayr.net/CVAppsDemoIntroPage.aspx[^] I appreciate any suggestions that are constructive and will help me enhance my skills. Thanks. Andre
Your still at the amateur level. Not even close to something someone would pay for. Well, you have no design skills, you should of just copies another website look and feel. You don't understand CSS, that's why the site won't fit on my monitor You don't understand font sizes, and using a master CSS file to set the initial font size, and sizing other fonts from there, or using em such as 0.8em It has nothing to do with ASP.net, Just looks like HTML to me, but I looked at the source and saw you used LinkButtons to make the menu on the left. [Suggestions] I think you have misinterpreted the use of ASP.Net. You should go back and learn how to make a website first using HTML and CSS, and then choose a technology after that. Hone your knowledge of HTML and design first, or just stick to writing code, and not HTML. Then you'll be a master at it. I was expecting some good looking code or scripts or something. [edit] Here's a great place to learn web design and CSS in 30 days http://learncss.tutsplus.com/[^]
-
Greetings and Salutations: I'm a junior level developer seeking to continue my career in software development. To that end I've created my own website and posted a few small applications, to demonstrate my skills. I'd like to get some [constructive] feedback on my work, but I'm not sure which thread I should post the link to my site in. Would it be OK to post here? If not, could someone suggest where the best place to post would be? Andre
Check my article in my signature on some general code review checklist for developers.
My Reading-o-Meter
Previous -> Read "CLR via C#" by Jeffrey Richter. Current -> Exploring WCF thru Apress' "Pro WCF" by Chris Peiris and Dennis Mulder. Next -> Need to read "The Art of Computer Programming" by Donald E. Knuth.
-
This is the website I build. The link is direct to the demos page, but feel free to look around the site and offer suggestions: http://xtraordinayr.net/CVAppsDemoIntroPage.aspx[^] I appreciate any suggestions that are constructive and will help me enhance my skills. Thanks. Andre
Changing the argument values without changing the signature of a method is not overloading. A CV objective and lists of skills are not articles. ArrayLists aren't really best practice now that generics are available. (List, List). Failing to use generics when you say that you know .Net 3.5 and 4.0 is not good. Using poorly named embedded styles like style11 does not show a high level of CSS knowledge. Well named styles broken out into a separate CSS file is a better idea. Your apostrophes don't look like apostrophes. You probably did a copy paste from word and got mangled smart quotes.