addressalign-toparrow-leftarrow-rightbackbellblockcalendarcameraccwcheckchevron-downchevron-leftchevron-rightchevron-small-downchevron-small-leftchevron-small-rightchevron-small-upchevron-upcircle-with-checkcircle-with-crosscircle-with-pluscrossdots-three-verticaleditemptyheartexporteye-with-lineeyefacebookfolderfullheartglobegmailgooglegroupsimageimagesinstagramlinklocation-pinm-swarmSearchmailmessagesminusmoremuplabelShape 3 + Rectangle 1outlookpersonStartprice-ribbonImported LayersImported LayersImported Layersshieldstartickettrashtriangle-downtriangle-uptwitteruseryahoo

Re: [softwaredev-113] code review?

From: Jaz S.
Sent on: Tuesday, December 11, 2012 3:25 PM
Constructive peer code review is really high up there for code quality and improvement.  (Perhaps the #1 driver)

Both participants learns a lot from the discussion, and the feedback is really useful, if done in a constructive manner.
You also find a lot of issues in your code, when you are required to explain it to another person.  (Ie: Wow, I am embarrassed that I wrote that, I guess I should clean it up before the code review...)

I would recommend finding a way to do frequent and small reviews, as opposed to huge code reviews.
As far as the mechanism, I am unsure.  As the only developer, who do you go to do a review? (Proprietary code is an issue)



On Tue, Dec 11, 2012 at 3:13 PM, Ryan <[address removed]> wrote:
What are thoughts on a code review? I am the only developer and can always use some feedback, esp with some of my recent code. I am sure that others feel similarly.

Input?

--
Ryan Brown
Bioinformaticist in Department of Psychiatry
University of Wisconsin - Madison





--
Please Note: If you hit "REPLY", your message will be sent to everyone on this mailing list ([address removed])
This message was sent by Ryan ([address removed]) from Madison Area Software Developers Meetup Group.
To learn more about Ryan, visit his/her member profile
Set my mailing list to email me As they are sent | In one daily email | Don't send me mailing list messages

Meetup, PO Box 4668 #37895 New York, New York[masked] | [address removed]

Our Sponsors

People in this
Meetup are also in:

Sign up

Meetup members, Log in

By clicking "Sign up" or "Sign up using Facebook", you confirm that you accept our Terms of Service & Privacy Policy