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

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

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

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

Python

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

Q&A

解決済

2回答

1584閲覧

pythonのリスト内包表記でelseで複数の処理を行いたい。

ImR0305

総合スコア2

for

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

Python

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

0グッド

1クリップ

投稿2020/11/11 19:48

編集2020/11/12 03:00

#pythonのリスト内包表記でelseで複数の処理を行いたい。
以下のようなfor文を内包表記で書くことは可能でしょうか?

※get_hash関数は与えられた変数に対して文字列を返す関数です。

##もともとやりたい処理
datetimeは時間、user_idはユーザーIDでこれらはログのデータになります。
同一ユーザーIDで30分以内でしたら、同一のセッションとみなし、user_id+そのセッションの最古のdatetimeでハッシュ化を行います。

python

1#get_hash関数は与えられた変数に対して文字列を返す関数です。 2def get_hash(text1,text2): 3 text = str(text1)+str(text2) 4 session_id=hashlib.sha256(text.encode()).hexdigest() 5 return session_id 6 7 8text=[] 9j=0 10for i in range(len(user_id)): 11 if user_id[i]==user_id[j] and datetime[i]-datetime[j] <=1800: 12 text.append(get_hash(user_id[i],datetime[j])) 13 14 else: 15 text.append(get_hash(user_id[i],datetime[i])) 16 j=i

このj=iが内包表記でうまく表せず、悩んでいます。
これを内包表記で書ければご教示いただけると幸いです。
(内包表記を使う理由が速度を上げたいからなので、このコードに速度を上げる余地があれば教えていただけると幸いです。)

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

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

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

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

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

toast-uz

2020/11/11 23:22 編集

前提としているlen(user_id))はどのくらいで、質問者様の環境ではどのくらいの時間がかかるのでしょうか? pandasを使えばリスト内包処理よりもさらに劇的に速くなりそうですが、前後の比較処理のところがやや難しく(コーディングできるけど逐次的な処理になりがちなので、速度的にボトルネック)、実際のどのくらい速くなるのかを見たいと思いました。
quickquip

2020/11/11 23:37

get_hashは Pythonで書かれた普通の関数 でしょうか? もしそうなら 内包表記にするだけで速くなるわけではない が事実なのでそういう回答が集まるだけと思いますよ。 (XY問題 https://ja.meta.stackoverflow.com/questions/2701/#answer-2702 で、やりたいことに対して間違った方法の質問をしてしまっています) あと user_id や datetime は list型 なのでしょうか?
ImR0305

2020/11/12 02:49

user_idは13000くらいで0.5秒前後になります。 実はこれらはもともとpandasのデータフレーム要素でそれらをリスト化したものになります。 get_hashはpythonで書かれた関数になります。 (コードに追加しました。)
toast-uz

2020/11/12 03:38

もともとpandasなのであれば、わざわざリストにして内包表現といったミスリードはせずに、そのまま高速化の手段を聞いた方が良いですね。pandasはfor(内包表現含む)をいかに使わないで済ませるかがコツですので。もちろん現在のコードやかかっている時間は明記した上でです。さらに、0.5秒の処理を速くするメリットがよくわからないですので、もう少し背景を記載したほうがよいです。回答者は「意義のあること」に回答したいですので、単なる興味関心なのか、意義があることなのか、わかるとよいです。 この質問はどれかにベストアンサー出してクローズして、質問を立て直すことをオススメします。
ImR0305

2020/11/12 03:53

ご教示ありがとうございます。 そうさせていただきます
guest

回答2

0

テストできてないですが、こんな感じですかね。

Python

1text = [ get_hash(user_id[i],datetime[j]) 2 if user_id[i]==user_id[j] and datetime[i]-datetime[j] <=1800 3 else get_hash(user_id[i],datetime[i])+0*(j:=i) 4 for i in range(len(user_id)) ]

有意に速くなる気はあまりしませんが。

投稿2020/11/11 23:05

otn

総合スコア84491

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

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

0

ベストアンサー

python

1j = 0 2 3def step(i): 4 global j 5 if user_id[i]==user_id[j] and datetime[i]-datetime[j] <=1800: 6 h = get_hash(user_id[i],datetime[j]) 7 else: 8 h = get_hash(user_id[i],datetime[i]) 9 j=i 10 return h 11 12 13text = [step(i) for i in range(len(user_id))]

速度が上がるかわかりません。

投稿2020/11/11 20:42

unknown_user

総合スコア68

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.49%

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

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

質問する

関連した質問