First of all:
How To Post Your Trigger
As for the actual trigger:
1. What I posted above should be a separate trigger, because it's meant to run itself over and over again until the selection is completed. This means that everything else that you put in that trigger (such as creating units) would also be run multiple times, which you probably don't want.
2. You picked the wrong unit group in the block where you're comparing unit types - you should pick all units in NeFarstriderGroup, not FarstriderGroup.
3. As for picking every unit in FarstriderGroup, I'd put it in the second trigger below removing the NeFarstriderGroup (inside the If/Then/Else block).
4. You shouldn't destroy the unit group inside the Pick All Units in Unit Group loop. Also, be warned that if you're going to destroy FarstriderGroup, you're not going to have an empty group to transfer units to, i.e. this set of triggers will only work once. If you want it to work multiple times, I'd recommend simply removing all units from FarstriderGroup instead of destroying the group.
5. You're leaking points when creating units.
It's possible I missed something, so do try to fix the above first, test it and tell me if it works - if not, I'll take another look at it.