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

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

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

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

Q&A

解決済

4回答

1026閲覧

Python 辞書が壊される

iota_yok

総合スコア16

Python

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

0グッド

0クリップ

投稿2018/09/21 01:59

前提・実現したいこと

 Python で辞書を作成するプログラムを作成しています。key は整数、value はリストのリストです。
実行中は、なんのエラーメッセージも出されず、表面的には正常に実行されています。
無関係でしょうが、ちなみに環境(IDE)は PyScripter です。

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

エラーメッセージ

該当のソースコード

Python 3

ソースコード(抜粋)

(global)
memos = defaultdict(list)

(while ループの中)
list = []
while ..
...
print('list:', list)
key = 0
list_res = list.copy()
list_res_bak = [] ##
for val in list:
print('val =', val)
key += val
if not (persons * 1//4 <= key <= persons * 3//4): break
list_res.pop(0)
# list_res_bak = list_res.copy() ##
print('key =', key, ', list_res:', list_res)
print('before append - memos:', memos)
memos[key].append(list_res)
# memos[key].append(list_res_bak) ##
print('after append - memos:', memos)
...

以下2とおりの実行結果(抜粋)を掲げます。

...
OK(6) table: [10, 5, 5, ] ※この出力に対するコードは上のソースにはない
list: [10, 5, 5]
val = 10
key = 10 , list_res: [5, 5]
before append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2]]})
after append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2], [5, 5]]})
val = 5
key = 15 , list_res: [5]
before append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2], [5]]})
after append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2], [5]], 15: [[5]]})
...

 問題は辞書 memos において、最初のほうの append 後の {.. ,[5, 5]]} は正常ですが、次の for の list における append 前では、何故か {.. ,[5]]} と変化しています。
この原因がわからず、苦し紛れに上のソースでリスト list_res_bak を設け、pop() しているリストとmemosにappendしているリストを別物(コピー)にしてみました。(すなわち、上のソースの ## の行を活かしたもの。)結果は、上の部分が次のように正常になりました。
ずいぶん悩みましたが、pop() が絡むバグではないか、と考えますが如何でしょうか。

...
OK(6) table: [10, 5, 5, ]
list: [10, 5, 5]
val = 10
key = 10 , list_res: [5, 5]
before append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2]]})
after append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2], [5, 5]]})
val = 5
key = 15 , list_res: [5]
before append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2], [5, 5]]})
after append - memos: defaultdict(<class 'list'>, {10: [[10], [8, 2], [7, 3], [6, 4], [6, 2, 2], [5, 5]], 15: [[5]]})
...

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

ここにより詳細な情報を記載してください。

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

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

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

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

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

guest

回答4

0

自己解決

これまでの回答内容から見て、これ以上的確な回答が期待できそうにないため、今後、自分で解決を図るべく心づもりしています。つまり、Pythonから別の言語、たとえば私の知っている言語であるGoコンパイラに変換して結果を比較してみようかと思っています。どういう結果になるかなるか予測できませんが、そのさいには、また相談(投稿)するかもしれません。

以上

投稿2018/09/24 03:31

iota_yok

総合スコア16

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

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

quickquip

2018/09/24 04:24 編集

質問のコードが読めないから具体的な=明確な話ができないだけなのに。何を言ってるんです? 言っていることと不完全なソースから多分こういうことなんだろうなあ、と想像するしかない状況だからコードを読めるようにしましょう、ってことですよ。その自覚はなかったということでしょうか。
iota_yok

2018/09/24 08:56

たいへん申し訳ありません。平にご容赦ください。またの機会があると思いますので、そのせつはよろしくお願いします。
guest

0

Python

1l = [0, 1, 2, 3] 2res = [] 3 4res.append(l) 5 6l.pop() 7 8print(res)

これの結果が想像できますか?

>>> l = [0, 1, 2, 3] >>> res = [] >>> res.append(l) >>> print(res) [[0, 1, 2, 3]] >>> l.pop() 3 >>> print(res) [[0, 1, 2]]

こうなりますけど、これが理解できていれば、何が起きているかわかるはずですよ。

これ以上の話は、質問中のソースは読めるように編集してからにしましょう。

投稿2018/09/21 13:35

quickquip

総合スコア11038

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

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

0

[5,5]がpop(0)で[5]になったというだけなのでは。
keyが10の場合も15の場合もlist_resという同じlistを参照しているせいなので,解決策として挙げられているようにcopyなどで新しいlistを作ればよいです。

投稿2018/09/21 02:40

Hirobou

総合スコア21

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

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

iota_yok

2018/09/21 03:04

回答ありがとうございます。私の理解しているところでは、pop(0)は[10, 5, 5]の10をremoveしたつもりです。現に、pop(0)の直後では、目にできるように、そうなっています。問題がやや込み入っているため、質問の意図がよく伝わらなかったかもしれません。残念ながら得心できません。
guest

0

まずpopの動作を理解していますか? 

s.pop([i]) s から i 番目の要素を取り出し、また取り除きます

https://docs.python.jp/3/library/stdtypes.html#typesseq-mutable

投稿2018/09/21 02:28

hayataka2049

総合スコア30933

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問