Próba przekszałcenie kodu proceduralnego w typowo obiektowy

Witam ponownie . Zacząłem się bawić w obiektówkę . Przykładowo mam kod proceduralny z liczeniem wieku :

import datetime

def pobierz_imie():
    return input('podaj imie: ')

def pobierz_rok_urodzenia():
    return int(input('podaj rok urodzenia: '))

def policz_ile_lat(rok_urodzenia, aktualny_rok):
    return aktualny_rok - rok_urodzenia

def odpowiedz_uzytkownikowi(imie, wiek):
    print(f'Czesc {imie}! Masz {wiek} lat.')

while True:
    imie = pobierz_imie()
    rok_urodzenia = pobierz_rok_urodzenia()
    aktualny_rok = datetime.datetime.now().year
    wiek = policz_ile_lat(rok_urodzenia, aktualny_rok)
    odpowiedz_uzytkownikowi(imie, wiek)

Kod w obiektówce:

import datetime

class Dane:
    def __init__(self):
        self.imie = self.Pobierz_imie   #konstruktor/  - powinno być self.imie = imie /jak takzrobię mam bład
        self.rok_urodzenia = self.pobierz_rok_urodzenia()
        self.aktualny_rok = datetime.datetime.now().year
        self.wiek = self.policz_ile_lat(self.rok_urodzenia, self.aktualny_rok)
        self.odpowiedz_uzytkownikowi(self.imie, self.wiek)

    def Pobierz_imie(self):
        return input('Podaj imie: ')

    def pobierz_rok_urodzenia(self):
        return int(input('podaj rok urodzenia: '))
        

    def policz_ile_lat(self,rok_urodzenia, aktualny_rok):
        return self.aktualny_rok - self.rok_urodzenia

    def odpowiedz_uzytkownikowi(self,imie,wiek):
        print(f'Czesc {self.imie}! Masz {self.wiek} lat.')

    def wyswietl(self):
      #print(self.Pobierz_imie,self.pobierz_rok_urodzenia,self.odpowiedz_uzytkownikowi,self.policz_ile_lat)
      print(self.imie,self.aktualny_rok,self.rok_urodzenia,self.wiek)
      

dane1 = Dane()
dane1.wyswietl()

I ten program próbuję przekształcić w typową obiektówkę... Niby wynik obliczenia wieku mi wychodzi (z błedami) , ale nie mam pytania o imie, żebym podał.... Tylko od razu pytanie o rok urodzenia... Czy w konstruktorze klasy dobrze dodałem te wszystkie "dane"? Powinno być self.imie = imie, self.rok_urodzenia = rok_urodzenia itp..., a zrobiłem trochę inaczej ! Czy jest możliwość całkowitego przerobienia tego programu w typową "obiektówkę" Ogólnie czy dobrze kombinuję, jeśli chodzi o przerobienie tego kodu na obiektówkę? Chętnie bym zobaczył prawidlowy kod obiektowy według was, zebym sobie porównał lub wskazówki co robię zle. Pozdrawiam

W konstruktorze powinno być self.imie = self.Pobierz_imie(), dlatego od razu pyta o rok urodzenia.
Generalnie to rozdziela się pobieranie danych od obiektów na których pracujesz, a dane umieszczasz w obiekcie przekazując je przez konstruktor np.

import datetime


class Dane:
    def __init__(self, imie, rok_urodzenia):
        self.imie = imie
        self.rok_urodzenia = rok_urodzenia
        self.aktualny_rok = datetime.datetime.now().year
        self.wiek = self.policz_ile_lat()
        self.odpowiedz_uzytkownikowi(self.imie, self.wiek)

    def policz_ile_lat(self):
        return self.aktualny_rok - self.rok_urodzenia

    def odpowiedz_uzytkownikowi(self):
        print(f'Czesc {self.imie}! Masz {self.wiek} lat.')

    def wyswietl(self):
        print(self.imie, self.aktualny_rok, self.rok_urodzenia, self.wiek)


def Pobierz_imie():
    return input('Podaj imie: ')


def pobierz_rok_urodzenia():
    return int(input('podaj rok urodzenia: '))


pobrane_imie = Pobierz_imie()
pobrany_rok = pobierz_rok_urodzenia()

dane1 = Dane(pobrane_imie, pobrany_rok) # można też dać Dane(rok_urodzenia=pobrany_rok, imie=pobrane_imie) wtedy kolejność parametrów nie ma znaczenia
dane1.wyswietl()

Nie musisz też przekazywać zmiennych to funkcji wewnątrz klasy jak np. w twoim policz_ile_lat. Ustawiasz wartości dla rok_urodzenia i aktualny_rok w konstruktorze, a potem możesz się odwołać do nich przez self.rok_urodzenia i będzie tam dokładnie to co ustawiłeś.

Ja bym te inputy wyrzucil poza klase i przekazywal jako atrybut do obiektu, cos jak kolega up ;)

Ja bym nie dodawał pola aktualny_rok - to powinno sie liczyc na bieżąco.

Niespójność: napisałeś metodę Pobierz_imie(self) nie wiadomo dlaczego z dużej litery (później autocomplete rozumiem, że umocniło ten błąd. Czy raczej nie tyle błąd, bo to jest technicznie poprawne, tylko niespójność stylu).

Ja bym sobie kod podzielił na klasę "ludka" i klasę odpowiadającą za komunikację z użytkownikiem. Klasy w osobnych plikach. Aktualny rok dałbym jako metodę prywatną, albo property nie widoczne za weznątrz. Wiek wystawione jako property readonly. Wyświetlanie obiektu poprzez __str__. Do obiektu osoby dopisałbym testy liczenia wieku. Komunikację rozszerzył o logi. Brakuje też zabezpieczeń przed niewłaściwymi danymi.

