Ocena kodu JavaScript

Question

Obecnie uczę się JS i w podręczniku był taki kawałek kodu:

function calculateArea(r)
{
    var area;
    if (r <= 0)
    {
        return 0;
    }
    else
    {
        area = Math.PI * r * r;
        return area;
    }
}
var radius = 5.2;
var theArea = calculateArea(radius);

console.log("Pole wynosi " +theArea);

Ale dla mnie to jest w ogóle niejasne, co jest czym. Dlatego napisałem program, który robi to samo, tylko trochę inaczej oraz użytkownik ma możliwość podania swojego promienia, a nie jest podany "na sztywno":

function calculateArea(r)
{
    var area;
    if (r < 0)
    {
        return 0;
    }
    else
    {
        area = Math.PI* r* r;
        return area;
    }
}
var r  = prompt("Podaj promień:");
console.log("Pole " +calculateArea(r));

Ale który kod jest lepszy? Czy zawsze mniej kodu == lepszy kod? (zakładając, że obydwie wersje dają dokładnie ten sam skutek)

Answer 1

Usunąłeś spacje przed gwiazdkami i nazywasz to krótszym kodem? O_o Według mnie lepiej albo dawać spacje po obu stronach mnożenia albo w ogóle nie dawać spacji bo twój zapis za bardzo się kojarzy ze wskaźnikami.

Mimo wszystko uważam, że zarówno pierwszy kod jak i drugi jest tak samo słabo napisany. Po pierwsze po co instrukcja else, skoro po return i tak funkcja nie jest dalej wykonywana. Po drugie po co jest zmienna area, skoro takie krótkie wyrażenia można wrzucać bezpośrednio w return.

Comments

pyr pyr pyr

f=r=>0<r?Math.PI*r*r:0,console.log(`Pole ${f(prompt(`Podaj promień:`))}`)

---

Twoje "pyr pyr pyr" bardzo dużo wnosi do tematu. Dobrze, że są jeszcze na tym forum tacy "inteligenci".

---

dzięki ; )

Answer 2

Nie zawsze krótszy kod = lepszy kod :)

Oczywiste jest to, że wygodniej jest podawać dane przed "prompt" niż je wklejać na sztywno w .js.

Co do krótszego kodu... Odpowiedz sobie sam na to pytanie.

Co jest wg. Ciebie lepsze:

1. Krótszy kod, pozbawiony wszelakich zabezpieczeń, nie czytelny dla innych programistów ORAZ, którego komputer dłużej "trawi".

2. Dłuższy, bezpieczny kod, zrozumiały dla innych ORAZ, szybszy w kalkulacji przez nasze PC :)