Ревью 🔍

Несколько ключевых моментов, которые помогут в процессе проверки:

⏱️ Уровень сложности ревью: 2 🔵🔵⚪⚪⚪

🧪 Нет релевантных тестов

🔒 Проблем с безопасностью не выявлено

⚡ Рекомендовано к рассмотрению

Неполная логика обработки скобок

Функция обрабатывает только круглые скобки, игнорируя квадратные и фигурные. Вызов stack.pop() без предварительной проверки длины стека может привести к некорректному поведению при невалидных входных данных.

function validParentheses(parenStr) {
  let stack = [];
  const sl = {')' : '(', '(' : ')' };

  for (let i = 0; i < parenStr.length; i++) {
      if  (parenStr[i] == '(') {
        stack.push(parenStr[i])
      }else{
        if (sl[stack.pop()] !=  parenStr[i]){
            return false
        } 
    }
  }
   if (stack.length == 0) {
     return true;
   } else {
     return false;

}
  }

Отсутствует возврат значения по умолчанию

Функция не возвращает явное значение (например, null или undefined), если подходящая пара чисел не найдена. Это может привести к неопределённому поведению в вызывающем коде.

function sumPairs(ints, s) {
  const sl = {};

  for (let i = 0; i < ints.length; i++) {
    const r = s - ints[i];
    if (r in sl) {
      return [r, ints[i]];
    } else {
      sl[ints[i]] = i;
    }
  }

}

ISUCT

Популярные репозитории