lepszy kod

pytanie zadane 20 października 2018 w Java przez MarcinZielony Użytkownik (710 p.)
edycja 20 października 2018 przez Arkadiusz Waluk

Witajcie

powtarzam Javę w przyspieszonym tempie, czyli robię po jednym rozdziale dziennie.

Stworzyłem coś takiego. Niby działa, ale jakoś ułomnie to wygląda.

Ma ktoś lepszy pomysł? Bardziej sexy kod?

import java.util.Scanner;

public class Gra {
    
    public static void main(String[] args){
        
        int a=17;
        int takeOneInt=0;
        
        
        
        while(a!=takeOneInt){
        
        System.out.println("podaj liczbe");
        
        Scanner scanner= new Scanner(System.in);
        
             takeOneInt= scanner.nextInt();
        
            
             if (takeOneInt<a)
                
                System.out.println("liczba za mala");
                
            else if (takeOneInt>a)
                
            System.out.println("liczba jest za duza");
                
            }
            
            System.out.println("Brawo! Zgadles, liczba wynosi "+a);
        
        
        
}}

komentarz 20 października 2018 przez Aisekai Nałogowiec (42,190 p.)

Kod w znaczniki umieść.

komentarz 20 października 2018 przez MarcinZielony Użytkownik (710 p.)

Dobra uwaga. Nie wiedziałem jak.

2 odpowiedzi

odpowiedź 20 października 2018 przez RafalS VIP (122,820 p.)

Jedyne co jest do poprawy to wywalenie inicjacji scanera przed petle, nie ma sensu tworzyc go i niszczyć caly czas.

No i nazwy zmiennych. Albo za krótkie - a, albo dlugie ale dezinformujace - takeOneInt. Ta druga odpowiada za pobieranie i zgadywanie. Czemu nie nazwiesz jej currentGuess, albo userGuess.

Poza tym nie ma sie do czego przyczepić bo ciezko napisac inaczej cos tak prostego.

komentarz 20 października 2018 przez MarcinZielony Użytkownik (710 p.)

Dzięki za odpowiedź.

Z inicjacją scannera to fakt.

Odnosnie zmiennych, to na moje potrzeby, ale w sumie racja- lepiej wyrabiać nawyki.

Pytanie brzmi, czy można coś zrobić z:

int takeOneInt=0;

czy da się bez przypisania? Żeby tylko zainicjować.

1	komentarz 20 października 2018 przez RafalS VIP (122,820 p.) Nie możesz bo niżej wykorzystujesz ją w porównaniu: a!=takeOneInt Nie możesz porównywać niezainicjowanej zmiennej.

komentarz 20 października 2018 przez MarcinZielony Użytkownik (710 p.)

Ok. Dzięki.

odpowiedź 20 października 2018 przez Aisekai Nałogowiec (42,190 p.)

Co znaczy "bardziej sexy kod"? Krócej? Z możliwością przyszłościowego rozwoju?

Przede wszystkim, nie potrzebne Ci tworzenie nowego skanera za każdym obiegiem pętli. Oczywiście, że dałoby się to rozbić na różne klasy, choćby cały kod odpowiedzialny za logikę przenieść do osobnej klasy i w main zrobić

new Game().start();

Tylko "problem" jest taki, że sam program jest bardzo krótki, a im krótszy tym mniej rzeczy można poprawić. Ale:

1. Zmiana nazwy klasy na angielskojęzyczną - używanie angielskiego nazewnictwa.

2. Zmieniłbym nazwę takeOneInt - nazwa charakterystyczna dla funkcji - na np userGuess.

3. Zmienną "a" możesz wylosować, zamiast na sztywno ją ustalać. Wtedy do zmiennej userGuess możesz przypisać wartość a+1, albo użyć pętli do-while zamiast while. Ogólnie zmienną "a" poleciłbym zmienić nazwę.

4. Możesz sformatować kod. "}}" na końcu nie wygląda dobrze. Ogólnie pełno pustych linii, których nie powinno być.

komentarz 20 października 2018 przez MarcinZielony Użytkownik (710 p.)

Dzięki za odpowiedź.

1) Ok

2) Ok (nie znam norm komercyjnych)

3) Fakt

4) Ok. Estetyka.

Wszystko da się zrobic lepiej. O to mi chodziło. Pozdrawiam

komentarz 20 października 2018 przez MarcinZielony Użytkownik (710 p.)

Odnosnie zastosowania klas, to to za tydzien. Tak Java jest obiektowa, ale rozpoczecie od obiektówki nie jest naturalnym prcesem nauki/ przypominania/ utrwalania/ wyrabiania nawyku.

komentarz 20 października 2018 przez Aisekai Nałogowiec (42,190 p.)

Wiem, że nie powinno się od razu rozpoczynać nauki od obiektówki gdy się dopiero podstawy zaczyna, ale pytałeś o to, czy da się to zrobić "bardziej sexy"

Edit: Nazwy powinny mówić co do czego służy. Do tego jest kilka konwencji do których powinno się stosować tworząc nazwy, takie jak np. cammelCase zmiennych/funkcji, czy rozpoczynanie nazw funkcji od czasowników. Dlatego też, zmienne nie powinny zaczynać się o czasowników.