[Trigger] 2 option Vote system

[Bond009]

Im kinda looking through some old triggers i made and figured i'd revise some a bit.

does this look semi efficient or could it be made a lot better?

i set up the dialog then run a small wait till i run the Tally trigger to give time to player to vote yes or no.

for the yes option i do nothing.
the no button i add +1

• Vote No SSD
  • Events
    • Dialog - A dialog button is clicked for Dialog_All
  • Conditions
    • (Clicked dialog button) Equal to Dialog_All_Buttons[2]
  • Actions
    • Game - Display to (All players) for 8.00 seconds the text: |cffBB1100- Voted N...
    • Set Vote_Count[1] = (Vote_Count[1] + 1)
    • Dialog - Hide Dialog_All for (Triggering player)

• Take Vote Tally All
  • Events
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 12, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • ((Player((Integer A))) controller) Equal to User
            • ((Player((Integer A))) slot status) Equal to Is playing
          • Then - Actions
            • Set Vote_Count[2] = (Vote_Count[2] + 1)
          • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • Vote_Count[2] Equal to 1
            • Vote_Count[2] Equal to 3
            • Vote_Count[2] Equal to 5
            • Vote_Count[2] Equal to 7
            • Vote_Count[2] Equal to 9
            • Vote_Count[2] Equal to 11
      • Then - Actions
        • Set Vote_Count[2] = (Vote_Count[2] + 1)
      • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Vote_Count[1] Greater than or equal to (Vote_Count[2] / 2)
      • Then - Actions
        • Game - Display to (All players) for 9.00 seconds the text: |c00FF0000Majority ...
      • Else - Actions
        • Game - Display to (All players) for 9.00 seconds the text: |c00FF0000Majority ...
        • Trigger - Run Trigger <gen> (ignoring conditions)
    • Set Vote_Count[1] = 0
    • Set Vote_Count[2] = 0

this works perfectly for what i use it for.. just wondering if it could be made alot better or not.

 

[Bond009]

lol so i take it this is good enough then..? no one can make this better