bellshade / Python

Repositori untuk belajar pemrograman Python dalam bahasa Indonesia
MIT License
157 stars 95 forks source link

feat: menambahkan algoritm counter(yaudah) #273

Closed kayabaakihiko13 closed 1 year ago

kayabaakihiko13 commented 1 year ago

Deskripsi (Description)

Checklist:

Umum:

Contributor Requirements (Syarat Kontributor) dan Lain-Lain:

Unit Testing dan Linting:

Environment

Saya menggunakan (I'm using):

linked issue #NOMOR_ISSUE

kayabaakihiko13 commented 1 year ago

LGTM, kode diatas terlihat sudah bagus, tapi kurangnya typehinting, jadi untuk kode tersebut mudah dibaca dan mengikuti peraturan dari CONTRIBUTING.md

kamu juga bisa menggunakan fungsi super().__init__() dalam konstruktor kelas dari counter dengan ini sebagai contoh

class Counter(dict):
    def __init__(self, iterable: Optional[Dict[Any, int]] = None, **kwargs: int) -> None:
        super().__init__()
        self.update(iterable, **kwarg

tujuannya antara lain untuk memastikan inisialisassi tepat dari kelas Counter yang bertipe dict

Berati di bagian dict pada class Counter(dict) tidak di perlukan kak?

slowy07 commented 1 year ago
    def __init__(self, iterable: Optional[Dict[Any, int]] = None, **kwargs: int) -> None:

sebenarnya boleh menggunakan kode diatas atau menggunakan contoh yang disarankan, keduanya memberikan informasi yang baik contoh yang diberikan bisa dijadikan acuan untuk improvisasi