1

Есть функция, не важно какая, главное, что в ней есть условие if (a>b) : см.

function ab(x) {
    var a, b;
    if (a > b) {
        // блок кода
    }
    if (a < b) {
        // тот же самый блок кода
    }
}

Смысл в том, чтобы не копировать один и тот же кусок кода по 10 раз. Это возможно ли как-нибудь сократить?

3
  • я этого и хочу достичь
    – Smash
    21 мар 2012 в 20:54
  • Что-бы этого достичь нужно видеть весь код, а не то, что предоставили вы
    – user3545
    21 мар 2012 в 20:56
  • читайте комменты ответа AlexWindHope
    – Smash
    21 мар 2012 в 21:10

4 ответа 4

5

Возможно я чего то не понимаю, но 2 условия: a>b и a<b

можно записать:

function ab(x){
    var a,b;
    if (a != b) {
       // блок кода
    }
}

Если же вы привели это просто для иллюстрации, то воспользуйтесь вариантом: @AlexWindHope т.е через функцию.

1
  • ну вы тоже в чем-то правы, но в моей глобальной задаче это не подойдет.
    – Smash
    21 мар 2012 в 22:15
4

.

var ALowerThenB = a > b;
if( ALowerThenB ) {/* code */}
if( ALowerThenB ) {/* code */}

А вообще, если это возможно - лучше всего продумать код так, что-бы поместить всю логику в 1о условие.

Возможно вы имели введу это:

function someFunc() {
    var a,b;
    // если кода много - можно сделать функцией, что-то типа
    function test(){
        if( a < b ) {
            // тут много много вашего кода
        }
    }
    // соответственно вызов сводиться к вызову функции
    test();
}
10
  • вот это уже более подходящий вариант
    – Smash
    21 мар 2012 в 21:00
  • если интересно, немного jsfiddle, это я сортировку массива хочу немного разнообразить, чтобы не только по возр., но и по убыванию было, правда еще 1-ну переменную нужно добавить на вход функции, но пока так: jsfiddle.net/Stasonix/h4fNN/2
    – Smash
    21 мар 2012 в 21:04
  • Если правильно понял - добавьте вызов прототипного метода reverse, для результата работы функции и будет вам счастье. [1,2,3,4,5].reverse(); // -> [5,4,3,2,1]
    – user3545
    21 мар 2012 в 21:07
  • а это разве стандартная функция?
    – Smash
    21 мар 2012 в 21:22
  • 1
    И для сортировки можете юзать метод sort, грубо говоря все что вы писали можно написать так: function sorted( arr, reverse ) { var res = arr.sort(); return reverse !== undefined ? res.reverse() : res; } // или вообще так function sorted( arr, reverse ) { return reverse !== undefined ? arr.sort().reverse() : arr.sort() } Вот это действительно логично :D
    – user3545
    21 мар 2012 в 21:31
1

Я бы написал так:

function ab(x) {
  function _ab() {
    // блок кода
  }

  var a,b;
  if (a>b)
    _ab();
  if (a<b)
    _ab();
}
1
  • вполне это тоже о чем я думал, но получилось еще проще.
    – Smash
    21 мар 2012 в 22:14
0

Ну и как же без меня:D

function ab(x) {
    var a, b;
    function c() {
        return true;
    }
    a>b&&c(); 
    a<b&&c();
}
7
  • a>b&&c(); это даже извращеннее, чем (a<b)?1:0 у меня вчера )
    – ReinRaus
    22 мар 2012 в 6:40
  • @ReinRaus, в чем извращение?
    – Oleg
    22 мар 2012 в 7:02
  • Наверно в плохой читаемости конструкции. Нужно помнить, что сначала будет выполнена операция сравнения, а потом &&, также надо знать, что если левая часть && имеет значение false, то правая часть проверяться не будет (и это не нечто само-собой разумеющееся- в некоторых языках всегда выполняются обе части операнда).
    – ReinRaus
    22 мар 2012 в 7:44
  • @ReinRaus, всё там правильно.
    – Oleg
    22 мар 2012 в 7:56
  • А почему функция c() описана с одним параметром, а вызывается без него? @ReinRaus идиома для таких языков обычная. В perl'е постоянно do_some_thing or die()
    – alexlz
    22 мар 2012 в 8:46

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.