質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

1817閲覧

any()が遅いのは書き方が悪いため?

退会済みユーザー

退会済みユーザー

総合スコア0

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

1クリップ

投稿2020/03/21 01:22

素数判定の関数で、小さい素数の倍数かを判定するためにif n % 2 == 0 or n % 3 == 0 or n % 5 == 0 or n % 7 == 0 or n % 11 == 0:と書いていたのをsmall_primes = {2, 3, 5, 7, 11}; if any(n % p == 0 for p in small_primes):に変えたところ、倍近く時間がかかるようになりました。
small_primesと変数を置かずにin [2, 3, 5, 7, 11]としても遅いままです。
any()は短絡評価だからそう遅くはならないと思っていたのですが、これは私の書き方が悪いのでしょうか?

Python

1def is_prime_miller_rabin(n: int) -> bool: 2 if n < 2: 3 return False 4 elif n in {2, 3, 5, 7, 11}: 5 return True 6 elif n % 2 == 0 or n % 3 == 0 or n % 5 == 0 or n % 7 == 0 or n % 11 == 0: 7 return False 8 # 以下に書き換えると遅くなる 9 # small_primes = {2, 3, 5, 7, 11} 10 # if n < 2: 11 # return False 12 # elif n in small_primes: 13 # return True 14 # elif any(n % p == 0 for p in small_primes): 15 # return False 16 17 if n < (1 << 32): 18 L = [2, 7, 61] 19 elif n < (1 << 48): 20 L = [2, 3, 5, 7, 11, 13, 17] 21 else: 22 L = [2, 3, 5, 7, 11, 13, 17, 19, 23, 29] 23 24 d = n - 1 25 d = d // (d & -d) 26 27 for a in L: 28 y = pow(a, d, n) 29 if y == 1: 30 continue 31 t = d 32 while y != n - 1: 33 y = (y * y) % n 34 if y == 1 or t == n - 1: 35 return False 36 t <<= 1 37 return True 38

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

もともと軽い処理なので、ジェネレーターを作って呼び出す部分のオーバーヘッドが比較的に大きいのだと思います。
素数チェック範囲を広げるなど全体が重くなれば、オーバーヘッド分の割合が減るかと思います。

投稿2020/03/21 01:45

otn

総合スコア85901

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2020/03/21 04:21

cProfileで確認すると確かに、genexprが重いですね。 試しに100以下の素数まで広げてみましたが、あまり効果はありませんでした。 大人しく or で書くことにします。 ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問