Przejrzyste pisanie kodu

Witam

Chciałbym się dowiedzieć czy kod prawidłowo powinien wyglądać tak

var Sprzedaz = "Toyota";

function TypyAut(nazwa) {
   if(nazwa == "Polonez")
      return nazwa;
   else
      return "Przykro nam, ale nie sprzedajemy marki " + nazwa + ".";
}

czy może tak

var Sprzedaz = "Toyota";

function TypyAut(nazwa) { {
   if(nazwa == "Polonez") }
      return nazwa; {
   } else
      return "Przykro nam, ale nie sprzedajemy marki " + nazwa + ".";
}

Bo w pętlach for nawias "{" pojawia się dość często

    for (var x=1; x<=10; x++) {
        var tekst = x*y; //tworzymy tekst do wstawienia do komórki

        if (y==1 || x==1) {  //jeżeli jest to pierwsza komórka w pionie lub poziomie
            var td = '<th>'+tekst+'</th>'; //stwórz nowe th
        } else {
            var td = '<td>'+tekst+'</td>'; //stwórz nowe td
       }

Wybierz któryś styl i się go spójnie w całym projekcie trzymaj

Wg mnie K&R best w tym przypadku

var sprzedaz = "Toyota";

function TypyAut(nazwa) {
	if (nazwa == "Polonez") {
		return nazwa;
	} else {
		return "Przykro nam, ale nie sprzedajemy marki " + nazwa + ".";
	}
}

@ekhart - Wg mnie zachęcasz do złej praktyki. Dlaczego? Bo tylko zwiększa się szansę na to że ktoś zechcę coś dopisać po ifie i dostanie błąd. Dopisanie tych klamerek nawet przy pojedynczej instrukcji (np w ifie) daje o tyle fajną możliwość - że w przypadku np javascriptu, można tam na szybko sobie machnąć console.log'a albo de facto inne instrukcje (!debug-mode).

Brak klamerek dla mnie nic nie daje, trudniej coś dopisać - bo i trzeba klamerki znowu dodawać, zmniejsza ryzyko błędów oraz nie stanowi jakiegoś strasznie nadmiarowego zwiększenia objętości pliku z takimi skryptami.
EDIT:
ooo widzę że jednak polecasz klamerki - prawidłowo. Dodatkowo istnieje jeszcze wojna pomiędzy

if(warunek){
    costam();
}

a

if(warunek)
{
    costam();
}
Ale to już temat na inna bajkę ;)

axelbest napisał(a):

@ekhart - Wg mnie zachęcasz do złej praktyki. Dlaczego? Bo tylko zwiększa się szansę na to że ktoś zechcę coś dopisać po ifie i dostanie błąd. Dopisanie tych klamerek nawet przy pojedynczej instrukcji (np w ifie) daje o tyle fajną możliwość - że w przypadku np javascriptu, można tam na szybko sobie machnąć console.log'a albo de facto inne instrukcje (!debug-mode).

Brak klamerek dla mnie nic nie daje, trudniej coś dopisać - bo i trzeba klamerki znowu dodawać, zmniejsza ryzyko błędów oraz nie stanowi jakiegoś strasznie nadmiarowego zwiększenia objętości pliku z takimi skryptami.
EDIT:
ooo widzę że jednak polecasz klamerki - prawidłowo. Dodatkowo istnieje jeszcze wojna pomiędzy

Kopiuj

if(warunek){
    costam();
}

a

Kopiuj

if(warunek)
{
    costam();
}
Ale to już temat na inna bajkę ;)

Dzięki za odpowiedź. Ale te dodatkowe klamerki dla javascriptu coś oznaczają czy to jest dla samego programisty?

@LynxBings: oznaczają - tak jak chyba we wszystkich językach które zawierają klamerki (python ich nie ma). Oznaczają blok kodu. Wiadomo że przy pojedynczej instrukcji nie są potrzebne, ale koszt dodania jest niski, a mamy same zyski.

A może:

function TypyAut(nazwa) { // zmień nazwę funkcji, bo nic nie mówi
    if (nazwa !== 'Polonez') {
        return "Przykro nam, ale nie sprzedajemy marki " + nazwa + ".";
    }

    return nazwa;
}

wtedy masz -1 zagnieżdżenie dla właściwego przebiegu sterowania, a nie musisz przewijać na sam koniec funkcji, żeby zobaczyć co się stanie jak to jednak nie polonez.

axelbest napisał(a):

@LynxBings: oznaczają - tak jak chyba we wszystkich językach które zawierają klamerki (python ich nie ma). Oznaczają blok kodu. Wiadomo że przy pojedynczej instrukcji nie są potrzebne, ale koszt dodania jest niski, a mamy same zyski.

Czyli w praktyce

else {
        return "Przykro nam, ale nie sprzedajemy marki " + nazwa + ".";
    }

Klamerki ograniczają kod żeby był wykonany w ramach przykładowego else?

@LynxBings: tak. Jak masz po if/else pojedyńcze wyrażenie to teoretycznie nie trzeba ich pisać, ale dobrą praktyką jest jednak ich dodawanie.

Ogólnie to polecam się nie zastanawiać i zainstalować linter (ESLint) z jednym z tych style guidów:

• Airbnb JavaScript Style Guide -> https://github.com/airbnb/javascript (używam w pracy)
• StandardJS -> https://github.com/standard/standard (wolę do swoich projektów)

<!DOCTYPE html>
<html>
    <head>
        <script>
            function isCarForSale(name) {
                var carsNotForSale = ['polonez'];

                return carsNotForSale.indexOf(name.trim().toLowerCase()) < 0;
            }

            function showTestMessage() {
                var msg = isCarForSale('Polonez2') ? 'Ok, można sprzedawać' : 'Nie można sprzedawać';

                alert(msg);
            }
        </script>
    </head>
    <body>
        <button onclick="showTestMessage()">Test</button>
    </body>
</html>

To tak jako przykład do komentarza @Maciej Cąderek Oczywiście można to rozwinąć o wiele bardziej (walidacje typów itp itd)

axelbest napisał(a):

Kopiuj

<!DOCTYPE html>
<html>
    <head>
        <script>
            function isCarForSale(name) {
                var carsNotForSale = ['polonez'];

                return carsNotForSale.indexOf(name.trim().toLowerCase()) < 0;
            }

            function showTestMessage() {
                var msg = isCarForSale('Polonez2') ? 'Ok, można sprzedawać' : 'Nie można sprzedawać';

                alert(msg);
            }
        </script>
    </head>
    <body>
        <button onclick="showTestMessage()">Test</button>
    </body>
</html>

To tak jako przykład do komentarza @Maciej Cąderek Oczywiście można to rozwinąć o wiele bardziej (walidacje typów itp itd)

Tutaj dodałem komentarze z tego jak ja widzę ten kod i z jednym pytaniem czy możecie mi sprawdzić czy dobrze to rozumiem?

        <script>
            function isCarForSale(name) { // wywołujemy funkcję
                var carsNotForSale = ['polonez']; // Tu tworzymy stringa o nazwie polonez
 
                return carsNotForSale.indexOf(name.trim().toLowerCase()) < 0; // Jeżeli jest mniejsze od zera to ma zwrócić ale co to za funkcja? I jak dowiedzieć się co ona oznacza?
            }
 
            function showTestMessage() { // Tutaj załączona funkcja która odpowiada za wyświetlanie wiadomości
                var msg = isCarForSale('Polonez2') ? 'Ok, można sprzedawać' : 'Nie można sprzedawać'; // Tutaj deklarujemy zmienną ze stringiem polonez2 i dwoma tekstami wiadomości w zależności od spełnionych warunków
 
                alert(msg); // pokazuje wiadomość
            }
        </script>