Click here to Skip to main content
15,889,595 members

The Insider News

   

The Insider News is for breaking IT and Software development news. Post your news, your alerts and your inside scoops. This is an IT news-only forum - all off-topic, non-news posts will be removed. If you wish to ask a programming question please post it here.

Get The Daily Insider direct to your mailbox every day. Subscribe now!

 
GeneralRe: Practical lessons in peer code review Pin
Mark_Wallace28-Sep-14 3:12
Mark_Wallace28-Sep-14 3:12 
GeneralRe: Practical lessons in peer code review Pin
PIEBALDconsult24-Sep-14 13:15
mvePIEBALDconsult24-Sep-14 13:15 
GeneralRe: Practical lessons in peer code review Pin
Kent Sharkey24-Sep-14 13:27
staffKent Sharkey24-Sep-14 13:27 
GeneralRe: Practical lessons in peer code review Pin
PIEBALDconsult24-Sep-14 13:29
mvePIEBALDconsult24-Sep-14 13:29 
GeneralRe: Practical lessons in peer code review Pin
Marc Clifton24-Sep-14 13:27
mvaMarc Clifton24-Sep-14 13:27 
GeneralRe: Practical lessons in peer code review Pin
Bernhard Hiller24-Sep-14 21:13
Bernhard Hiller24-Sep-14 21:13 
GeneralRe: Practical lessons in peer code review Pin
Nelek24-Sep-14 23:52
protectorNelek24-Sep-14 23:52 
GeneralRe: Practical lessons in peer code review Pin
Marc Clifton25-Sep-14 3:49
mvaMarc Clifton25-Sep-14 3:49 
Bernhard Hiller wrote:
Does your statement mean that you are not at all interested in sharing your knowledge?


While I understand the reasons you quoted, they're sort of pie-in-the-sky wishful thinking. From my personal experiences, "familiarity with code" is really only achieved by working with the code and, more importantly, knowing the over-arching architectural decisions. I've found that I can show other people (even experienced devs) my code but, unless they are actively working with it, then its an "in one ear, out the other" experience. This stuff doesn't really stick. Certainly, that's my experience when someone shows me their code -- it only starts to stick when I have to work with it.

I've been down the path of helping to train new staff as well. The phrase "more experienced develoeprs" is a rabbit hole. Are the devs more experienced generally, for example in the framework or the language, or are they "merely" more experienced in the specific domain?

If the former, showing green devs complex code simply doesn't work. They need training, not show-and-tell. Ironically, green devs learn best by having people review their code, not my code, which results in a eye-glazing experience.

If the latter, then usually code is the worst way to learn about the business domain as it's often too narrow (and often biased) perspective of the business domain. New senior devs should be reading documentation, working with the product(s), and talking to both management and other devs about goals / issues. Another great technique for new senior devs is to participate in pair programming, that way they learn about how the architecture drives the code that supports the domain. A code review is far too disconnected from that process.

So, that's my hopefully more intelligent response to your very good question.

Marc

GeneralRe: Practical lessons in peer code review Pin
Member 861782925-Sep-14 13:03
Member 861782925-Sep-14 13:03 
GeneralRe: Practical lessons in peer code review Pin
Marc Clifton25-Sep-14 13:11
mvaMarc Clifton25-Sep-14 13:11 
NewsMicrosoft unveils Chromecast-like Wireless Display Adapter Pin
Kent Sharkey24-Sep-14 12:08
staffKent Sharkey24-Sep-14 12:08 
GeneralRe: Microsoft unveils Chromecast-like Wireless Display Adapter Pin
MacSpudster24-Sep-14 12:33
professionalMacSpudster24-Sep-14 12:33 
NewsUnix/Linux Bash: Critical security hole uncovered Pin
Kent Sharkey24-Sep-14 11:10
staffKent Sharkey24-Sep-14 11:10 
GeneralRe: Unix/Linux Bash: Critical security hole uncovered Pin
PIEBALDconsult24-Sep-14 11:32
mvePIEBALDconsult24-Sep-14 11:32 
GeneralRe: Unix/Linux Bash: Critical security hole uncovered Pin
Kent Sharkey24-Sep-14 11:46
staffKent Sharkey24-Sep-14 11:46 
GeneralRe: Unix/Linux Bash: Critical security hole uncovered Pin
Deflinek25-Sep-14 3:21
Deflinek25-Sep-14 3:21 
NewsAnalyst Watch: The 12 characteristics of modern application development Pin
Christopher Shields24-Sep-14 10:36
Christopher Shields24-Sep-14 10:36 
GeneralRe: Analyst Watch: The 12 characteristics of modern application development Pin
PIEBALDconsult24-Sep-14 11:00
mvePIEBALDconsult24-Sep-14 11:00 
GeneralRe: Analyst Watch: The 12 characteristics of modern application development Pin
Marc Clifton24-Sep-14 13:31
mvaMarc Clifton24-Sep-14 13:31 
GeneralRe: Analyst Watch: The 12 characteristics of modern application development Pin
Dan Neely25-Sep-14 3:02
Dan Neely25-Sep-14 3:02 
GeneralRe: Analyst Watch: The 12 characteristics of modern application development Pin
Nemanja Trifunovic25-Sep-14 5:16
Nemanja Trifunovic25-Sep-14 5:16 
NewsCortana: How Microsoft will bring Windows 9, Windows Phone and Bing closer together Pin
Christopher Shields24-Sep-14 10:29
Christopher Shields24-Sep-14 10:29 
GeneralRe: Cortana: How Microsoft will bring Windows 9, Windows Phone and Bing closer together Pin
Kent Sharkey24-Sep-14 11:00
staffKent Sharkey24-Sep-14 11:00 
GeneralRe: Cortana: How Microsoft will bring Windows 9, Windows Phone and Bing closer together Pin
tgrt24-Sep-14 11:24
tgrt24-Sep-14 11:24 
NewsTwo little words that destroy your credibility Pin
Kent Sharkey24-Sep-14 8:06
staffKent Sharkey24-Sep-14 8:06 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.