Looks like a pretty typical first AMV to me. I'm not real crazy about using the OPs/EDs...but that's just me. Now that I think about it...it might be a good way to learn.
I won't go on and on, but there's a few points I'll mention.
There was some sync, but most of the action and transitions weren't synced to anything. There's more than one way to sync, but for your next amv I'd recommend getting familiar with beat sync.
Try to make your cuts/fades/other transitions happen on the beat of the music. It doesn't have to be the beat, but the beat is usually the easiest to find. From there, you can start syncing the action in your clips to the beat (or some other element in the music, like guitar or synth)
Try to match the motion in your clips to the pace of the song. Slow phrases in the music tend to work better with clips that have slower motion. Likewise, faster phrases in the music tend to work with clips with more/faster motion.