|
I experienced to work with the code which was reviewed, but still of low quality. Actually reviewer looked through the code and could think that this is more formal procedure and didn't pay enough attention to the process of review and said: "I have reviewed...
Started by sergdev on
, 7 posts
by 7 people.
Answer Snippets (Read the full thread at stackoverflow):
With that in mind:
A review must never be undertaken by a single wrote some code....
The only purpose for a code review is to improve the quality of the code and the quality of the processes that produce the code.
|
|
Everyone that has ever had to perform any form of code review has probably been here. You're a member of a code review team, and during the code review you come to an issue where a member of the team is overly aggressive about their viewpoint of a solution...
Started by Jason Mock on
, 12 posts
by 12 people.
Answer Snippets (Read the full thread at stackoverflow):
To answer responsible in the end? Is the code review team considered just a help for the programmers, or has?
If they are getting aggressive....
That's what makes a code review difficult on the human level.
You tend to hurt feelings.
|
|
Disclaimer I don't do code review, so this question is purely of academic interest.
I have seen plenty of posts on stackoverflow that advocates/requires code review before it is allowed into source control.
If you do this, where do you store code the ...
Started by tomjen on
, 8 posts
by 8 people.
Answer Snippets (Read the full thread at stackoverflow):
Maybe, so I'm in the camp....
If it's a reasonable amount of code, it can be emailed to reviewers.
From there some gets it for review too long to review.
The answer is that the code belongs just to the versioning system.
|
Ask your Facebook Friends
|
How many of you use walkthroughs instead of reviews? Or in conjunction with Reviews? Or only use walkthroughs?
Are Walkthroughs beneficial and how have you all implemented code walkthroughs?
Started by Marcusg on
, 6 posts
by 6 people.
Answer Snippets (Read the full thread at stackoverflow):
A walkthrough is a thorough type of code review where you actually go through the code line types of code review where general coding style is checked for or certain structures are examined a peer code....
|
|
What are the best places for freelancers or small companies to get code reviewed for PHP and JavaScript? Forums are an option, but are there any sites dedicated specifically to code reviews?
Edit: Just for clarification, I'm looking more for a website...
Started by VirtuosiMedia on
, 5 posts
by 5 people.
Answer Snippets (Read the full thread at stackoverflow):
Snipplr is handy too, but is more like a web notebook rather than a dedicated code discussion....
It's a web application code review tool developed by Google engineer and Python BDFL GuidoRefactor My Code comes to mind.
Into Rietveld.
|
|
Hello , Which code-Analyzer or Code-Review tool do you suggest ,
For Analyzing DotNet 2.0 & 3.5 code and getting
All classes , Methods , Properties , Instances , Definition , Databases & and their relation to code ,
I want to get print-ready Information...
Started by Sypress on
, 4 posts
by 4 people.
Answer Snippets (Read the full thread at stackoverflow):
Doxygen can analyze your source code.
This may not give you the print-readiness you need.
The Code Metrics add-in will give you good quality metrics.
Maybe NDepend ?
Reflector is very good and free.
|
|
After reading this blog post , I'm thinking that it might be a good idea...occasionally.
the neurotic paranoid in me began to wonder how close of an eye they were putting on my code. Especially after a few times where I noticed bugs right before I checked...
Started by Thomas Owens on
, 16 posts
by 15 people.
Answer Snippets (Read the full thread at stackoverflow):
If you think people aren't reviewing the code when you practice into a game....
Emailing a few hundred or thousand lines of code to somebody and saying "Please review this" is never going to result in a thorough review.
|
|
I have a software engineer that I supervise and management needs a "code review" as part of his annual review. I'm thinking that there might be something out there that could help with this. Something along the lines of a list of categories and maybe ...
Started by Keltex on
, 9 posts
by 9 people.
Answer Snippets (Read the full thread at stackoverflow):
review' your management means what everybody else means, your engineer should present some of his code matter who picks the code, but your engineer should agree that it is ready for peer review and should soften excessively....
|
|
I'm just now learning to programming at age 17. It's hard for me to talk to other programmers as I'm just out of high school (which means I can't take programming courses). I know that I write terrible code, and not like Jeff Atwood terrible code , my...
Started by Bob Dylan on
, 8 posts
by 8 people.
Answer Snippets (Read the full thread at stackoverflow):
Get to see great code in action and learn from it Any changes you submit will be reviewed by an experienced developer and they will often give you excellent suggestions as to how to improve your code) and check out the code....
|
|
I want to better understand the patterns that work and the patterns that don't work for successful code reviews.
Should there be a difference be a Formal Code Review and an Informal Code Review?
What level of documentation of the Code Review do people...
Started by lillq on
, 8 posts
by 8 people.
Answer Snippets (Read the full thread at stackoverflow):
Only review code for which a design....
It is not suitable for that.
Of the reviewed code, and the review itself in fact!
And always remember to criticise the code review tool as a design review tool.
|