Should I ask my project manager to review my work?

The name of the pictureThe name of the pictureThe name of the pictureClash Royale CLAN TAG#URR8PPP





.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty margin-bottom:0;







up vote
3
down vote

favorite












I completed my coding part in our project and I would like to have a code review with my overseas project manager (who is also involved in code development). But I'm a little scared of asking him because I am a fresher.



I think that a code review could help for two reasons:



  1. A senior programmer might catch errors.

  2. The exercise will serve as a mentoring/learning experience and help me grow.

But on the other hand:



  1. I don't want the project manager to think that I lack confidence in my code.

  2. And I don't want the project manager to lose confidence in me. In the future, I want more interesting, engaging work that will help me grow.


While my specific situation applies to programming, I'm wondering more generally:



  • Should I ask my project manager to review my work?






share|improve this question






















  • I've flagged this for migration to programmers given how specific it is to coding.
    – Elysian Fields♦
    May 24 '13 at 14:40






  • 4




    @enderland I don't see what's specific to coding here, to me this reads more like a question on how to approach one's manager (It's a "people problem", not a coding problem). user1671639 isn't asking if code reviews are a good practice, they already know it is. And in case I'm reading this wrong and it's indeed a coding problem, we already have a very good duplicate: Is code reviewing good practice?.
    – yannis
    May 24 '13 at 15:20







  • 2




    Fully agreed with @Yannis on this one. "Should my boss check my work?" is a universal question that has value in almost any type of job you will ever have.
    – jmac
    May 26 '13 at 23:20
















up vote
3
down vote

favorite












I completed my coding part in our project and I would like to have a code review with my overseas project manager (who is also involved in code development). But I'm a little scared of asking him because I am a fresher.



I think that a code review could help for two reasons:



  1. A senior programmer might catch errors.

  2. The exercise will serve as a mentoring/learning experience and help me grow.

But on the other hand:



  1. I don't want the project manager to think that I lack confidence in my code.

  2. And I don't want the project manager to lose confidence in me. In the future, I want more interesting, engaging work that will help me grow.


While my specific situation applies to programming, I'm wondering more generally:



  • Should I ask my project manager to review my work?






share|improve this question






















  • I've flagged this for migration to programmers given how specific it is to coding.
    – Elysian Fields♦
    May 24 '13 at 14:40






  • 4




    @enderland I don't see what's specific to coding here, to me this reads more like a question on how to approach one's manager (It's a "people problem", not a coding problem). user1671639 isn't asking if code reviews are a good practice, they already know it is. And in case I'm reading this wrong and it's indeed a coding problem, we already have a very good duplicate: Is code reviewing good practice?.
    – yannis
    May 24 '13 at 15:20







  • 2




    Fully agreed with @Yannis on this one. "Should my boss check my work?" is a universal question that has value in almost any type of job you will ever have.
    – jmac
    May 26 '13 at 23:20












up vote
3
down vote

favorite









up vote
3
down vote

favorite











I completed my coding part in our project and I would like to have a code review with my overseas project manager (who is also involved in code development). But I'm a little scared of asking him because I am a fresher.



I think that a code review could help for two reasons:



  1. A senior programmer might catch errors.

  2. The exercise will serve as a mentoring/learning experience and help me grow.

But on the other hand:



  1. I don't want the project manager to think that I lack confidence in my code.

  2. And I don't want the project manager to lose confidence in me. In the future, I want more interesting, engaging work that will help me grow.


While my specific situation applies to programming, I'm wondering more generally:



  • Should I ask my project manager to review my work?






share|improve this question














I completed my coding part in our project and I would like to have a code review with my overseas project manager (who is also involved in code development). But I'm a little scared of asking him because I am a fresher.



I think that a code review could help for two reasons:



  1. A senior programmer might catch errors.

  2. The exercise will serve as a mentoring/learning experience and help me grow.

But on the other hand:



  1. I don't want the project manager to think that I lack confidence in my code.

  2. And I don't want the project manager to lose confidence in me. In the future, I want more interesting, engaging work that will help me grow.


While my specific situation applies to programming, I'm wondering more generally:



  • Should I ask my project manager to review my work?








share|improve this question













share|improve this question




share|improve this question








edited May 24 '13 at 15:58









gnat

