KingYes/phpguide — comment on commit d2654c70af834e2f75a0f8a18d6010b2aa444d6f
File:  line 0
By intval:

Very good job ;)

Please consider the following:
-------------------------------------------

2. Your coding level has increased. I'm proud of you ;) keep it up. pretty fast you'll become a tough coder ;)

3. When you write a new method, please add PHPDOC comments to it, starting with `/**`

4. I'm not sure what is the last parameter $force doing, since nobody ever uses it and the name is not very self-explanatory.
Renaming it into $addToVisitedList might help better understand.
Readable, understandable code values lot more than short code, full of `$i`, `$j`, `$k` ;)

5. Since the only place in the project to use this method is in QNA, there is no good reason to move it out to the Helpers class. I know you have been thinking about the future, where visit counts may be required in another places, but there is a principle in software development saying : keep it as easy as possible and don't overcomplicate it before actually needed. You will always be able to move it out to helpers in a later stage.
For now it's best place would be as a method of the qnacontroller class itself.

*on the bottom line:* I suggest moving this method to be part of qnaController class, making sure you give it a name *fully* describing it's *entire* functionality, because now it doesn't only check, but adds stuff. This means function's name is lying ;)

6. methods returning booleans should start with *is*. `isAdmin()`, `isViewable`, `isQuestionViewedEarlier`. same goes about variables