Question

Alright lets see if I can describe this.

I have a handful of functionality that was created sometime ago, and works swimmingly. However there is a new desired spec, so without having to rewrite the code base in a matter of speaking, and without having to double up on code to pull the same effect off I am trying to figure out how I can go about making something jump back higher in the code within the same function to repeat the run of the function rather then doing the same code again below.

What I have is a click based triggers ie:

$('.selector').click(function(){});

In this function is about 30 lines of functionality to create a new element and populate it accordingly. However unfortunately in that same bit of functionality there is conditions to wether it should or not.

*The previous requirement was when the element it creates is open and populated just throw an alert() saying essentially wrap up what your doing, and then go on to the next.

*Now the new requirement is just close that and open a new element.
Which I’ve gotten to close out the existing, and do everything I want it to do, except the population of the new element which is above where the condition is currently.

Knowing there is no “go to” type of logic in javascript (or last I knew). the only thing I can think of is taking the same code from above and putting it in the condition as well, doubling up on the code and having litterally 2 copies of the same bit. I want to avoid that, but cant think of a way to do it. So here I am looking for ideas

Answer 1

Knowing there is no “go to” type of logic in javascript (or last I
knew). the only thing I can think of is taking the same code from
above and putting it in the condition as well, doubling up on the code
and having litterally 2 copies of the same bit. I want to avoid that,
but cant think of a way to do it. So here I am looking for ideas

Why don’t you just pull this piece of code out into a function? You can run the function if the conditional is true in the original instance, and run it all the time in your callback? This is fairly minimal refactoring, just move the code out of the logic into a separate function, keeping it as is and maybe making some of the referenced variables into parameters.

So something like this if you want to run all the actions regardless of the conditional statements:

...
if(condition){
  actionA();
}
if(condition2){
  actionB();
}
...

$('.selector').click(function(){
    actionA();
    actionB();
});