Skip to content

50 pow x n#46

Open
kitano-kazuki wants to merge 9 commits into
mainfrom
50-pow-x-n
Open

50 pow x n#46
kitano-kazuki wants to merge 9 commits into
mainfrom
50-pow-x-n

Conversation

@kitano-kazuki
Copy link
Copy Markdown
Owner

@kitano-kazuki kitano-kazuki commented May 9, 2026

Comment thread memo.md
x = 1 / x
n *= -1

multiplier = 1
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

余分にかけたxはまとめて反映していると思いますが、multiplierだとその意図がやや伝わりにくい気がします
remaining_productとかextra_factorなどはどうでしょうか

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

その方が良いと思いました!

Comment thread memo.md
Comment on lines +137 to +144
* 自然言語でのループ部分の引き継ぎの説明
* x^(2^k)を計算します
* nの2^kに対応するbitが0
* 自分の役目は終了
* 次に見るべきbitを引き継ぐ
* nの2^kに対応するbitが1
* 結果に対してx^(2^k)をかける
* 次に見るべきbitを引き継ぐ
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ビット演算での解き方もあるんですね。勉強になりました。

@5103246
Copy link
Copy Markdown

5103246 commented May 17, 2026

指摘されていた箇所以外は気になりませんでした。全体的に読みやすかったです。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants