Das Makro funktioniert natürlich, allerdings würde ich die Bedingung für den feindlichen Zauber einfach weglassen. Wenn ein Ziel nicht freundlich gesinnt ist, dann kann es dabei nur feindlich gesinnt sein, wodurch eine weitere, spezifische Abfrage dafür nicht mehr nötig ist. 
Sollte es doch irgendein Ziel geben, das weder freundlich noch feindlich ist, dann würde dein Makro einfach überhaupt nichts anzeigen, da für diesen Fall keine Aktion definiert ist.
Ich würde das also eher so machen, auch wenn es im Endeffekt auf das gleiche rauskommen wird:
#showtooltip
/cast [help] ZauberFürFreundlicheZiele; ZauberFürFeindlicheZiele
Ansonsten gibt es hier im Forum auch einen deutschen Makro-Guide, der leider kein Sticky ist, da dieser seit einiger Zeit nicht mehr aktualisiert wurde. Bis auf einige wenige Details ist dieser aber immer noch absolut zutreffend: