Блог

У меня есть следующий плагин:

 (function($) {
    $.fn.myPlugin = function(options) {
    var opt = $.extend({}, $.fn.myPlugin.defaults, options);

        if (!opt.a) {
            console.log('a is required!');
            return false;
        }
        if (!opt.b) {
            console.log('b is required!');
            return false;
        }
        if (!opt.c) {
            console.log('c is required!');
            return false;
        }
        //Rest of the logic
    }
    $.fn.myPlugin.defaults = {
    };
});
  

Теперь этот плагин будет вызываться извне следующим образом:

 $('div.x').myPlugin({
    a:'aa',
    b:'bb',
    c:'cc'
});
  

Как вы можете видеть из плагина, мне нужны параметры a, b и c извне, т. е. они обязательны. Но там может быть 10-15 обязательных опций и этот код

         if (!opt.a) {
            console.log('a is required!');
            return false;
        }
        if (!opt.b) {
            console.log('b is required!');
            return false;
        }
        if (!opt.c) {
            console.log('c is required!');
            return false;
        }
  

может стать длинным и громоздким. Есть ли какой-нибудь более короткий или разумный способ написать это? Я думал о каком-нибудь общем коде.

Если их так много, вы могли бы поместить их в массив и проверить таким образом:

 var required = ['a', 'b', 'c'];
var index, optname;
for (index = 0; index < required.length;   index) {
    optname = required[index];
    if (!(optname in opt)) {
        console.log(optname   " is required");
        return false;
    }
}
  

Обратите внимание, что я сделал if (!(optname in opt)) проверку там (а не if (!opt[optname]) как у вас изначально было), чтобы разрешить параметры, которые должны быть указаны, но для которых допустимы 0 , false , undefined или другие ложные значения. in Проверка проверяет, присутствует ли опция, не беспокоясь о том, что ее значение соответствует действительности.

Немного не по теме: вы можете предпочесть дождаться сбоя, пока не проверите все свойства, как указывает @Marcus в комментариях. Кроме того, вы могли бы рассмотреть возможность создания исключения, а не возврата false , поскольку тот, кто не смог правильно указать параметры, должен быть исключительным условием… Но это второстепенные моменты.

 $.each(['a','b','c'], function(key, val) {
    if (!opt[val] !== void 0) console.log(val   " is require");
});
  

$.each Просто намного короче / аккуратнее.

Хотя это то же самое, что и ответ @T.J.Crowder .

P.S. void 0 === undefined