if の中にimport

Question

list = input()

if len(list) > 1:
    import random
    random.shuffle(list)

リストの要素が0や1だとシャッフルしても意味が無いので、randomモジュールをimportする必要も無いですよね？ このような時にif文を用いてimportするのは有りなのでしょうか？ それでも最初にrandomモジュールをimportするべきなのでしょうか？

Answer 1

そうする必要がないなら冒頭でimportすることが推奨されています。

とても時間がかかる処理

if なにか:
    import randam

とか

for d in すごくたくさんのループ:
    if 滅多に起こらない条件:
        import randam

みたいな馬鹿なミスを避けるために、失敗するべき時はなるべく早いうちに失敗してエラーを出してもらった方がいいので。
fail-fast の考えでしょう。

---

(追記)

https://www.python.org/dev/peps/pep-0008/#imports

Imports are always put at the top of the file, just after any module comments and docstrings, and before module globals and constants.

Comments

[mito.2357]

必要性が無いなら冒頭importが推奨されているのですね。
ありがとうございます！

分からなった点があります。
滅多に起こらない条件で、importするのはあまり良くないのですか？ 個人的には良く感じます。

[quickquip]

ほら、スペルミスで import がエラーになることにすぐに気づかなかったでしょう?
先頭に書いていたら起動してすぐにエラーになって分かったはずのことが、余計なことをしたせいでしばらく時間が経ってから間違いに気づく、という事態を引き起こすのです。

[mito.2357]

なるほど笑笑
このような間違いもありうるので、
エラーを防ぐ上では冒頭importの方が良いということですね。体験させてくださりありがとうございます！

[quickquip]

タイプミスはいつでもどこでもやってしまう可能性が高いので、想定するべきは標準以外のライブラリのインストール不備でしょうけれども。
ある環境で動いていたスクリプトを別の環境に持ってきた時に、import失敗が即座に出ないケースが面倒そうですね。

[mito.2357]

返信遅れて申し訳ありません。
なるほど。詳しくありがとうございます。私はまだ初心者なので頭の片隅に置いておいておこうと思います。

Answer 2

アリですけれど、if文がスコープを作らない以上意義は薄いと思います。

lst = [1, 2, 3]

if len(lst) > 1:
    import random
    random.shuffle(lst)
   
random.shuffle(lst)   # 外でも普通にrandomモジュールを使える

私はスコープの先頭、つまりこの場合モジュールの先頭でimportすることがより自然に思えます。

これはOK

tf = True
if tf:
    import random

random.shuffle

これはNG

tf = False
if tf:
    import random

random.shuffle  # => NameError: name 'random' is not defined

特に真偽値が外部からの操作に依って決まるのであれば、影響を考慮しづらくなります。

Comments

[mito.2357]

スコープを作る場合は有りということですね！ 勉強になりました。
ありがとうございます！

Answer 3

エラーの可能性を減らしたいのであれば、quiquiさんのおっしゃる通り冒頭でimportするほうがいいでしょう。

ただ、「処理速度を上げる」という観点から考えれば、if文を用いてimportするのは有りだと思います。
下記のリンクによると、モジュールを(初回)importする時間は実行時間に大きく影響する、とされています。容量の大きいモジュールならばなおさら実行時間への影響は大きくなります。

Pythonの処理速度を上げる方法　その３

開発時の制約や要望によっては、あえてif文を用いてimportすることが正解になる場合もあるかもしれません。…極稀だとは思いますが。

Comments

[mito.2357]

なるほど、
エラーを減らすなら冒頭import。
処理速度を上げるなら有り。ですか、
非常に分かりやすかったです。
ありがとうございました！

Answer 4

普通に有りだと思います。
特に積極的にそうする理由がないのであれば、必要なタイミングでimportすればよいです。

Comments

[can110]

そっか。if文スコープ作らないのは気づかなかった…

[mito.2357]

迅速な回答ありがとうございます！
importは冒頭でしないといけない。という風に聞いていたので、ある程度好きなタイミングで良いというのは大変参考になりました。
ありがとうございます！