How I solved code review process

12
225
software, testing, service
Spread the love

 

Table of Contents

How I solved code review process
Last day I wrote an article on Problems of any automation testing….http://www.askqtp.com/2012/01/problems-of-any-automation-testing.html
Hope you have enjoyed that. Today I am going to tell you that how my manager and I solved those problem.
We were trying hard , team meeting, team building exercise, effective training ….but all went on vain. One fine afternoon I found an interesting article while setting up a process for myself. It was 11 Best Practices for Peer Code Review — a White Paper from Smart Bear.
I strongly recommend you to go through the original 
 As per the author—The review should be efficient light weight–
           1.  Review fewer than 200-400 lines of code at a time. Study says review 200-400 lines of code at a time in 60-90 minutes yields 70-90% efficiency. This is efficient as per my views.
we implemented this technique.
           2.  Aim for an inspection rate of less than 300-500 LOC/hour–
As per author if inspection rate (how fast are reviewers able to review code?) is optimal if he targets 300-500 LOC/hour.As per this white paper defect rate (how fast reviewer is getting defects) will be high in this zone.But when I tried to implement the same, I found,200-250 LOC/hour for testing  of Code is more than enough. Rest of time reviewer can still find out and validate the logic. We implemented this technique.
           3.  Author should annotate source code-yes,I agree with author,reviewer should write comment–let me twist a bit here for testing, reviewer should comment what logic is written as a block  of code. If it passes
4 Eye principle,it is done.we implemented this technique with some modification.
           4.  Smart Bear’s whitepaper’s author made a point that reviewer should quantify the code review and capture matrics. This can be done if a dedicated team is involved.on the other hand I must advocate for a new dimention to this–the matrix should be LOC vs logic corrected while running these script will tally application with requirement and find bugs.we implemented this technique.
            5.  Checklists improve result :
Well,well,I really admit this point of the author. Even as per the author “A Checklist” will remind authors and reviewers to confirm omission of any important aspect. I was very much motivated when they wrote–we can develop our own checklist.Even PSI,SEI,EMMI recommand such personal checklist. When I tried to implement in my current process.It really turned best.It yields nice result. We implemented this technique.
            
6. Fix problem on the fly: 
I like the author’s point here, verify the defects that are actually fixed. Yes I implemented in two folds. Moreso when a requirement review is done we changed our automation guideline on the fly and when we find some problem in automation Script, We fixed them during review.we implemented this technique.

Reference:-PEER code review checklist at www.CodeReviewBook.com


7. Initially,everything  went against us. All the given problem data was used against us.
And our objective became to pass a script anyway.Yes,I strongly agree with author’s line. If developers believe that metrics will be used against them,not only will they be hostile to the process,but they will probably focus on improving matrics rather than bug tracking. It happened with us also.We made our point very clear. We encouraged and educated them. It worked. 

Last point is the EGO effect:-

                                 I agree with the author of this white paper but also agree that seniors of a process become very much egoistic if any problem found on their code.Once,I remember one of my fellow senior test engineer blamed the junior reviewer when his test cases failed to execute.
                                 And have seen several occasions when seniors feel insulted when his/her test script(code) is reviewed by a junior member of the group.
                                 I have seen people has to influence reviewer to make his/her piece of code as pass,even if these are going to fail in long run just because,he is senior,he is from the same region,he is from same culture,languages etc etc.
Well, my manager was there to handle this..:)

                                 So,with the help of these points we found out a better solution .It was a great review process indeed.

READ  What is User Acceptance Testing(UAT)?


Spread the love
I am the founder and owner of the blog - TechTravelHub.com, always love to share knowledge on test automation,tools,techniques and tips.I am a passionate coder of Java and VBScript.I also publish articles on Travel ideas and great honeymoon destinations.Apart from these, I am a gear-head,love to drive across India. I have shared lots of articles here on How to travel several parts of India.Customization of cars aka car modification is my another hobby.Get in touch with me on ani01104@gamil.com

12 COMMENTS

  1. I haven’t checked in here for some time because I thought it was getting boring, but the last few posts are great quality so I guess I will add you back to my daily bloglist. You deserve it my friend 🙂

  2. Great post. I was checking constantly this blog and I am impressed! Extremely helpful information specially the last part 🙂 I care for such info a lot. I was seeking this certain information for a very long time. Thank you and good luck.

  3. Good – I should definitely pronounce, impressed with your website. I had no trouble navigating through all tabs and related info ended up being truly simple to do to access. I recently found what I hoped for before you know it in the least. Quite unusual. Is likely to appreciate it for those who add forums or something, web site theme . a tones way for your client to communicate. Nice task..

  4. Hi! Someone in my Facebook group shared this site with us so I
    came to take a look. I’m definitely enjoying the information. I’m bookmarking and will be tweeting
    this to my followers! Fantastic blog and amazing style
    and design.

  5. Hiya very nice blog!! Man .. Excellent .. Wonderful ..

    I will bookmark your blog and take the feeds additionally?
    I’m happy to search out numerous helpful info right here within the put up, we want develop more strategies on this regard, thank you for sharing.
    . . . . .

  6. Link exchange is nothing else except it is simply placing the other person’s weblog link on your page at suitable
    place and other person will also do same in support of
    you.

  7. I loved this post! I read your blog fairly often and you are always coming out with some great
    stuff. I will shares this on my facebook, instagram and
    some of my loyalty followers. Great jobs! Keep work it with it.

LEAVE A REPLY

Please enter your comment!
Please enter your name here