Skip to content

Conversation

Ateng-labrador
Copy link
Contributor

Deskripsi (Description)

Disini saya menambahkan algoritma baru pada dictionary other tentang lucky number/Angka keberuntungan

Checklist:

Umum:

  • Saya menambah algoritma terbaru.
  • Saya memperbaiki algoritma yang sudah ada.
  • Saya memperbaiki dokumentasi.
  • Saya menambah dokumentasi.

Contributor Requirements (Syarat Kontributor) dan Lain-Lain:

  • Saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua syarat.
  • Saya telah menambahkan docstring yang memberikan penjelasan maksud dari kode yang saya buat.
  • Saya menggunakan bahasa Indonesia untuk memberikan penjelasan dari kode yang saya buat.

Unit Testing dan Linting:

  • pytest
  • flake8

Environment

Saya menggunakan (I'm using):

  • os = windows
  • python = py -3 -V (Windows-Python launcher)

linked issue #

Copy link

@sejutaimpian sejutaimpian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Langkah awal pencarian lucky number adalah mengeliminasi semua bilangan genap, jadi angka 80 seharusnya bukan lucky number. Silakan cek kembali logikanya agar sesuai sebagaimana definisinya di Lucky Number

@Ateng-labrador
Copy link
Contributor Author

@sejutaimpian bisa di cek lagi

Copy link

@sejutaimpian sejutaimpian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saya mencoba mengecek angka 9 untuk memastikan bahwa angka 9 adalah lucky number tapi ternyata hasil dari program menampilkan "Bukan Angka Lucky".

Ini terjadi karena pada iterasi kedua idx bertambah sehingga melewatkan angka 3.

Simulasi
Pada iterasi pertama:
nbr = 1,2,3,4,5,6,7,8,9,10,11,12,13,14,15, ...
idx = 1
step = 2
Maka akan menghapus semua bilangan genap.

Pada iterasi kedua:
nbr = 1,3,5,7,9,11,13,15, ...
idx = 2
step = 5 ini yang jadi masalah karena dia skip angka 3

Solusi terbaiknya adalah langsung saja nbr nya diisi oleh bilangan ganjil. Atau kalau kamu masih mau tetap seperti itu, silakan perbaiki agar logic nya terpenuhi.

Copy link
Member

@slowy07 slowy07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, terima kasih atas kontribusinya @Ateng-labrador

@slowy07
Copy link
Member

slowy07 commented Oct 23, 2025

@bellshade/python-team

@mergify mergify bot requested a review from a team October 23, 2025 02:45
@slowy07
Copy link
Member

slowy07 commented Oct 23, 2025

build fail

@Ateng-labrador
Copy link
Contributor Author

@sejutaimpian baik terimah kasih atas masukannya 🙏 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants