ある処理をリスト内法表記や高階関数へ書き換えたいがエラーが出る

Question

ある処理をリスト内法表記や高階関数へ書き換えたいがエラーが出る

受付中

回答 4

投稿 2018/09/07 15:40 ・編集 2018/09/11 06:23

評価
クリップ 0
VIEW 752

hello0130

score 6

前提・実現したいこと

PyQというプログラミング学習サービスで
for row in f:
if 'good' in path: good_list.append(row)
if 'bad' in path: bad_list.append(row)
上記の処理をリスト内法表記や高階関数を使ったものに書き換えて見たのですが、
good_list, bad_listを参照しようとするとエラーが出てしまいます。

なぜなのか考えて見たのですがさっぱり見当がつきません。
よろしくお願いいたします。

発生している問題・エラーメッセージ

Traceback (most recent call last)

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 2309, in __call__

    return self.wsgi_app(environ, start_response)

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 2295, in wsgi_app

    response = self.handle_exception(e)

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 1741, in handle_exception

    reraise(exc_type, exc_value, tb)

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/_compat.py", line 35, in reraise

    raise value

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 2292, in wsgi_app

    response = self.full_dispatch_request()

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 1815, in full_dispatch_request

    rv = self.handle_user_exception(e)

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 1718, in handle_user_exception

    reraise(exc_type, exc_value, tb)

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/_compat.py", line 35, in reraise

    raise value

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 1813, in full_dispatch_request

    rv = self.dispatch_request()

    File "/home/appuser/venv/lib/python3.6/site-packages/flask/app.py", line 1799, in dispatch_request

    return self.view_functions[rule.endpoint](**req.view_args)

    File "/home/appuser/work/main.py", line 32, in index

    fortune['message'] = random.choice(good_list)

    File "/usr/lib/python3.6/random.py", line 260, in choice

    raise IndexError('Cannot choose from an empty sequence') from None

    IndexError: Cannot choose from an empty sequence

該当のソースコード

# 1: 大吉、2: 吉、3: 中吉、4: 小吉、5: 凶、6:大凶
import random
from flask import Flask, render_template, request

app = Flask(__name__)

GOOD_FILE = 'input/fortune_good.txt'
BAD_FILE = 'input/fortune_bad.txt'


@app.route("/")
def index():
    good_list = []
    bad_list = []
    for path in [GOOD_FILE, BAD_FILE]:
        with open(path, encoding='utf-8') as f:
            #うまく動く
            for row in f:
                if 'good' in path: good_list.append(row)
                if 'bad' in path: bad_list.append(row)

            #fortune_list = list(f)

            #リスト内法表記 エラー発生
            #good_list = [row for row in fortune_list if 'good' in path]
            #bad_list = [row for row in fortune_list if 'bad' in path]

            #高階関数へ書き換え エラー発生
            #good_list = list(filter(lambda x: 'good' in path, fortune_list))
            #bad_list = list(filter(lambda x: 'bad' in path, fortune_list))

            #うまく動く
            #good_list.extend([row for row in fortune_list if 'good' in path])
            #bad_list.extend([row for row in fortune_list if 'bad' in path])
    fortune = {}
    if request.args.get('fortune', ''):
        fortune['no'] = random.randint(1, 6)
        if fortune['no'] <= 4:
            fortune['message'] = random.choice(good_list)
        if fortune['no'] >= 5:
            fortune['message'] = random.choice(bad_list)
    return render_template('index.html', fortune=fortune)

試したこと

補足情報（FW/ツールのバージョンなど）

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

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

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

クリップを取り消します
良い質問の評価を上げる

以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します

評価を下げられる数の上限に達しました

評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる

teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します

この機能は開放されていません

評価を下げる条件を満たしてません

評価を下げる理由を選択してください
プログラミングに関係のない質問やってほしいことだけを記載した丸投げの質問問題・課題が含まれていない質問意図的に内容が抹消された質問過去に投稿した質問と同じ内容の質問広告と受け取られるような投稿
詳細な説明はこちら

上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。
質問の評価を下げる機能の利用条件

この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
- メールアドレスの認証
  メールアドレスの認証
- 質問評価に関するヘルプページの閲覧
  質問評価に関するヘルプページの閲覧

8 件の質問への追記・修正依頼

質問への追記・修正、ベストアンサー選択の依頼

quickquip

2018/09/07 15:44 編集

ipythonを使ってませんか? ←すみません、勘違いでした。お気になさらずに

ipythonを使ってませんか? ←すみません、勘違いでした。お気になさらずに

キャンセル
hello0130

2018/09/07 15:55

返信ありがとうございます。ipythonは使っていないのですが、PyQというプログラミング学習サービス上で発生しています。あまり関係ないと思っていましたが、書き換え方に問題がないのであればPyQ固有の問題なのかもしれません。

返信ありがとうございます。ipythonは使っていないのですが、PyQというプログラミング学習サービス上で発生しています。あまり関係ないと思っていましたが、書き換え方に問題がないのであればPyQ固有の問題なのかもしれません。

キャンセル
fuzzball

2018/09/07 16:16

randomが吐いている例外のようなので、質問のコードは関係ないのでは？（正しく動くかどうかは別にして）テストデータ生成にrandom使ってるとか。

randomが吐いている例外のようなので、質問のコードは関係ないのでは？（正しく動くかどうかは別にして）テストデータ生成にrandom使ってるとか。

キャンセル
hello0130

2018/09/07 17:06

回答ありがとうございます。コードを変更するとエラーがでるのでそこが原因だと考えました。エラーメッセージの内容はlistが空だと言っていると理解していますがあっているのでしょうか？エラーメッセージをいまいち理解できていません。エラーコード全文を追記させていただきましたので見ていただけたらと思います。

回答ありがとうございます。コードを変更するとエラーがでるのでそこが原因だと考えました。エラーメッセージの内容はlistが空だと言っていると理解していますがあっているのでしょうか？エラーメッセージをいまいち理解できていません。エラーコード全文を追記させていただきましたので見ていただけたらと思います。

キャンセル
fuzzball

2018/09/07 17:10

このエラーってリスト内法表記だけじゃないですか？高階関数でも出ます？

このエラーってリスト内法表記だけじゃないですか？高階関数でも出ます？

キャンセル
hello0130

2018/09/07 17:30

返信ありがとうございます。PyQには判定ボタンがありコードをテストして確認しているようです。高階関数の場合はたまに合格判定になり、リスト内法表記だと何回押しても合格判定にはなりません。どういうことでしょうか？

返信ありがとうございます。PyQには判定ボタンがありコードをテストして確認しているようです。高階関数の場合はたまに合格判定になり、リスト内法表記だと何回押しても合格判定にはなりません。どういうことでしょうか？

キャンセル
quickquip

2018/09/07 21:23

fortune['message'] = random.choice(good_list) は自分で書いたコードですか? 書いてないコードですか?

fortune['message'] = random.choice(good_list) は自分で書いたコードですか? 書いてないコードですか?

キャンセル
hello0130

2018/09/07 21:59

返信ありがとうございます。その部分は自分で書いた部分です。その部分に適当なリストを入れてみたところエラーが出なくなったのでgood_list自体に問題があるのではと考えています。

返信ありがとうございます。その部分は自分で書いた部分です。その部分に適当なリストを入れてみたところエラーが出なくなったのでgood_list自体に問題があるのではと考えています。

キャンセル

回答 4 件

評価が高い順
新着順
古い順

同じタグがついた質問を見る

Python 3.x

score 1 · Answer 1 · 2018-09-07T16:27

+1

f.readlines()をlist(f)にしてみるとか。

投稿 2018/09/07 16:27

回答の評価を上げる

以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
回答の評価を下げる

下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

2018/09/07 17:16

回答ありがとうございます。試してみましたがエラーが出てしまいました。
同じ処理をしていると理解しているのですが合っているのでしょうか？
さらにそうすることによってエラーがなくなると考えられた意図は何でしょうか？
言葉がとげとげしい気がしますが、そういう意図はありません。単純に知りたいんです。
よろしくお願いいたします。

回答ありがとうございます。試してみましたがエラーが出てしまいました。同じ処理をしていると理解しているのですが合っているのでしょうか？さらにそうすることによってエラーがなくなると考えられた意図は何でしょうか？言葉がとげとげしい気がしますが、そういう意図はありません。単純に知りたいんです。よろしくお願いいたします。

キャンセル
2018/09/07 17:23

基本的に同じ結果になる処理ですが、相違箇所はそこかなぁ、と。

基本的に同じ結果になる処理ですが、相違箇所はそこかなぁ、と。

キャンセル
2018/09/07 17:24 編集

いま思い付いたのですが
good_list = [row for row in f if 'good' in path]
を
good_list.extend([row for row in f if 'good' in path])
みたいにするとどうです？(badのほうも)

いま思い付いたのですが good_list = [row for row in f if 'good' in path] を good_list.extend([row for row in f if 'good' in path]) みたいにするとどうです？(badのほうも)

キャンセル
2018/09/07 17:59 編集

返信ありがとうございます。
教えていただいたことをもとに、下記のようにしたところエラーが出なくなりました。
なぜうまくいくのでしょうか？もしよろしければ教えていただければと思います。
fortune_list = list(f)
good_list.extend([row for row in fortune_list if 'good' in path])
bad_list.extend([row for row in fortune_list if 'bad' in path])

返信ありがとうございます。教えていただいたことをもとに、下記のようにしたところエラーが出なくなりました。なぜうまくいくのでしょうか？もしよろしければ教えていただければと思います。 fortune_list = list(f) good_list.extend([row for row in fortune_list if 'good' in path]) bad_list.extend([row for row in fortune_list if 'bad' in path])

キャンセル
2018/09/07 22:08

good_list.extend([row for row in fortune_list if 'good' in path])
を思いついたのは、オブジェクトの同一性絡みかな？　と考えたからです。
「# 元の処理」のコードではいずこかで宣言されたgood_list, bad_listに要素が追加されますが、内包表記、filterはともに新しいリストオブジェクトを作ります。
PyQのシステムを知らないのですが、動かし方によってはオブジェクトの同一性が絡んでいるのかなぁ、と。
ただ、fuzzballさんの回答へのコメントで条件式を常にTrueになるようにしたらエラーが出くなったというのはこれでは説明できないので、違うのかも。

good_list.extend([row for row in fortune_list if 'good' in path]) を思いついたのは、オブジェクトの同一性絡みかな？　と考えたからです。「# 元の処理」のコードではいずこかで宣言されたgood_list, bad_listに要素が追加されますが、内包表記、filterはともに新しいリストオブジェクトを作ります。 PyQのシステムを知らないのですが、動かし方によってはオブジェクトの同一性が絡んでいるのかなぁ、と。ただ、fuzzballさんの回答へのコメントで条件式を常にTrueになるようにしたらエラーが出くなったというのはこれでは説明できないので、違うのかも。

キャンセル
2018/09/07 23:14 編集

返信ありがとうございます。何度も質問に答えていただきとてもありがたいです。
オブジェクトの同一性とはid()が同じかどうかだと、なんとなくですがそう理解しています。
試しにgood_list = [],bad_list = []をコメントアウトさせてみましたが同じエラーが出ました。

[row for row in fortune_list if 'bad' in path]は結果を確認するとうまく動作しているのですが
[row for row in fortune_list if 'good' in path]だけがなぜかうまく動いてくれません。
なので、もう本当に謎です。

返信ありがとうございます。何度も質問に答えていただきとてもありがたいです。オブジェクトの同一性とはid()が同じかどうかだと、なんとなくですがそう理解しています。試しにgood_list = [],bad_list = []をコメントアウトさせてみましたが同じエラーが出ました。 [row for row in fortune_list if 'bad' in path]は結果を確認するとうまく動作しているのですが [row for row in fortune_list if 'good' in path]だけがなぜかうまく動いてくれません。なので、もう本当に謎です。

キャンセル
2018/09/09 20:39

ふと思ったのですが、'good' in rowじゃなくて'good' in pathなんですね
コードや問題の全貌がわかりませんが、'good'ないし'bad'のどちらかだけpathに含まれているなら、片方のリストにすべて入る動作になるのでは
元のコードでもそれは同じことですが・・・

ふと思ったのですが、'good' in rowじゃなくて'good' in pathなんですねコードや問題の全貌がわかりませんが、'good'ないし'bad'のどちらかだけpathに含まれているなら、片方のリストにすべて入る動作になるのでは元のコードでもそれは同じことですが・・・

キャンセル
2018/09/10 18:36

返信ありがとうございます。good,badそれぞれが含まれたデータが必ず存在します。
何か根本的に勘違いしているのかもしれません。考えていただきありがとうございます。

返信ありがとうございます。good,badそれぞれが含まれたデータが必ず存在します。何か根本的に勘違いしているのかもしれません。考えていただきありがとうございます。

キャンセル

score 1 · Answer 2 · 2018-09-07T17:45

+1

リスト内表記のコードは、こちらでは正常に動作しなかったので、とりあえずそれを修正しておきます。

# リスト内法表記
with open(path, encoding='utf-8') as f:
    good_list = [row for row in f if 'good' in path]
    f.seek(0) #追加
    bad_list = [row for row in f if 'bad' in path]

【追記】

処理時間とかは分かりませんが、こっちの方が気持ち悪くないかも。

# リスト内法表記
with open(path, encoding='utf-8') as f:
    rows = list(f) #リスト化
    good_list = [row for row in rows if 'good' in path]
    bad_list = [row for row in rows if 'bad' in path]

投稿 2018/09/07 17:45

編集 2018/09/07 18:11

回答の評価を上げる

以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
回答の評価を下げる

下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

2018/09/07 18:07

回答ありがとうございます。補足情報からの続きです。まったく合格判定が出なかったものがたまに合格判定が出るようになりました。
インタープリンタでは部分的にしか試していなかったので気づかなかったようです。
good_listでファイルのポインタの位置が最後尾になるのでポインタを0に戻す必要があると理解しました。

回答ありがとうございます。補足情報からの続きです。まったく合格判定が出なかったものがたまに合格判定が出るようになりました。インタープリンタでは部分的にしか試していなかったので気づかなかったようです。 good_listでファイルのポインタの位置が最後尾になるのでポインタを0に戻す必要があると理解しました。

キャンセル
2018/09/07 18:15

>>ポインタを0に戻す必要がある
はい、その通りです。

で、エラーは出なくなったのでしょうか？
質問は「なぜエラーが出るのか？」であって、「なぜ不合格になるのか？」ではないと思いますが。

>>ポインタを0に戻す必要があるはい、その通りです。で、エラーは出なくなったのでしょうか？質問は「なぜエラーが出るのか？」であって、「なぜ不合格になるのか？」ではないと思いますが。

キャンセル
2018/09/07 18:26

返信ありがとうございます。追記の場合の方がすっきりするように感じます。
合格以外の場合は同じくIndexError: Cannot choose from an empty sequenceとエラーが出ます。

返信ありがとうございます。追記の場合の方がすっきりするように感じます。合格以外の場合は同じくIndexError: Cannot choose from an empty sequenceとエラーが出ます。

キャンセル
2018/09/07 18:38

仕様の漏れとかはないでしょうか？例えば「ファイルが存在しないとき」とか。
エラーの頻度など分からないので何とも言えませんが、特殊な条件のときにエラーが出ているような気がするようなしないような。
特殊な条件ではなく、単純な仕様漏れとか、仕様の誤認とか。

仕様の漏れとかはないでしょうか？例えば「ファイルが存在しないとき」とか。エラーの頻度など分からないので何とも言えませんが、特殊な条件のときにエラーが出ているような気がするようなしないような。特殊な条件ではなく、単純な仕様漏れとか、仕様の誤認とか。

キャンセル
2018/09/07 19:42

返信ありがとうございます。ちょっと頭の中がごちゃごちゃしているので時間をかけて考えてみようと思います。
必要ないかもしれませんが何かしら進展があった報告させていただきます。
お付き合いいただきありがとうございました。とてもありがたかったです。

返信ありがとうございます。ちょっと頭の中がごちゃごちゃしているので時間をかけて考えてみようと思います。必要ないかもしれませんが何かしら進展があった報告させていただきます。お付き合いいただきありがとうございました。とてもありがたかったです。

キャンセル
2018/09/07 21:54 編集

教えていただいたことをもとに、下記のようにしたところリスト内法表記ではエラーが出なくなりました。
fortune_list = list(f)
good_list.extend([row for row in fortune_list if 'good' in path])
bad_list.extend([row for row in fortune_list if 'bad' in path])

さらに、extendを使わない内法表記・高階関数ともに good_list の if 'good' in pathの部分をTrueにしてみたところ、
欲しい結果は得られませんがエラーが全くでなくなりました。
なので if 'good' in path の部分が原因なのではないかと考えていますが、なぜなのかは全く分かりません。

教えていただいたことをもとに、下記のようにしたところリスト内法表記ではエラーが出なくなりました。 fortune_list = list(f) good_list.extend([row for row in fortune_list if 'good' in path]) bad_list.extend([row for row in fortune_list if 'bad' in path]) さらに、extendを使わない内法表記・高階関数ともに good_list の if 'good' in pathの部分をTrueにしてみたところ、欲しい結果は得られませんがエラーが全くでなくなりました。なので if 'good' in path の部分が原因なのではないかと考えていますが、なぜなのかは全く分かりません。

キャンセル

score 1 · Answer 3 · 2018-09-08T10:22

+1

fortune['message'] = random.choice(good_list)

というコードを書いていてそれがエラーになっているのに、なぜそれを質問に含めなかったんでしょうか。
最初から原因はこれで、潜在的なバグがここにあります。

https://docs.python.jp/3.6/library/random.html#random.choice

seq が空のときは、 IndexError が送出されます。

と明確に書いてあります。

good_listが空な状態でここに到達するとエラーになるのです。推測ですが、bad_listも同様のコードを書いているのではありませんか?

質問のソース、リスト内法表記とコメントされている部分と、高階関数ととコメントされている部分には明確にバグがあってbad_listが空になります。
そのせいでエラーが発生しました。
（で、これは修正されたと思います）

もう一つ、PyQが準備しているテストデータ群の中か、あるいはランダム生成されるテストデータか、good_listやbad_listが空になるようなエッジケースでエラーになっていたのではないでしょうか?

そのようなテストデータをたまたま踏むとエラーになるわけです。
現実にもよくあるエッジケースでだけ失敗するパターンですね。

投稿 2018/09/08 10:22

回答の評価を上げる

以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
回答の評価を下げる

下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

2018/09/08 20:32

返信ありがとうございます。お付き合いいただきとてもありがたいです。
質問するにあたって何を書けば適切なのかが分かっていないようです。
自分で判断せずにわかっていることをすべて書くようにしようと思います。

bad_listも同様なコードを書いています。ランダムな数字を基にして、
それを判定してfortune['message'] にgood_list,bad_listどちらかが入るようになっているのですが、
good_listの場合のみエラーが出ます。bad_listが選択された場合は予定通りの動きをしています。

# 元の処理の場合は全くエラーが出ないので別の原因なのではないかと考えます。

まだ何がどうなっているか理解しきれていないので、いろいろ考えて試して理解を深めようと思います。

返信ありがとうございます。お付き合いいただきとてもありがたいです。質問するにあたって何を書けば適切なのかが分かっていないようです。自分で判断せずにわかっていることをすべて書くようにしようと思います。 bad_listも同様なコードを書いています。ランダムな数字を基にして、それを判定してfortune['message'] にgood_list,bad_listどちらかが入るようになっているのですが、 good_listの場合のみエラーが出ます。bad_listが選択された場合は予定通りの動きをしています。 # 元の処理の場合は全くエラーが出ないので別の原因なのではないかと考えます。まだ何がどうなっているか理解しきれていないので、いろいろ考えて試して理解を深めようと思います。

キャンセル
2018/09/08 21:09

> good_listの場合のみエラーが出ます。
bad_listが空になるようなデータをテストしてみてくださいよ。

> good_listの場合のみエラーが出ます。 bad_listが空になるようなデータをテストしてみてくださいよ。

キャンセル
2018/09/09 18:40 編集

返信ありがとうございます。
元の処理のif 'bad' in path: bad_list.append(row)をコメントアウトさせたところ、
予想通りエラーメッセージにfortune['message'] = random.choice(bad_list)が出ました。

自分が思っていることとしては以下が発生しているので、
どういうことだろうか？となっています。

# エラー発生
good_list.extend([row for row in fortune_list if 'good' in path])
bad_list = ['a', 'b']

# エラー出ない
good_list = ['a', 'b']
bad_list.extend([row for row in fortune_list if 'bad' in path])

返信ありがとうございます。元の処理のif 'bad' in path: bad_list.append(row)をコメントアウトさせたところ、予想通りエラーメッセージにfortune['message'] = random.choice(bad_list)が出ました。自分が思っていることとしては以下が発生しているので、どういうことだろうか？となっています。 # エラー発生 good_list.extend([row for row in fortune_list if 'good' in path]) bad_list = ['a', 'b'] # エラー出ない good_list = ['a', 'b'] bad_list.extend([row for row in fortune_list if 'bad' in path])

キャンセル
2018/09/09 18:43

random.choice のドキュメントのリンクを貼ったはずですが

random.choice のドキュメントのリンクを貼ったはずですが

キャンセル
2018/09/09 19:00

返信ありがとうございます。
random.choiceに空のリストを入れればエラーが出るのは理解しています。
なぜgood_listが空になるのかわからない。ということです。

返信ありがとうございます。 random.choiceに空のリストを入れればエラーが出るのは理解しています。なぜgood_listが空になるのかわからない。ということです。

キャンセル
2018/09/09 20:00

goodと書かれていないデータを与えたら空になりますよね? 最低1件は入っている保証はありますか? 極論、空ファイルを与えるだけでエラーです。

goodと書かれていないデータを与えたら空になりますよね? 最低1件は入っている保証はありますか? 極論、空ファイルを与えるだけでエラーです。

キャンセル
2018/09/09 20:29

質問を引用するのは非常識な話なのでしょうがないですが、こちらでできるのは、空になって当然のコードでかつ空になったらエラーになるコードなので空になってエラーになったのでしょう、という指摘までですよ。

質問を引用するのは非常識な話なのでしょうがないですが、こちらでできるのは、空になって当然のコードでかつ空になったらエラーになるコードなので空になってエラーになったのでしょう、という指摘までですよ。

キャンセル
2018/09/10 18:27

返信ありがとうございます。good,badそれぞれが含まれたデータが必ず存在します。

返信ありがとうございます。good,badそれぞれが含まれたデータが必ず存在します。

キャンセル
2018/09/10 18:50 編集

goodが入ったファイル名から読んだデータはgood_listに、badが入ったファイル名から読んだデータはbad_listに格納しているのでしょうけど、ファイルの存在と、ファイルが空でないことを確認しているということですか?
合格判定という言葉が出ていることから、プログラムコンテストの要領になっていると想像しました。
PyQ側にある種のテストデータが複数パターン用意されていて、それらを全部、あるいは必須セット+ランダムにいくつかとかテストされていると思ってました。
そうではないのですか?
もしくは
用意されているテストデータは回答者側は確認できるのでしょうか? 「good,badそれぞれが含まれたデータが必ず存在します」と断言できるということは、エラーが起きた時にどういうデータで発生するか把握できているということですか?

goodが入ったファイル名から読んだデータはgood_listに、badが入ったファイル名から読んだデータはbad_listに格納しているのでしょうけど、ファイルの存在と、ファイルが空でないことを確認しているということですか? 合格判定という言葉が出ていることから、プログラムコンテストの要領になっていると想像しました。 PyQ側にある種のテストデータが複数パターン用意されていて、それらを全部、あるいは必須セット+ランダムにいくつかとかテストされていると思ってました。そうではないのですか? もしくは用意されているテストデータは回答者側は確認できるのでしょうか? 「good,badそれぞれが含まれたデータが必ず存在します」と断言できるということは、エラーが起きた時にどういうデータで発生するか把握できているということですか?

