I agree with you. While being fancy when you're performing is a style of performance and ultimately the choice of the magician, there is such a thing as too much. Just as there's such thing as too simple. If it's too simple, you run the risk of making the effect boring to watch, as most of an effect is lead up and premise setting, they won't make it to the ending without tuning out. The journey between the start and the end have to be clear enough to follow and exciting/stimulating enough to keep you engaged.
This video is borderline, but not too fancy. I think this might become a growing problem in the generation of the hybrid flourisher/magicians
Agreed. I've seen videos where someone returns the card, and then to shuffle they do a sybil cut. Really? All those packets moving around does make for good eye candy, but it should be in between effects, not during one. To shuffle cards, simply shuffle the cards , either riffle or overhand. Both can be used for controls, or be false, or whatever. Plus they look completely innocent. Does a 5 packet display to "lose" the card in the deck seem suspicious? Yes. I used to do a "fancy" triple false cut when I first started, and I always got burned on it. Then I realized I'm doing too much, I need to keep it simple, and it's worked out tremendously for me.
If you want to show someone flourishes, do it to either gather attention and then do magic, or in between effects while you're talking to them show them some cool cuts. But don't incorporate it with the magic too much. Just my opinion, but the more simple and streamlined an effect, the better reactions is garners.