Говнокод или нет?

Question

SlideUp @SlideUp

JavaScript

Говнокод или нет?

Ребятки всем привет. Решаю разные задачки с помощью JS и вот решил одну(квадратное уравнение). Подкажите нормальный ли код или лучше как-то по ф-циям разбить?

let a = prompt("Enter a value", 10);
let b = prompt("Enter b value", 10);
let c = prompt("Enter c value", 10);
let result;
let discr = (b * b ) - 4 * a * c;
let sqrtDiscr = Math.sqrt(discr);
if(a === 0 && b === 0 && c === 0){
  throw new Error("The values must be bigger than 0");
} else if(discr < 0 ){
  throw new Error("This Equation have not solution");
} else if(discr === 0 ){
  console.log("This Equation have only 1 solution")
  result = (-b) / (2 * a);
} else if (discr > 0){
  console.log("This Equation have 2 solution");
  result = (- b + sqrtDiscr) / (2 * a) + ";     " + (- b - sqrtDiscr) / (2 * a);
}

console.log(result);

Вопрос задан более трёх лет назад
448 просмотров

6 комментариев

Подписаться 2 Оценить 6 комментариев

Andrey Perov @SnaIP
SlideUp по мне так удобнее и код не очень
let a =1, b = 2, c = 3;

Выносите в функцию это, например то что считает и возвращайте return результат ее
Написано более трёх лет назад
SlideUp @SlideUp Автор вопроса

Andrey Perov, Лет подправил, а код не очень через if'ы?

Написано более трёх лет назад
Алексей Шашенков @teknik2008

Я так понял до функций еще не дошли?

Написано более трёх лет назад
Andrey Perov @SnaIP

SlideUp лучше делать код логичным более и разделять на куски, на функции каждая функция должна заниматься какой то своей задачей

Написано более трёх лет назад
ⓒⓢⓢ @JRK_DV

Так и планировалось при значениях a=0,b=1,c=1 выводить
This Equation have 2 solution
NaN; -Infinity

Или это только на моём браузере так...

Написано более трёх лет назад
SlideUp @SlideUp Автор вопроса

JRK_DV, Принял ошибку, спасибо!

Написано более трёх лет назад

SlideUp по мне так удобнее и код не очень
let a =1, b = 2, c = 3;

Выносите в функцию это, например то что считает и возвращайте return результат ее
Andrey Perov, Лет подправил, а код не очень через if'ы?
Я так понял до функций еще не дошли?
SlideUp лучше делать код логичным более и разделять на куски, на функции каждая функция должна заниматься какой то своей задачей
Так и планировалось при значениях a=0,b=1,c=1 выводить
This Equation have 2 solution
NaN; -Infinity

Или это только на моём браузере так...

Answer 1 · 2017-09-13 15:09:20

Код ~~г-но~~ неидеален, и вот, почему:

см. комменты в коде

let a = prompt("Enter a value", 10); // a, b, c дальше не меняются
let b = prompt("Enter b value", 10); // поэтому можно
let c = prompt("Enter c value", 10);  // const
let result;
let discr = (b * b ) - 4 * a * c;
let sqrtDiscr = Math.sqrt(discr); // при отрицат. discr здесь NaN
if(a === 0 && b === 0 && c === 0) { // проверяем уже после вычислений
  throw new Error("The values must be bigger than 0");
} else if(discr < 0 ){ // проверяем уже после вычисления sqrtDiscr
  throw new Error("This Equation have not solution");
} else if(discr === 0 ){
  console.log("This Equation have only 1 solution")
  result = (-b) / (2 * a);
} else if (discr > 0){
  console.log("This Equation have 2 solution");
  result = (- b + sqrtDiscr) / (2 * a) + ";     " + (- b - sqrtDiscr) / (2 * a);
}

console.log(result);

По функциям разбивать есть смысл если из нескольких мест идёт обращение, код повторяется более двух раз. Тут можно в двух последних случаях считать два корня одной формулой, и при нулевом дискриминанте брать только одно из двух равных значений. Но это излишняя, дурная, вредная переоптимизация.

Ещё вы не проверили случай, когда a === 0: у вас будет деление на ноль.

Говнокод или нет?

Войдите на сайт