Как правильно проверить, существует ли массив в JS?

Попробуйте поменять порядок проверок:

if (Array.isArray(drugPrice.mailPrice.rejectMessage) && drugPrice.mailPrice.rejectMessage.length !== 0) {
                                        code goes here 
}

Проверьте свои данные, замена условия может помочь, но не предотвратит возникновение некоторых ошибок. Например, Array.isArray(drugPrice.mailPrice.rejectMessage) выдаст ошибку, если drugPrice.mailPrice не определено.

if (drugPrice.mailPrice 
    && drugPrice.mailPrice.rejectMessage 
    && drugPrice.mailPrice.rejectMessage.length !== 0 
    && Array.isArray(drugPrice.mailPrice.rejectMessage)) {
  // code goes here 
}

var drugPrice = { mailPrice: { rejectMessage: {} } };

if (drugPrice.mailPrice 
    && drugPrice.mailPrice.rejectMessage 
    && drugPrice.mailPrice.rejectMessage.length !== 0 
    && Array.isArray(drugPrice.mailPrice.rejectMessage)) {

  console.info('success');
} else {
  console.info('fail')
}

ПРИМЕЧАНИЕ

Всегда проверяйте свои данные. Не думайте, что вы всегда будете получать правильные данные. При работе с объектами всегда проверяйте их, так как выполнение data.name может сломать ваше приложение, если data имеет значение null или не определено. например, для следующего объекта.

const drugPrice = { mailPrice: null };

делает, выдает ошибку.

const drugPrice = { mailPrice: null };

// throws an error, Cannot read property 'rejectMessage' of undefined
if (Array.isArray(drugPrice.mailPrice.rejectMessage)) {
}

чтобы этого не произошло, нам нужно проверить, существует ли свойство, как показано ниже.

const drugPrice = { mailPrice: null };
console.info(drugPrice.mailPrice && Array.isArray(drugPrice.mailPrice.rejectMessage) || 'Price is null or undefined')

Вам действительно не нужно на самом деле делать .length !== 0. Вы можете просто сделать:

if (Array.isArray(A.B.C) && A.B.C.length) {  // <-- order is important here
    //...
}

.length будет оцениваться как логическое значение, и это даст вам тот же результат, что и проверка с помощью !==0

При этом, однако, ваши пути довольно длинные, поэтому вы, вероятно, захотите убедиться, что они действительны. Это означает, что если drugPrice или mailPrice ложны, у вас возникнет проблема. Поэтому обычно вы также хотели бы проверить их. Поскольку ваш вопрос был о части массива, я пропущу их, но только для вашего сведения.

Вы можете создать свою собственную проверку пути или, если вы используете библиотеки, такие как lodash/underscore и т. д., у них всегда есть удобные функции get/has для проверки, как это (с lodash):

if (_.has(drugPrice, 'mailPrice.rejectMessage.length'))
    //...
}

Очевидно, не используйте эти библиотеки только для этого, но если они у вас уже есть, эти методы весьма удобны. Вы также можете просто проверить каждый из путей через:

if (A && A.B && Array.isArray(A.B.C) && A.B.C.length) {
    //...
}

Это просто становится утомительным, если у вас длинные пути к объектам и т. д.

Проблема в вашем коде заключается в том, что javascript проверяет длину массива, прежде чем проверять, является ли массив типом массива. Вы должны изменить порядок в операторе if.

Вы можете попробовать с:

if (myArr && Array.isArray(myArr) && myArr.length !== 0) {
    // your code
}

Теперь код выполняется в правильном порядке.

1. Первое условие проверяет, определен ли myArr,

2. Второе условие проверяет, является ли myArr типом Array, вы также можете сделать это так:

   если (myArr && myArr.push && myArr.length !== 0) { // ваш код }

3. Третье условие проверяет, не является ли myArr пустым.