It's a good move. The material trade is about even but your two minor pieces are doing more in the middle game than the computer's rook and pawn. In such situations you shoudn't take on f7 unless you can see a way to continue the attack.
Sacrificing Rook for bishop and knight?

well ,just a give imagine thats ur question is just knight and bishop for 1 rook < i would conside say that u should not take give 1bishop and 1knight for 1 rook ..well we can count this in such following way <> 1Knight + 1 Bishop = 6 points ... but 1 rook = just 5 points ... did u see this ? by the way , bishop and knight are best friends in middle game .. try to check the value of each piece when u trade off ! and remember , your oppenets are pesky.. they would trade if they cant hold on your powerfull attack ... so, dont simply use the trading method for all of games .. Tq

Rook vs. Bishop and Knight is playable. On the endgames, the rook become a powerful piece, while the bishop and the knight can make more pressure.
It's something like this:
Advantage Rook Bishop and Knight
Can exchange pieces x ok
Can work on two sides of
board x ok
Can lock the opponent's
king easier ok x
Pawn hunting easier harder
Pressure minor major (two pieces)
Mobility major minor
On general, since Bishop and Knight are two pieces and Rook is only one, I prefer the BK setup. In fact, the lonely Rook have a slight disadvantage on this case.
Over all, taking the Bxf7+ followed by Nxf7 is, in most cases, loss of material.
I played against Computer1-EASY for a few Standard games and I realized that in 2 of the games I played Bxf7+ while their King had castled and I had a knight supporting that bishop. However, the computer always replied with Rxf7, losing a rook and a pawn to a bishop and knight. Is that considered a good move or did the computer make a bad move?
See the two games where the computer made that move:
http://www.chess.com/livechess/game?id=914860733
http://www.chess.com/livechess/game?id=914866515