3,23473066




3,23473066










asked May 24 '13 at 9:28









Gotcha

1661210




1661210











  • I've flagged this for migration to programmers given how specific it is to coding.
    – Elysian Fields♦
    May 24 '13 at 14:40






  • 4




    @enderland I don't see what's specific to coding here, to me this reads more like a question on how to approach one's manager (It's a "people problem", not a coding problem). user1671639 isn't asking if code reviews are a good practice, they already know it is. And in case I'm reading this wrong and it's indeed a coding problem, we already have a very good duplicate: Is code reviewing good practice?.
    – yannis
    May 24 '13 at 15:20







  • 2




    Fully agreed with @Yannis on this one. "Should my boss check my work?" is a universal question that has value in almost any type of job you will ever have.
    – jmac
    May 26 '13 at 23:20
















  • I've flagged this for migration to programmers given how specific it is to coding.
    – Elysian Fields♦
    May 24 '13 at 14:40






  • 4




    @enderland I don't see what's specific to coding here, to me this reads more like a question on how to approach one's manager (It's a "people problem", not a coding problem). user1671639 isn't asking if code reviews are a good practice, they already know it is. And in case I'm reading this wrong and it's indeed a coding problem, we already have a very good duplicate: Is code reviewing good practice?.
    – yannis
    May 24 '13 at 15:20







  • 2




    Fully agreed with @Yannis on this one. "Should my boss check my work?" is a universal question that has value in almost any type of job you will ever have.
    – jmac
    May 26 '13 at 23:20















I've flagged this for migration to programmers given how specific it is to coding.
– Elysian Fields♦
May 24 '13 at 14:40




I've flagged this for migration to programmers given how specific it is to coding.
– Elysian Fields♦
May 24 '13 at 14:40




4




4




@enderland I don't see what's specific to coding here, to me this reads more like a question on how to approach one's manager (It's a "people problem", not a coding problem). user1671639 isn't asking if code reviews are a good practice, they already know it is. And in case I'm reading this wrong and it's indeed a coding problem, we already have a very good duplicate: Is code reviewing good practice?.
– yannis
May 24 '13 at 15:20





@enderland I don't see what's specific to coding here, to me this reads more like a question on how to approach one's manager (It's a "people problem", not a coding problem). user1671639 isn't asking if code reviews are a good practice, they already know it is. And in case I'm reading this wrong and it's indeed a coding problem, we already have a very good duplicate: Is code reviewing good practice?.
– yannis
May 24 '13 at 15:20





2




2




Fully agreed with @Yannis on this one. "Should my boss check my work?" is a universal question that has value in almost any type of job you will ever have.
– jmac
May 26 '13 at 23:20




Fully agreed with @Yannis on this one. "Should my boss check my work?" is a universal question that has value in almost any type of job you will ever have.
– jmac
May 26 '13 at 23:20










3 Answers
3






active

oldest

votes

















up vote
5
down vote













Asking a senior developer (or even just another developer) to review your code is a good thing - most companies have a formalised process for this.



Asking a project manager seems odd especially if they don't do any day to day development.






share|improve this answer




















  • My project manager also involved in code development.
    – Gotcha
    May 24 '13 at 10:17











  • Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
    – Michael
    May 24 '13 at 11:00

















up vote
1
down vote













Focus on what is good for your company, and you'll get pretty far. If getting a code review is right, ask for it.



Just remember that your view of the situation may be skewed because it can be hard to compare "getting a review" against "not getting a review" because "not getting a review" actually means that you'll free up resources in your organization to do something else that you can't estimate the value of.



But anyway, if your peer seems to have time and you believe it adds value, go ahead. Asking for a review can just as well be a sign of confidence - not lack thereof.






