Rozwiązanie problemu logicznego(?) logowania w aplikacji konsolowej

Question

Hej hej,

postanowiłem sobie dziś stworzyć ToDo apkę. Stanąłem na problemie z logowaniem. 

Na samym początku mam do wyboru 2 opcję albo stworzenia nowego konta albo zalogowania się już do istniejącego.

Main:

switch (option) {
                        case 1:
                            accountMaker.CreateAccount();
                            isTrue = false;
                            System.out.println("Now let's log into you account.");
                        case 2:
                            System.out.println("loguje"); // tutaj ma byc ta metoda, ktora bedzie odpowiadala za logowanie
                            isTrue = false;
                            break;
                    }

Jako, że nigdy nie bawiłem się jeszcze w bazy danych, także konto trzeba utworzyć w programie :).

Użyłem do tego HashMapy, gdzie key to jest login a value to jest hasło.

Tak wygląda ta klasa:

AccountMaker

import java.util.HashMap;
import java.util.Scanner;

public class AccountMaker {

    private HashMap<String, String> loginDetails = new HashMap<>();
    private Scanner input = new Scanner(System.in);
    private String login, password;

    void CreateAccount() throws InterruptedException {
        System.out.println("Input your login");
        login = input.next();

        System.out.println("Input your password");
        password = input.next();

        loginDetails.put(login, password);
        System.out.println("Account has been created successfully");
        Thread.sleep(2000);
    }
    
}

 

Następnie musimy się zalogować i tutaj mam problem, bo nei wiem jak to zrobić w drugiej klasie.

Mam coś takiego:

AccountLogger

public class AccountLogger {
    private AccountMaker accountMaker;

    private AccountLogger(AccountMaker accountMaker) {
        this.accountMaker = accountMaker;
    }

    private boolean logIntoAccount() {
        
        return true;
    }

}

I teraz nie wiem jak zrobić, by móc się zalogować oraz  sprawdzić czy np. dany użytkownik wpisał dobry key i dobre value.Jak zły key czy value to wyskoczy o tym informacja, jak dobry to wyskoczy informacja ze zalogowano...  Po prostu jak działać na tej HashMapie.

Próbowalem za pomoca gettera HashMapy z klasy AccountMaker, ale bez skutku. Może powinienm użyć czegoś innego aniżeli HashMapy?

Z góy dziękuję za pomoc.

Answer 1

W tej metodzie logIntoAccount to powinieneś podawać hasło i login który chcesz sprawdzić. To tak na szybko:
 

  private boolean logIntoAccount(String login, String password) {
loginDetails = accountMaker.getLoginDetails(); 
          if (loginDetails.containsKey(login) && loginDetails.get(login).equals(password) {
         return true;
        } else {
            return false;
        }
    }

Wcześniej jednak inicjalizujesz accountLogger i wywołujesz na nim metodę logIntoAccount. Ja bym jeszcze się zastanowił nad nazewnictwem np. logIntoAccount oraz pisał CreateAccount z małej litery. 

Później jak już ograniesz trochę javy to zainteresuj się bazą danych lub poćwiczy zapisywanie haseł do pliku, 

Comments

Mam na githubie. Wkleiłbym link, gdybym wiedzial ze tak bedzie lepiej :D Na SO właśnie trzeba wklejać fragmenty kodu. :P Ja usunąłem po prostu to co dałem w pierwszym poście tutaj, bo uważałem, ze jest to niepotrzebne. 

No tak, ja już problem mam rozwiązany. Tylko nie rozumiem dlaczego: 

private HashMap<String, String> loginDetails = accountMaker.getLoginDetails();

To nie sprawia, że nie działam na tej samej mapie.

---

Ponieważ accountMaker utworzone w Main i AccountLogge to dwa różne obiekty. Możesz zmienić ich nazwy np. na accountMaker1 i accountMaker2 i nie będziesz mieć błędu.

---

Rozumiem, że mam juz dwa różne obiekty.

Teraz mam to tak zrobione:

https://github.com/must1/ToDoApp

Za pomocą tego konstuktora teraz wszystko działa.

Tylko nie wiem czy dobrze to rozumuje. Przesyłam do AccountLogger obiekt accountMaker, przez co jak używam metod z accountmaker to wszystko się tak jakby zapisuje w AccountLoggerze.

---

W main masz

AccountLogger accountLogger = new AccountLogger(accountMaker);

przez co w konstruktorze bierzesz accountMaker utworzony z maina i przypisujesz go do accountMaker utworzonego w AccountLogger. Czyli operujesz później na jednym obiekcie.

this.accountMaker (z AccountLogger) = accountMaker (z Main);

---

Pierwszy raz się z takim czymś spotykam dlatego pytam:P Wciąż jest to dla mnie zawiłe, ale na pewno bardziej przejrzyste niż wcześniej.

Dzięki wielkie ;)!

Answer 2

