check

gistfile1.txt

/**
 * Студент делает длинную проверку и незнает как сократить
 * if (popupName.value.length > 1 && popupName.value.length < 31 && ((popupLink.value.slice(0, 7) == "http://" && popupLink.value.length > 7) || 
 * (popupLink.value.slice(0, 8) == "https://" && popupLink.value.length > 8))) {
 * Ну во первых мы в качестве параметров передаём объекты
 * и разбиваем на две функции
 */

function _checkLength(item, min, max) {
  return item.value.length > min && item.value.length < max;
}

function _checkProtocol(item) {
  return item.value.slice(0, 7) == "http://" || item.value.slice(0, 8) == "https://";
}

function myCheck(itemName, itemLink) {
  return _checkProtocol(itemLink) && _checkLength(itemName, 1, 31);
}

// вызов 
myCheck(itemName, itemLink);

/** вызов
 * Функцию можно переиспользовать
 * В функциях нет ни одного if
 *
 * @koras
 */