I would suggest it is not good practice to set the NOTCOUNT to OFF at the end of the procedure. Firstly it is not necessary as it is scope based, and secondly there is no way of telling what it was before we set it to NOCOUNT ON so arbitrarily setting it to the default OFF could actually be misleading as well as being totally unnecessary and bloating the code.