Tag Info

New answers tagged

3

Getting code reviews completely promptly is a management issue, not a technical issue. That said, and I'll acknowledge this will generate howls of protest, the best way to ensure timely completion is to make timely completion a part of their performance review process. You'll need / want to have tooling support in order to make it work. But with a modest ...


0

This has not actually been placed in an answer yet (but has been added as a comment to an existing answer) Review your code after a good nights sleep, e.g. start the day by reviewing the code that you wrote the previous day. This will of course not give you the collective experience of a team, but it will give let you review the code from a new ...


12

One of the key resources for Extreme Programming is that of Ward's Wiki aka Portland Pattern Repository aka C2.com. This is where a number of people hashed out various methodologies and documented them as they used them. Within this wiki, there is a page: Extreme Programming Code Reviews that has a number of contributors to it, including Ron Jeffries and ...


-1

I would use AutoProperties only for DTO where you just create object to transfer between boundaries and without any behaviour. My experience with auto properties are: No way to create readonly fields! -> Solved in C# vNext Developers do not think about encapsulation. They tend to make to much public Properties force you to create to much mutable state ...


6

IMO, Manual code reviews helps in forming a shared understanding of the codebase, reducing dependency on a single or a small group of individuals. It may also teach a thing or two in programming style/language features for the reviewer reviewing the code. It works great if everyone reviews every other person's code or pair with someone senior who is good at ...


15

Yes, you should keep code reviews. SonarQube, like any other automated tool, is great as a tool which helps pointing out some of the possible problems, including problems which are not easily seen by humans. Duplicate code, for example, can be checked automatically, as shown by Visual Studio or SonarQube, while a reviewer may not easily see that the same ...



Top 50 recent answers are included