7/10, only reason its not a ten out of ten is because of a few small things
.make camera scroll ahead a little earlier (you dont have a lot of time to react to enemies)
.make the animation for the attack only play once.
easy way to do this is by adding a variable when the character is created called "cooldown" and making it equal 0. then when the character attacks you can check to see if "cooldown = 0" and after the player attacks you can make cooldown = 1 and then set an alarm (if your game runs at 60 frames id say put the alarm to 60 or 90) then when the alarm is over make it set the cooldown variable back to 0.
that will basically make it so your only able to attack after a second of cooldown and will make it so the animation cant play till the cooldown is back to 0
Also i started following you, it can be difficult to program games and gain a following, i myself have only just started, so i wanted to follow you and see what you work on next
p.s.
Check out Shaun Spalding's platforming series on youtube, its pretty long but teaches you basically EVERYTHING you need to know on a beginners level :)