Как лучше оптимизировать и сделать мой код на JavaScript красивее и короче?

Question

[Sp1keazyYT]

Добрый день. Имеется такой кусок кода на javascript:

Price2 = 0;
if (roofpt == null || roofpt == 0) Price2 = 5500;
else if (roofpt !== null || roofpt !== 0 && roofmat == 1) Price2 = 7900;
else if (roofpt !== null || roofpt !== 0 && roofmat == 2) Price2 = 8900;

Посоветуйте пожалуйста и скажите, как лучше оптимизировать и сделать мой код JS короче (выглядит не красиво)?

Comments

[Сергей Соколов]

это у вас работа и действительно внутреннее чувство прекрасного не даёт покоя, или мы всем Тостером за вас собеседование проходим?

[Sp1keazyYT]

Сергей Соколов, это просто попутная микро-задача и всё, что я делаю, я пытаюсь довести до идеала и в том числе, чтобы код был написан правильно и красиво.

[alex-1917]

Сергей Соколов, что меня насторожило, из всех 30 вопросов, заданных этим персонажем, в том числе и текущий вопрос, ВСЕ содержат выдранный откуда-то код и ВСЕ НЕ СОДЕРЖАТ попыток решения самим ТС...

[EvgenyJozef3000]

alex-1917, А может у данного "персонажа" уже нашлись варианты решения и он просто их не писал?

[alex-1917]

EvgenyJozef3000, может и у бабушки яйца, щас все возможно

Answer 1

[Сергей Соколов]

Если это практическая задача, а не собеседование с глубоко зарытыми подвохами, то так:

if (!roofpt) Price2 = 5500; // null, 0, undefined, false, пустая строка
else if (roofmat == 1) Price2 = 7900; // и так известно, что не пустое
else if (roofmat == 2) Price2 = 8900; // и так известно, что не пустое
else Price2 = 0; // по умолчанию

Если же это из собеседования

В исходном коде есть такая деталь. Приоритет операторов в JS ставит && выше, чем ||. Эти два условия выглядят похоже, имелась в виду, наверное, лишняя перепроверка на null/ноль:

(roofpt  == null || roofpt  == 0) 
(roofpt !== null || roofpt !== 0 && roofmat == 1)

Но сначала &&, потом ||:

A || B && C === A  ||  (B && C)
// и второе условие на деле считается так:
(roofpt !== null || (roofpt !== 0 && roofmat == 1))

Т.е. второе условие выполнится и просто при любом rootpt, неравном null.

Тут же ещё одна деталь. Нестрогое равенство == и строгое неравенство !==. Разжёвывать не буду, чтобы вы провалили собеседование.

Comments

[Sp1keazyYT]

Спасибо! На счёт собеседования хотел бы сказать, что это не оно. Эта одна из попутных задач в процессе работы. Все, что я делаю, весь код, который я пишу, я хочу довести до идеала, чтобы он был написан корректно и красиво, на этот счёт я уже ответ у вас под первым комментарием.

Answer 2

[kova1ev]

Sp1keazyYT вы бы дали пояснение к коду, что он должен делать. Потому что сейчас он написан так, что при любом ненулевом значении roofpt, значение Price2 будет равно 7900, соответственно второй else if просто бессмыслен.

Поэтому, если именно этот код переписывать, то можно так:

let Price2 = 0;

if (!roofpt) Price2 = 5500;
else Price2 = 7900;

или так

let Price2 = 0;

Price2 = (roofpt) ? 7900 : 5200;

Answer 3

[coderisimo]

Можно решить довольно компактно.
вот здесь можно поиграться с ф-цией
песочница

function getPrice() {
    return (
      (!parseInt(roofpt) && 5500) ||
      (roofmat == 1 && 7900) ||
      (roofmat == 2 && 8900) ||  0 ); }

Comments

[Sp1keazyYT]

Спасибо, но думаю все-таки решение:

if (!roofpt) Price2 = 5500; // null, 0, undefined, false, пустая строка
else if (roofmat == 1) Price2 = 7900; // и так известно, что не пустое
else if (roofmat == 2) Price2 = 8900; // и так известно, что не пустое
else Price2 = 0; // по умолчанию

Будет по практичнее

[coderisimo]

Sp1keazyYT, а чем практичнее ? ). Пугает доп ф-ция ?

let price = (function(){
      return (roofpt && 5500) ||
      (roofmat == 1 && 7900) ||
      (roofmat == 2 && 8900) ||
      0
  })();

впрочем, дело вкуса )

[Sp1keazyYT]

Василий, чтобы лишним функции не использовать

[coderisimo]

Sp1keazyYT, см. мой ответ выше )

[coderisimo]

Sp1keazyYT,
вернее, даже так :

let Price2 = 
      (roofpt && 5500)||
      (roofmat == 1 && 7900) ||
      (roofmat == 2 && 8900) ||  0 ;

Answer 4

[Александр]

Price2 = 0;
if (roofpt == null || roofpt == 0) Price2 = 5500;
else if (roofpt !== null 
      || roofpt !== 0 
      && roofmat == 1) Price2 = 7900;
else if (roofpt !== null 
      || roofpt !== 0 
      && roofmat == 2) Price2 = 8900;

Если в строке с if более 2-х условий, то рекомендуется разделять их на разные строки.