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

    +150

    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
    14. 14
    //
        getMode: function(nMode) {
            switch (nMode) {
                case this.MODES.LEFT_BOOKEND:
                    return this.aModes[nMode];
                case this.MODES.RIGHT_BOOKEND:
                    return this.aModes[nMode];
                case this.MODES.BOTH_BOOKENDS:
                    return this.aModes[nMode];
                case this.MODES.NONE:
                default:
                    return this.aModes[this.MODES.NONE];
            }
        },

    Наверное уже боян, но вот же он, опять!

    https://github.com/scirelli/ExtjsBreadCrumbs/blob/master/js/ux/breadCrumbs.js

    Запостил: Elvenfighter, 11 Июня 2013

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

    • ок
      Ответить
    • ?
      Ответить
    • Про баян тоже непонятно
      Ответить
      • Что-то похожее уже встречалось.

        1. case непосредственно перед default не нужно, ибо default покрывает и этот случай.
        2. строки 04-09 утрамбовываются:
        case this.MODES.LEFT_BOOKEND:
        case this.MODES.RIGHT_BOOKEND:
        case this.MODES.BOTH_BOOKENDS:
                        return this.aModes[nMode];
        Ответить

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