Any time your code sample cannot fit on someone's monitor screen completely, the chances of getting an answer go way down. No one here gets paid to do this, so no one has much incentive to read through a lot of stuff. Try and keep your examples as brief as possible. Also, copy/paste text is better than a screenshot. Your code sample is too long, but the information in the screenshot is too short!
A superficial look at what you posted did ring some basic alarms.
1. Use ViewScoped or SessionScoped. One of the fundamental rules of
JSF is that in JSF Request Scope is almost 100% useless. It loses critical data when postbacks are done.
2. A JSF backing bean needs a no-element constructor. The JSF bean factory cannot supply arguments to constructors.
Hope that is of some help.
The secret of how to be miserable is to constantly expect things are going to happen the way that they are "supposed" to happen.
You can have faith, which carries the understanding that you may be disappointed. Then there's being a willfully-blind idiot, which virtually guarantees it.