Skip to content

Conversation

yusuf601
Copy link
Contributor

@yusuf601 yusuf601 commented Oct 8, 2025

Deskripsi (Description)

Menambahkan source code baru untuk mengimplementasikan frekuensi harapan. Frekuensi harapan (Expected Frequency) adalah jumlah kejadian yang diharapkan terjadi berdasarkan peluang.
Contoh output

Koin (HEAD 1/2) dari 60 kali lempar: 30
Kartu Remi (AS) dari 60 kali ambil: 1.15385
Dadu (angka 6) dari 60 kali lempar: 10
Dadu (angka 1 atau 2) dari 60 kali lempar: 20
Dadu custom (genap) dari 60 kali lempar: 30
Distribusi {2,3,5}, peluang event bobot=3 dari 60 kali percobaan: 18
Negasi frequency (peluang gagal, p=0.4) dari 60 kali percobaan: 36

diharapkan compile dengan flags c++20 misal pada clang

clang++ -std=c++20 frekuensi_harapan.cpp -o m   

Checklist:

Umum:
  • Saya menambah algoritma terbaru.
Contributor Requirements (Syarat Kontributor) dan Lain-Lain:
  • Saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua syarat.
  • Saya telah menambahkan komentar kode yang memberikan penjelasan maksud dari kode yang saya buat.
  • Saya menggunakan bahasa Indonesia untuk memberikan penjelasan dari kode yang saya buat.

Environment

Saya menggunakan (I'm using):

  • OS = linux
  • g++ = 15.2.1

Link Issues

Issues : #

@yusuf601 yusuf601 requested a review from slowy07 as a code owner October 8, 2025 14:40
@mergify mergify bot added the cpp file label Oct 8, 2025
@yusuf601
Copy link
Contributor Author

yusuf601 commented Oct 8, 2025

fail test munkin karena pada saat workflow jalan tidak memakai flags CMAKE_CXX_STANDARD=20 saat cmake -B ./build -S . @slowy07

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.

berikan minimal 1 blank space dalam jarak antara header dan namespace

Comment on lines 17 to 18
namespace expected_frequency {
namespace koin {
Copy link
Member

Choose a reason for hiding this comment

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

jarak antara header dan namespace

Comment on lines 79 to 80
} // namespace kartu_remi
namespace dadu {
Copy link
Member

Choose a reason for hiding this comment

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

jarak header antara kartu_remi dan dadu, seterusnya

@yusuf601
Copy link
Contributor Author

karena clang-format mengakibatkan keterbacaan code menjadi tidak begitu baik,harusnya tiap namespace misal Expected Frequency ketika ada nested namespace harus nya diberi jarak 4 space

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants