Writing functions for multiple buttons

Question

I'm pretty sure I'm doing this 100% wrong, but I have three buttons that toggle separate functions. I'm trying to determine the best way to write the functions in my controller.

The buttons:

<button ng-class="{selected:vm.selectedOption==='1'}" ng-click="vm.selectOption('1')">1</button>
<button ng-class="{selected:vm.selectedOption==='2'}" ng-click="vm.selectOption('2')">2</button>
<button ng-class="{selected:vm.selectedOption==='3'}" ng-click="vm.selectOption('3')">3</button>

My methods:

vm.selectOption = function(1) {
   // function
};
vm.selectOption = function(2) {
   // function
};
vm.selectOption = function(3) {
   // function
};

or

vm.selectOption = function(option) {
 if (vm.selectedOption==='1') {
   // function
 };
 if (vm.selectedOption==='2') {
   // function
 };
 if (vm.selectedOption==='3') {
   // function
 };
};

or I imagine there is a more correct way to do this that I'm not seeing.

Lex · Accepted Answer · 2016-06-03 22:39:39Z

2

You are passing the option to select to the function so:

vm.selectOption = function(option) {
    vm.selectedOption = option;
}

If you are looking to do something specific based on the option you just set:

vm.selectOption = function(option) {
    vm.selectedOption = option;
    switch(option) {
        case '1':
            // do something
            break;
        case '2':
            // do something else
            break;
        case '3':
            // do something completely different
            break;
    }
}

edited Jun 3, 2016 at 22:39

answered Jun 3, 2016 at 22:37

Lex

7,2732 gold badges31 silver badges49 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

nightowl Over a year ago

So, I understand that part, but how do I then set functions based on the different options?

Lex Over a year ago

If I'm understanding what you're asking then my update should apply. If not, could you explain a little more what you're after?

nightowl Over a year ago

Switch! I knew I was missing something obvious. This is exactly what I was looking for. Thank you!

Collectives™ on Stack Overflow

Writing functions for multiple buttons

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related