Vote Details for "PagamentoCerto" by shupp

» Details
  • Voter: Bill Shupp 
  • Vote: +1 (not conditional)
  • Reviews: Cursory source review
» Comment
Very nice code. I do agree with the other vote comments, and would like to add a couple of my own:

1) I think employing the pattern Michael recommended in his proposal comment for early returns / Exceptions will rid your code of a *lot* of else statements.

2) Why do you have an empty constructor in Payment_PagamentoCerto_Order? It's not overwriting anything. Just remove it.