2 февр. 2011 г.

Плохой стиль программирования в ExtJS

После  упорного рефакторинга своего и чужого кода решил выделить несколько моментов
непрофессионального и неперспективного стиля программирования
с которыми мне пришлось неоднократно сталкиваться и исправлять:

  1. Куча входных параметров в функции

    function myfunction(node, record, xtype, subject) {
    ...
    }

    минусы:
    - трудно добавить новый параметр
    - выглядит некрасиво и не оптимально

  2. Явное задание обработчиков
    var btn = new Ext.Button({
                      xtype:'button',
                      text: 'apply',
                      handler:  function(p){
                                    ...
                                    }
    });

    минусы:
    - нельзя использовать данный обработчик в другом месте
    - выглядит громоздко

    мое предложение:

    function myfunc(p) {
               ...
    }

    var btn = {
                      xtype:'button',
                      text: 'apply',
                      handler:  myfunc(p)
    }

ps:
Если есть комментарии или прпедложения готов рассмотреть