1. JavaScript / Говнокод #5370

    +152

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    var _t=false;
if ( a < this.options.min ) {
	this.options.min = a;
	_t=true;
}
if ( b > this.options.max ) {
	this.options.max=b;
	_t=true;
}
if ( _t ) {
	this.refreshValue();
	this.setMinMax();
}

    [:|||:] ?

    Запостил: istem, 24 Января 2011

    Комментарии (11) RSS

    • ava artureg 24.01.2011 11:21 # 0

      _t лишнее, но не фатально же ,)
      Ответить

    • ava sectus 24.01.2011 11:50 # +1

      Призываю КО
      Ответить

      • ava artureg 24.01.2011 11:54 # −2

        Всего лишь лишняя переменная

        if ( a < this.options.min ) {
        this.options.min = a;
        }
        if ( b > this.options.max ) {
        this.options.max=b;
        }
        if ( a < this.options.min || b > this.options.max) {
        this.refreshValue();
        this.setMinMax();
        }
        Ответить

    • ava Lure Of Chaos 24.01.2011 13:44 # −1

      а теперь представим, что между 1ой и 2ой строчкой куча кода, который не трогает _t
      Ответить

    • ava gegMOPO4 24.01.2011 15:01 # 0

      И что не так? Всё правильно.
      Ответить

    Добавить комментарий