It seems like a pretty good routine at first glance. Truthfully, everything works.
The best workout program is the one you aren't currently doing. There is 'no magic' program. However, it's good to mix it up so your body keeps responding. Your body responds to different stimuli. If you keep training the same way, you will reach a plateau and your body will no longer keep responding.
Whenever a new client comes to me for training I ask for their current workout program. What I do then is send a workout program COMPLETELY different from their current workout program.
What a good workout program does is:
1. Pushes you out of your comfort zone
2. Gets you try something new
3. Makes you train HARD