Создать JavaScript «HasValue» Util метод

Question

Хороший Javascript программист хорошо знает этот тест:

if (typeof(dummy) != 'undefined' && dummy != null && dummy != '') { 
    // Do stuffs 
} 

На самом деле этот тест убедитесь, что фиктивная переменная имеет значение. Поэтому я попытался написать эту функцию. Мои тесты случаи:

hasValue(undefined) => should returns false 
hasValue(null) => should returns false 
hasValue('') => should returns false 
hasValue(' ') => should returns true 
hasValue([]) => should returns false 
hasValue({}) => should returns false 
hasValue(true) => should returns true 
hasValue(false) => should returns true 
hasValue(0) => should returns true 
hasValue(new Date()) => returns false, but should returns true ? 

Вот функция:

function hasValue (v) { 
    if (typeof v === 'object') { 
    for (key in v) { 
     if (v.hasOwnProperty(key)) { 
     return true; 
     } 
    } 
    return false; 
    } 
    if (v === false) { 
    return true; 
    } 
    if (v === 0) { 
    return true; 
    } 
    return (typeof(v) != 'undefined' && v != null && v != ''); 
} 

Кто-нибудь увидеть что-то не так, или какой-либо оптимизации, или, может быть, я забыл Somes случаи?

Answer 1

Не так много, но некоторые сокращения кода, где ваш тестируют для объекта

if (typeof v === 'object') { 
    return Object.keys(v).length > 0; 
} 

Answer 2

Это ужасный способ Util. Вы засучив следующие решения в одну функцию:

• Хотите [] означает «пользователь не выбрал никаких опций, переспрашивать» или «пользователь не выбрал никаких вариантов, по-прежнему»?
• Вы хотите, чтобы {} означало «никакие параметры не переданы функции» или {} означает «параметры, не переданные функции, используйте параметры по умолчанию»?
• Вы хотите, чтобы ' ' был действительным пользователем для среднего имени, но '' недействительным?
• Вы хотите, чтобы 0 был действительным идентификатором пользователя, даже если ваша база данных возвращает его, что означает, что нет идентификатора пользователя?

Логика зависит от домена. Если вы программируете, скажем, панель пользовательских параметров, вы можете реализовать UserDashboard.hasValue, который вернет false для [], а остальное, как указано, но это зависящая от домена логика, а не некоторые общие символы underscore.js или jQuery или node.js функция полезности модуля.

Даже тогда это было бы беспорядок для чтения. Если я вижу линию

if(hasValue(v)) { 
    ... 
} 

я теперь должен прочитать тайную логику в hasValue, чтобы выяснить, какие возможные комбинации значений falsey, [], {} и ' 'вы разрешаете и запрещая. Если вам просто нужно проверить, что v не null напишите код, чтобы сделать это.

Существуют методы на разных языках, как isNullOrEmpty, который является метод полезности, как Вы сказали, только его название говорит вам точно то, что она делает, и он не пытается сделать все. Вероятно, он имеет более подробное имя, чем вы думаете. Это верно; он должен, или же неясно, что он на самом деле делает.