50. Pow(x, n) by tom4649 · Pull Request #43 · tom4649/Coding

tom4649 · 2026-04-05T01:41:00Z

https://leetcode.com/problems/powx-n/submissions/1969061269/

hroc135 · 2026-04-05T03:06:26Z

+        if n == 0:
+            return 1.0
+        if n < 0:
+            return 1.0 / self.myPow(x, -n)


個人的には return self.myPow(1.0/x, -n) の方が好みです。
x^n = (1/x)^(-n) と書く方が、x^n = 1 / (x^(-n)) より自然だと思ったからです

そちらを自然だと考える理由はなるほどです。
しかし逆数をとった時に生じる誤差が累積されることも考えて今回はこのままにしておこうと思います。
こういう複数の選択肢を選べるようにしておきたいです。

オーバーフローを引き起こしているので、ここでは良い選択肢とは言えないかもしれません。

「指数を半分にしつつ、底を毎回 x**2 に置き換える」やりかただとオーバーフローが生じたので、底を変えない形にした

こちらについてですが、オーバーフローした時に、**演算だとエラーを投げますが、*演算だとinfを返すので結果的に治っているという形かと。数値的にはpow(x ** 2, n // 2)も、pow(x, n // 2) ** 2も同じですよね。

ご指摘の通りで、x*xを渡すようにしたらエラーはなくなりますね。

**演算だとエラーを投げますが、*演算だとinfを返す

これは知らなかったです。勉強になりました

hroc135 · 2026-04-05T03:08:22Z

+        half = self.myPow(x, n // 2)
+        if n % 2 == 1:
+            return half * half * x
+        return half * half


x^n * y を計算するヘルパーを定義する方法もありますね

これを用いると末尾最適化ができそうですが、Pythonでは末尾最適化が行われないので、Trampolineという手法があることを知り、これを使って実装してみました

trampoline 知らなかったので勉強になりました！

hroc135 · 2026-04-05T03:12:16Z

+
+https://github.com/hroc135/leetcode/pull/43/changes#diff-788d4e4d3d8fc9d0d4ea879b815deba3b2c21e39a44274fe799eaa496c5d2e43
+
+組み込み関数のコードを読んでいる


nit: 組み込み関数ではなく、標準ライブラリですね。少なくとも自分が良く扱う言語ではここの区別はできた方がいいと思いました！

math.Powとありましたね。失礼しました。

liquo-rice · 2026-04-05T07:03:56Z

+
+sol2.pyを再帰でかいた。
+
+「指数を半分にしつつ、底を毎回 x**2 に置き換える」やりかただとオーバーフローが生じたので、底を変えない形にした


これはもうちょっと深掘りしてほしいですね。x * xで底を渡してやると動くと思います。

ここでもコメントいただいていましたね、すみません。上で返答させていただきました。

liquo-rice · 2026-04-05T07:09:58Z

+        shifted = abs(n)
+        current_pow = x
+        while shifted > 0:
+            if shifted & 1:


これはなんらかの意図で、//や%ではなく、ビット演算を行なっているんですかね？

ビット演算の方が高速だと思います。また//や%よりも一般的だと個人的には思います

自分の感覚だと%の方が直感的だと思いました

直感的なのは剰余演算かもしれないですね。Pythonは高速化の恩恵がなさそうなので%で十分かもしれません

どちらが一般的かは文脈によると思いますが、この問題の場合だと、x^2n = (x^n)^2, x^(2n+1)=x(x^n)^2を素直に表現するなら、//, %かなと。

ビット演算はCPUの動作に忠実で効率も良いため計算の意図を正確に表す書き方だと思っていました。
どちらも選べるようにしておこうと思います。

2進数のメンタルモデルで考えているなら、ビット演算で処理をするのも自然だと思います。この問題だとどっちもありに思いました。扱っている対象の抽象度に応じて適切な表現は変わりそうですね。

liquo-rice · 2026-04-05T07:14:32Z

+        shifted = abs(n)
+        current_pow = x


変数名がイマイチに感じます。下のhalfも同様です。

shifted: the variable I'm going to shift later. I won't tell you what it is, though. No spoiler.
って感じですね。

shiftedはexponentにしました。halfは個人的にはこれで良いかと思ったので差し当たりこのままにしておきます。

一応、何が違和感があるのか説明しますと、半分にしたのは、指数部分ですよね。2^2を2^4の半分とは呼びませんよね。

修正しました。ご指摘ありがとうございます。

nodchip · 2026-04-06T10:08:56Z

+        if n == 0:
+            return 1.0
+        if n < 0:
+            return 1.0 / self.myPow(x, -n)


個人的には、引数を書き換えるために関数を再帰的に呼ぶのは、ややパズルに感じます。引数を直接書き換えてしまったほうがシンプルに感じます。

if n < 0: x = 1.0 / x n = -n

趣味の範囲かもしれません。

なるほど、再帰的に呼ばなくても解決するのですね。気づきませんでした。
revisedのファイルで採用させていただきました。

50. Pow

9aa9c27

hroc135 reviewed Apr 5, 2026

View reviewed changes

liquo-rice reviewed Apr 5, 2026

View reviewed changes

tom4649 added 3 commits April 5, 2026 19:36

Add tail_recursive

5838c33

Change shifted to exponent

dd1d0dd

Add suggested changes

bf38e66

nodchip reviewed Apr 6, 2026

View reviewed changes

Add suggested changes

2ab6bef


		https://github.com/hroc135/leetcode/pull/43/changes#diff-788d4e4d3d8fc9d0d4ea879b815deba3b2c21e39a44274fe799eaa496c5d2e43

		組み込み関数のコードを読んでいる


		sol2.pyを再帰でかいた。

		「指数を半分にしつつ、底を毎回 x**2 に置き換える」やりかただとオーバーフローが生じたので、底を変えない形にした

Conversation

tom4649 commented Apr 5, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hroc135 Apr 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

hroc135 Apr 5, 2026 •

edited

Loading