share|improve this answer



























    up vote
    1
    down vote













    The project manager is assigned to get the promised deliverable on time. A good project manager will also ensure the deliverable is of the best quality possible without further delaying the project and without asking the team members to put in more resources than the project plan!
    Since your project manager wears the 'Developer's hat' too, and I am guessing (in your company) He (the project manager) has a long experience being a developer then rose up to the rank/position of the project manager, so he is an experienced developer as well. If that's how he rose up to the rank in your company and thats how your company interprets 'project manager' too then you be fine running your code through him. It doesn't suggest that you lack confidence in your code, rather suggests that you don't hesitate to ask questions even on your own work in pursuit of a higher quality deliverable. Also, research about the logic blocks you are coding and see what are other practices over the internet. During the code review, suggest those different practices to your 'project manager' to show that you are well aware of how else it could be done, just didn't know what would best suit your organization in the longer run. This will suggest that you have a wide perspective and can think out of the box to find the best route to the solution. A good skill for a programmer.



    But usually (by PMI conventions) a project manager is a temporary position that a fellow takes up for the term of the project and he is given the hat to manage the project, not necessarily to make sure that the deliverable are of the utmost quality. There are other people to take care of that, even peers of the developers.






    share|improve this answer




















      Your Answer







      StackExchange.ready(function()
      var channelOptions =
      tags: "".split(" "),
      id: "423"
      ;
      initTagRenderer("".split(" "), "".split(" "), channelOptions);

      StackExchange.using("externalEditor", function()
      // Have to fire editor after snippets, if snippets enabled
      if (StackExchange.settings.snippets.snippetsEnabled)
      StackExchange.using("snippets", function()
      createEditor();
      );

      else
      createEditor();

      );

      function createEditor()
      StackExchange.prepareEditor(
      heartbeatType: 'answer',
      convertImagesToLinks: false,
      noModals: false,
      showLowRepImageUploadWarning: true,
      reputationToPostImages: null,
      bindNavPrevention: true,
      postfix: "",
      noCode: true, onDemand: false,
      discardSelector: ".discard-answer"
      ,immediatelyShowMarkdownHelp:true
      );



      );








       

      draft saved


      draft discarded


















      StackExchange.ready(
      function ()
      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fworkplace.stackexchange.com%2fquestions%2f11962%2fshould-i-ask-my-project-manager-to-review-my-work%23new-answer', 'question_page');

      );

      Post as a guest

























      StackExchange.ready(function ()
      $("#show-editor-button input, #show-editor-button button").click(function ()
      var showEditor = function()
      $("#show-editor-button").hide();
      $("#post-form").removeClass("dno");
      StackExchange.editor.finallyInit();
      ;

      var useFancy = $(this).data('confirm-use-fancy');
      if(useFancy == 'True')
      var popupTitle = $(this).data('confirm-fancy-title');
      var popupBody = $(this).data('confirm-fancy-body');
      var popupAccept = $(this).data('confirm-fancy-accept-button');

      $(this).loadPopup(
      url: '/post/self-answer-popup',
      loaded: function(popup)
      var pTitle = $(popup).find('h2');
      var pBody = $(popup).find('.popup-body');
      var pSubmit = $(popup).find('.popup-submit');

      pTitle.text(popupTitle);
      pBody.html(popupBody);
      pSubmit.val(popupAccept).click(showEditor);

      )
      else
      var confirmText = $(this).data('confirm-text');
      if (confirmText ? confirm(confirmText) : true)
      showEditor();


      );
      );






      3 Answers
      3






      active

      oldest

      votes








      3 Answers
      3






      active

      oldest

      votes









      active

      oldest

      votes






      active

      oldest

      votes








      up vote
      5
      down vote













      Asking a senior developer (or even just another developer) to review your code is a good thing - most companies have a formalised process for this.



      Asking a project manager seems odd especially if they don't do any day to day development.






      share|improve this answer




















      • My project manager also involved in code development.
        – Gotcha
        May 24 '13 at 10:17











      • Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
        – Michael
        May 24 '13 at 11:00














      up vote
      5
      down vote













      Asking a senior developer (or even just another developer) to review your code is a good thing - most companies have a formalised process for this.



      Asking a project manager seems odd especially if they don't do any day to day development.






      share|improve this answer




















      • My project manager also involved in code development.
        – Gotcha
        May 24 '13 at 10:17











      • Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
        – Michael
        May 24 '13 at 11:00












      up vote
      5
      down vote










      up vote
      5
      down vote









      Asking a senior developer (or even just another developer) to review your code is a good thing - most companies have a formalised process for this.



      Asking a project manager seems odd especially if they don't do any day to day development.






      share|improve this answer












      Asking a senior developer (or even just another developer) to review your code is a good thing - most companies have a formalised process for this.



      Asking a project manager seems odd especially if they don't do any day to day development.







      share|improve this answer












      share|improve this answer



      share|improve this answer










      answered May 24 '13 at 9:41









      Michael

      4,7461423




      4,7461423











      • My project manager also involved in code development.
        – Gotcha
        May 24 '13 at 10:17











      • Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
        – Michael
        May 24 '13 at 11:00
















      • My project manager also involved in code development.
        – Gotcha
        May 24 '13 at 10:17











      • Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
        – Michael
        May 24 '13 at 11:00















      My project manager also involved in code development.
      – Gotcha
      May 24 '13 at 10:17





      My project manager also involved in code development.
      – Gotcha
      May 24 '13 at 10:17













      Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
      – Michael
      May 24 '13 at 11:00




      Then in this particular instance, don't think of him as your project manager, think of him as wearing the developer hat and he's just another developer.
      – Michael
      May 24 '13 at 11:00












      up vote
      1
      down vote













      Focus on what is good for your company, and you'll get pretty far. If getting a code review is right, ask for it.



      Just remember that your view of the situation may be skewed because it can be hard to compare "getting a review" against "not getting a review" because "not getting a review" actually means that you'll free up resources in your organization to do something else that you can't estimate the value of.



      But anyway, if your peer seems to have time and you believe it adds value, go ahead. Asking for a review can just as well be a sign of confidence - not lack thereof.






      share|improve this answer
























        up vote
        1
        down vote













        Focus on what is good for your company, and you'll get pretty far. If getting a code review is right, ask for it.



        Just remember that your view of the situation may be skewed because it can be hard to compare "getting a review" against "not getting a review" because "not getting a review" actually means that you'll free up resources in your organization to do something else that you can't estimate the value of.



        But anyway, if your peer seems to have time and you believe it adds value, go ahead. Asking for a review can just as well be a sign of confidence - not lack thereof.






        share|improve this answer






















          up vote
          1
          down vote










          up vote
          1
          down vote









          Focus on what is good for your company, and you'll get pretty far. If getting a code review is right, ask for it.



          Just remember that your view of the situation may be skewed because it can be hard to compare "getting a review" against "not getting a review" because "not getting a review" actually means that you'll free up resources in your organization to do something else that you can't estimate the value of.



          But anyway, if your peer seems to have time and you believe it adds value, go ahead. Asking for a review can just as well be a sign of confidence - not lack thereof.






          share|improve this answer












          Focus on what is good for your company, and you'll get pretty far. If getting a code review is right, ask for it.



          Just remember that your view of the situation may be skewed because it can be hard to compare "getting a review" against "not getting a review" because "not getting a review" actually means that you'll free up resources in your organization to do something else that you can't estimate the value of.



          But anyway, if your peer seems to have time and you believe it adds value, go ahead. Asking for a review can just as well be a sign of confidence - not lack thereof.







          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered May 25 '13 at 21:16









          Michael Zedeler

          42723




          42723




















              up vote
              1
              down vote













              The project manager is assigned to get the promised deliverable on time. A good project manager will also ensure the deliverable is of the best quality possible without further delaying the project and without asking the team members to put in more resources than the project plan!
              Since your project manager wears the 'Developer's hat' too, and I am guessing (in your company) He (the project manager) has a long experience being a developer then rose up to the rank/position of the project manager, so he is an experienced developer as well. If that's how he rose up to the rank in your company and thats how your company interprets 'project manager' too then you be fine running your code through him. It doesn't suggest that you lack confidence in your code, rather suggests that you don't hesitate to ask questions even on your own work in pursuit of a higher quality deliverable. Also, research about the logic blocks you are coding and see what are other practices over the internet. During the code review, suggest those different practices to your 'project manager' to show that you are well aware of how else it could be done, just didn't know what would best suit your organization in the longer run. This will suggest that you have a wide perspective and can think out of the box to find the best route to the solution. A good skill for a programmer.



              But usually (by PMI conventions) a project manager is a temporary position that a fellow takes up for the term of the project and he is given the hat to manage the project, not necessarily to make sure that the deliverable are of the utmost quality. There are other people to take care of that, even peers of the developers.






              share|improve this answer
























                up vote
                1
                down vote













                The project manager is assigned to get the promised deliverable on time. A good project manager will also ensure the deliverable is of the best quality possible without further delaying the project and without asking the team members to put in more resources than the project plan!
                Since your project manager wears the 'Developer's hat' too, and I am guessing (in your company) He (the project manager) has a long experience being a developer then rose up to the rank/position of the project manager, so he is an experienced developer as well. If that's how he rose up to the rank in your company and thats how your company interprets 'project manager' too then you be fine running your code through him. It doesn't suggest that you lack confidence in your code, rather suggests that you don't hesitate to ask questions even on your own work in pursuit of a higher quality deliverable. Also, research about the logic blocks you are coding and see what are other practices over the internet. During the code review, suggest those different practices to your 'project manager' to show that you are well aware of how else it could be done, just didn't know what would best suit your organization in the longer run. This will suggest that you have a wide perspective and can think out of the box to find the best route to the solution. A good skill for a programmer.



                But usually (by PMI conventions) a project manager is a temporary position that a fellow takes up for the term of the project and he is given the hat to manage the project, not necessarily to make sure that the deliverable are of the utmost quality. There are other people to take care of that, even peers of the developers.






                share|improve this answer






















                  up vote
                  1
                  down vote










                  up vote
                  1
                  down vote









                  The project manager is assigned to get the promised deliverable on time. A good project manager will also ensure the deliverable is of the best quality possible without further delaying the project and without asking the team members to put in more resources than the project plan!
                  Since your project manager wears the 'Developer's hat' too, and I am guessing (in your company) He (the project manager) has a long experience being a developer then rose up to the rank/position of the project manager, so he is an experienced developer as well. If that's how he rose up to the rank in your company and thats how your company interprets 'project manager' too then you be fine running your code through him. It doesn't suggest that you lack confidence in your code, rather suggests that you don't hesitate to ask questions even on your own work in pursuit of a higher quality deliverable. Also, research about the logic blocks you are coding and see what are other practices over the internet. During the code review, suggest those different practices to your 'project manager' to show that you are well aware of how else it could be done, just didn't know what would best suit your organization in the longer run. This will suggest that you have a wide perspective and can think out of the box to find the best route to the solution. A good skill for a programmer.



                  But usually (by PMI conventions) a project manager is a temporary position that a fellow takes up for the term of the project and he is given the hat to manage the project, not necessarily to make sure that the deliverable are of the utmost quality. There are other people to take care of that, even peers of the developers.






                  share|improve this answer












                  The project manager is assigned to get the promised deliverable on time. A good project manager will also ensure the deliverable is of the best quality possible without further delaying the project and without asking the team members to put in more resources than the project plan!
                  Since your project manager wears the 'Developer's hat' too, and I am guessing (in your company) He (the project manager) has a long experience being a developer then rose up to the rank/position of the project manager, so he is an experienced developer as well. If that's how he rose up to the rank in your company and thats how your company interprets 'project manager' too then you be fine running your code through him. It doesn't suggest that you lack confidence in your code, rather suggests that you don't hesitate to ask questions even on your own work in pursuit of a higher quality deliverable. Also, research about the logic blocks you are coding and see what are other practices over the internet. During the code review, suggest those different practices to your 'project manager' to show that you are well aware of how else it could be done, just didn't know what would best suit your organization in the longer run. This will suggest that you have a wide perspective and can think out of the box to find the best route to the solution. A good skill for a programmer.



                  But usually (by PMI conventions) a project manager is a temporary position that a fellow takes up for the term of the project and he is given the hat to manage the project, not necessarily to make sure that the deliverable are of the utmost quality. There are other people to take care of that, even peers of the developers.







                  share|improve this answer












                  share|improve this answer



                  share|improve this answer










                  answered May 26 '13 at 13:47









                  camelbrush

                  1316




                  1316






















                       

                      draft saved


                      draft discarded


























                       


                      draft saved


                      draft discarded














                      StackExchange.ready(
                      function ()
                      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fworkplace.stackexchange.com%2fquestions%2f11962%2fshould-i-ask-my-project-manager-to-review-my-work%23new-answer', 'question_page');

                      );

                      Post as a guest

















































































                      Comments

                      Popular posts from this blog

                      What does second last employer means? [closed]

                      Installing NextGIS Connect into QGIS 3?

                      One-line joke