Niby pomysł dobry, ale wydaje mi się, że nie do końca. Popatrz na to tak. Login i hasło są ze sobą ściśle powiązane więc lepiej będzie jeśli do tego celu stworzysz osobną klasę User z dwoma polami. W mapie możesz trzymać sobie obiekty klasy User. Takie podejście jest bardziej obiektowe niż Twoje. Mapy służą do przechowywania obiektów według jakiegoś klucza, a ten klucz ma jednoznacznie identyfikować cały obiekt w tym wypadku User'a. Jako klucz można zastosować jego login, co prawda będzie to dublowanie informacji, bo w obiekcie User będzie też takie pole, ale w tym wypadku login będzie identyfikował jednoznacznie cały obiekt (można powiedzieć że login będzie polem za indeksowanym co ułatwi wyszukiwanie).

Jak operować na mapie. To musisz wygooglować ponieważ to są podstawy Javy więc znajdziesz mnóstwo poradników w sieci.

Jeśli ktoś podał Ci login to możesz prosto pobrać wartość z mapy poprzez metodę get() (nie musisz iterować po całej mapie, bo masz klucz).

Samo logowanie możesz zrobić tak, że pobierasz z mapy obiekt User, i sprawdzasz prostym ifem czy hasło się zgadza z hasłem podanym przez użytkownika (loginu nie trzeba sprawdzać bo jest sprawdzany z automatu - jeśli login jest zły to hasła i tak nie będą pasować (no chyba że ktoś ma takie samo, ale na to już nie ma rady) lub metoda get zwróci null bo nie będzie użytkownika w "bazie").

Comments

Niestety niezbyt rozumiem ideę tego wszystkiego.

Tworzę klase User w której mam dwa pola dajmy na to:

String name;
String password

W klasie AccountLogger tworzę:

HashMap<User, String> loginDetails = new HashMap<>();

Czy jak? Kompletnie nie rozumiem tego pomysłu ani jego realizacji. Mimo to, dziękuję za odpowiedź :)!

 

---

HashMap<User, String> loginDetails = new HashMap<>();

Odwrotnie, pierwsza wartość w mapie to klucz, a więc:

HashMap<String, User> loginDetails = new HashMap<>();

Jeśli masz już to, to dalej jest już łatwo bo:

Dodajesz użytkownika poprzez:

String login = "Login";
String password = "haslo";
User newUser = new User(login, password);

loginDetails.put(login, newUser);


a tak pobierasz:

User user = loginDetails.get(login);

//ewentualne sprawdzenie hasla podczas logowania:
if (user.getPassword().equals(password) )
{
    //jakies akcje
}
else
{
    System.out.println("Błędny login lub hasło");
}

trzeba pamiętać ze loginDetails.get(login); może zwrocić nulla.

 

---

Bardzo ciekawe rozwiązanie, przy kolejnej apce, a mam już w planach użyję tego.

Tymczasem skoro mam już coś zrobione, to użyje rozwiązania @miro.

Dzięki!

---

@mbabane, hej, tak jak juz wspomnialem muszę użyć Twojego rozwiązania. Lecz zacząłem się w tym mieszać przy przerabianiu wszystkiego...ale działa.

Nie rozumiem tego pobierania u Ciebie.

Dlaczego tak robimy:

User user = loginDetails.get(login);

przypisujemy do obiektu user klucz naszej Hashmapy, czyli login i password, kompletnie nie rozumiem tego ruchu.

---

@mbabane, Zaktualizowałem mojego githuba, zeby był lepszy pogląd na to: https://github.com/must1/ToDoApp

---

Ten kawałek kodu pobiera z mapy obiekt, który znajduje się pod kluczem login.

Może przeanalizuj ten przykład. Jeśli dalej nie będziesz za bardzo rozumiał to poczytaj o mapach w javie:

import java.util.HashMap;

public class HashMapExample
{
    public static void main(String[] args)
    {
        HashMap<String, User> userHashMap = new HashMap<>();

        String login;
        String password;

        System.out.println("Dodaje nowe usera");
        login = "login";
        password = "password";
        User user = new User(login, password);
        userHashMap.put(login, user);

        System.out.println("Pobranie z mapy usera o loginie: " + login);
        User user1 = userHashMap.get("login");
        System.out.println(user1);


        System.out.println("Dodanie kolejnego uzytkownika");
        login = "must";
        password = "1234";
        User user2 = new User(login, password);
        userHashMap.put(login, user2);

        System.out.println("Pobranie z mapy usera o loginie: " + login);
        User user21 = userHashMap.get("must");
        System.out.println(user21);

        System.out.println("Próba pobranie użytkownika o loginie, ktorego nie ma w mapie");
        User user3 = userHashMap.get("loginKtoregoNieMaWMapie");
        System.out.println(user3);

    }
}

class User
{
    private String login;
    private String password;

    public User(String login, String password)
    {
        this.login = login;
        this.password = password;
    }

    @Override
    public String toString()
    {
        return "User{" +
                "login='" + login + '\'' +
                ", password='" + password + '\'' +
                "}\n";
    }
}

 

---

W sumie nie pamiętam już czego dokładnie nie rozumiałem rano, ale teraz jeszcze bardziej mi się rozjaśniło.

Dzięki wielkie :)