Good in what way? That's an intensive core routine, but it doesn't do much of anything for cardio, you arms, shoulders, chest, or even legs. Of course, it depends some on whether you are using any weights and how much those weights weigh.
I'd suggest that you consider any of the follow as changes:
1. Do 30 minutes of cardio before or after your workout. If after, warm up for 5 minutes.
2. Mix in a 2-minute high intensity cardio segment between each set: shuffle sprints, jumping jacks, high knee jogging, burpees, etc.
4. Include push ups and dips to get some arm, shoulder, and chest work in.
5. Drop either the crunches or sit-ups (or cut the number) and add lunges or other leg work.