osoby.py:

from datetime import datetime

class Osoba:
    def __init__(self, imie=None, rok_urodzenia=None):
        self.imie = imie
        self.rok_urodzenia = rok_urodzenia

    @property
    def __aktualny_rok(self):
        return datetime.now().year

    @property
    def wiek(self):
        if not self.rok_urodzenia: return None
        return self.__aktualny_rok - self.rok_urodzenia

    def __str__(self):
        return f'{self.imie} {self.__aktualny_rok} {self.rok_urodzenia} {self.wiek}'

komunikacja.py:

class Komunikacja:
    def pobierz_imie(self):
        return input('Podaj imie: ')

    def pobierz_rok_urodzenia(self):
        return int(input('podaj rok urodzenia: '))

    def odpowiedz_uzytkownikowi(self, osoba):
        print(f'Cześć {osoba.imie}! Masz {osoba.wiek} lat.')

main.py:

from osoba import Osoba
from komunikacja import Komunikacja

def main():
    komunikacja = Komunikacja()
    osoba = Osoba()
    osoba.imie = komunikacja.pobierz_imie()
    osoba.rok_urodzenia = komunikacja.pobierz_rok_urodzenia()
    print(osoba)
    komunikacja.odpowiedz_uzytkownikowi(osoba)
    
if __name__ == '__main__':
    main()

Dzięki Wam wszystkim za pomocne informacje....

@Arthan Jeśi chodzi o zabezpiecznie danych przed nieprawidłową odpowiedzią , czy mogę tu w klasach zastosować tzw. wyjątki tzn:

def pobierz_rok_urodzenia(self):
        while True: 
            try:
                return int(input('podaj rok urodzenia: '))
            except ValueError:
                print('Wpisałeś złą wartosć, wpisz liczbę jeszcze raz')

Z wyjatkiem kod działa, tylko czy mogę to w klasach stosować?. Chodzi mi o poprawność kodu w obiektówce. Tzw while , for to tu się w obiektówce stosuje podobnie jak w proceduralnym kodzie? , co do testów, logów , może pokombinuję (choć nie wiem do konca o co chodzi tu ), co do property to wiedzę sobie uzupełnię na ten temat

Generalnie założyłbym, że możesz robić w klasach wszystko, co do tej pory robiłeś. Obiekty służą tylko po to, żeby opakować dane w wygodne pudełka i zdefiniować operacje, które można na nich wykonywać.

Dzięki za info, wskazówki.... W wolnej chwili posprawdzam co i jak.... Tak się ostatnio zastanawiałem nad bootcampem python ( dostaję od nich zaproszenie na szkolenia online/stacjonarne) , trochę to kosztuje.... Ale jednak wolę iść samemu w naukę, online, ksiązki , fora... Po pierwsze szkoda mi kasy na to ( jak sam mogę wiedzę zdobyć) ale za to więcej wysiłku i czasu mi to zajmnie, bo nie mam nauczyciela który by poprawiał moje błedy w programowaniu. Czasowo dłużej nauka mi zajmnie jak się domyślam. Druga sprawa to czas, którego za bardzo nie mam i bym się musiał dostosować do rytmu szkolenia takiego. A u mnie z tym byłoby krucho. Na razie kupiłem cienką książkę z przykładami ćwiczeń , zadaniami, i wytłumaczeniami jak co działa. Po trochu coś tam zdziałam dla siebie. A po trzecie wolę wydać kasę na jakieś ebooki, filmiki instruktażowe itp...., niż ponad 10 K za szkolenia z którymi jestem uwiązany czasowo. Opinie na temat bootcampów są zróżnicowane w sieci. . Pozdrawiam

@ObywatelRP: Własnie analizuję kod, uruchamia się z błedem.
W klasie Dane na końcu powinno być bez argumentów: self.imie , self wiek tzn:

 self.odpowiedz_uzytkownikowi()

choć jak tej funkcji w ogóle nie podam , to kod też się wywołuje poprawnie.
Grunt , ze kod się uruchamia :) A tak z ciekawośći, dlaczego dwie ostatnie funkcje w kodzie są akapitami różne? Tak sobie porównuję do innych kodów.

  
      def wyswietl(self):
        print(self.imie, self.aktualny_rok, self.rok_urodzenia, self.wiek)

def Pobierz_imie():
        return input('Podaj imie: ')

def pobierz_rok_urodzenia():
        return int(input('podaj rok urodzenia: '))

Generalnie, to może zacznijmy od tego, że python to taki śmieszny język, gdzie wcięcia służą do definiowania bloków kodu

def test_indent():
    if False:
        print("To się nie wyświetli")
        print("To też się nie wyświetli")
    print("A to już, tak bo jest inne wcięcie")

test_indent()

Tak samo jest z klasami

class Clazz:
    def method_in_class(self):
        print("method")

    def method2_in_class(self):
        print("method 2")

def method3_not_in_class(self):
    print("method 3")


clazz = Clazz()
clazz.method_in_class()
clazz.method2_in_class()
clazz.method3_not_in_class() # error

Nie chciałem, żeby metody pobierające dane były w klasie, która na nich operuje, więc nie są na tym samym poziomie wcięcia . Popatrz na kod, który dał ci @Arthan na poprzedniej stronie, Klasa osoba ma tylko metody służące do operowania na danych, które dostaje, a cała reszta, czyli pobieranie danych jest w klasie Komunikacja. U mnie też można by dać je w osobnej klasie, ale jestem leniwy i nie chciało mi się tego pisać, więc dałem tylko inne wcięcie żeby zaznaczyć, że te metody nie powinny tam być.

@ObywatelRP: Dzięki za info. Dla testów dałem to w osobnej klasie (inny plik) , działa bez problemu :)