Parzystość liczby javascript liczby ujemne

Question

Serdecznie witam!

Piszę funkcję sprawdzającą parzystość liczb. Jaki dać warunek dla liczb ujemny?

Wpadłem na coś takiego 

function isEven (numer)
{
	if (numer == 0) return document.write('Liczba jest parzysta');
	if ((numer < 0) && (numer%(-2)==0)) return document.write("Liczba jest parzysta");
	else if (numer%2==0) return document.write("Liczba jest parzysta");
	else return document.write("Liczba jest niepatrzysa");
}
isEven(-56)

działać, działa tylko nie wiem czy nie wieje amatorszczyzną

Answer 1

Kod jest ostro przekombinowany. Całą funkcję isEven można wyrazić JEDNĄ linijką: return number % 2 === 0;. Dodatkowo: funkcja ma zwracać swój wynik – nic więcej. Wyświetlanie informacji, że liczba jest/nie jest parzysta nie jest już działką funkcji sprawdzającej parzystość. Po tych zmianach dostajemy ostatecznie taki kod:

function isEven( number ) {
	return number % 2 === 0;
}

if ( isEven( -56 ) ) {
	document.write( 'Liczba jest parzysta' );
} else {
	document.write( 'Liczba nie jest parzysta' );
}

Oczywiście document.write nie powinno być używane, a zamiast tego lepiej zastosować element.innerHTML.

Comments

funkcja ma zwracać swój wynik – nic więcej. Wyświetlanie informacji, że liczba jest/nie jest parzysta nie jest już działką funkcji sprawdzającej parzystość.

O tym właśnie myślałem, kiedy pisałem że dziwnie wygląda zwracanie document.write. :)

Całą funkcję isEven można wyrazić JEDNĄ linijką

Ilość kodu nie jest wyznacznikiem jakości kodu. :) Ale popieram. Kod - zarówno autora wątku, jak i moją propozycję - można znacznie uprościć, co na złe nie wyjdzie. :)

---

Ilość kodu nie jest wyznacznikiem jakości kodu. :)

No można to rozpisać na 30 linijek + 8 komentarzy… ale po co? Ani to nie poprawi czytelności, ani nie przyśpieszy kodu.

---

Sądzę że Ty wiesz, jak pisać kod. Myślę że ja też mniej-więcej to czuję. ;)

Nie pisałem tego dla Ciebie ani dla siebie... Bardziej dla autora wątku i przyszłych młodych programistów, którzy tu zajrzą. :)

Answer 2

Ja bym to logicznie poukładał trochę inaczej. W JS nigdy w życiu nie pisałem i trochę dziwnie mi wygląda "return document.write()", ale Ty pewnie wiesz, co napisałeś i dlaczego. :)

function isEven (number)
{
    if (number < 0) number = -number;

    if (number%2 == 0) return document.write("Liczba jest parzysta");
    else return document.write("Liczba jest niepatrzysa");
}