No, it's a terrible idea. Based on that piece of code the following thoughts come up to my mind:
- This line is commented out because the developer was debugging it and forgot restore the line to its former state
- This line is commented out because it once was part of the business logic, but it is no longer the case
- This line is commented out because it caused performance problems on production and the developer wanted to see what the impact was on a production system
After seeing thousands of lines of commented out code, I'm now doing the only sensible thing when I see it: I immediately remove it.
There is no sensible reason to check in commented out code into a repository.
Also, your code uses a lot of duplication. I suggest you optimize that away for human readability as soon as possible.