キャンセル
2018/09/10 18:52

どういうデータで発生するか把握できているなら、「なぜgood_listが空になるのかわからない」なんてことがありえるんでしょうか?
「なぜgood_listが空になるのかわからない」と言っていることから、どういうデータで発生するか把握できていない(把握できないシステムを用いている)という仮定で回答しているのですが。

どういうデータで発生するか把握できているなら、「なぜgood_listが空になるのかわからない」なんてことがありえるんでしょうか? 「なぜgood_listが空になるのかわからない」と言っていることから、どういうデータで発生するか把握できていない(把握できないシステムを用いている)という仮定で回答しているのですが。

キャンセル
2018/09/11 06:26

返信ありがとうございます。

>どういうデータで発生するか把握できているなら、「なぜgood_listが空になるのかわからない」なんてことがありえるんでしょうか?

自分の認識ではそれが起きています。

初めからすべて書くのが適切だったということを学ばせていただきましたが、
おみくじを表示するWebアプリケーションをつくろう。というものです。
該当のソースコードを変更したので、もしよろしければ見ていただければと思います。

返信ありがとうございます。 >どういうデータで発生するか把握できているなら、「なぜgood_listが空になるのかわからない」なんてことがありえるんでしょうか? 自分の認識ではそれが起きています。初めからすべて書くのが適切だったということを学ばせていただきましたが、おみくじを表示するWebアプリケーションをつくろう。というものです。該当のソースコードを変更したので、もしよろしければ見ていただければと思います。

キャンセル
2018/09/11 08:10 編集

「うまく動く」方は**必ず**うまく動くし、「エラー発生」は**必ず**エラーが発生するのが現状で、"たまに合格判定が出るようになりました"は古い情報or間違いということでいいですか?
ソースからは当然そうなると思いますが念のため。

「うまく動く」方は**必ず**うまく動くし、「エラー発生」は**必ず**エラーが発生するのが現状で、"たまに合格判定が出るようになりました"は古い情報or間違いということでいいですか? ソースからは当然そうなると思いますが念のため。

キャンセル
2018/09/11 22:36

返信ありがとうございます。
「うまく動く方」は必ずうまく動きます。bad_list,good_lisetどちらが選択された場合にも狙い通りの表示がされます。
「エラー発生」の方はgood_listが選択された場合のみエラーが発生します。
実際に動かして確認できるのですがbad_listが選択された場合のみ狙った結果が表示されています。

返信ありがとうございます。「うまく動く方」は必ずうまく動きます。bad_list,good_lisetどちらが選択された場合にも狙い通りの表示がされます。「エラー発生」の方はgood_listが選択された場合のみエラーが発生します。実際に動かして確認できるのですがbad_listが選択された場合のみ狙った結果が表示されています。

キャンセル

score 0 · Answer 4 · 2018-09-07T16:15

0

インタープリンタで試してみたところ書き換えたコードは問題なく動きました。なので、PyQ固有の問題という結論になりました。
もし何かこういう事じゃないか？というのがあれば教えていただければと思います。ありがとうございました。

投稿 2018/09/07 16:15

回答の評価を上げる

以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
回答の評価を下げる

下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

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

ある処理をリスト内法表記や高階関数へ書き換えたいがエラーが出る

前提・実現したいこと

発生している問題・エラーメッセージ

該当のソースコード

試したこと

補足情報（FW/ツールのバージョンなど）

関連した質問

同じタグがついた質問を見る

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

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

ある処理をリスト内法表記や高階関数へ書き換えたいがエラーが出る

前提・実現したいこと

発生している問題・エラーメッセージ

該当のソースコード

試したこと

補足情報（FW/ツールのバージョンなど）

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

関連した質問

同じタグがついた質問を見る

SNSアカウントでログイン

teratailアカウントでログイン