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

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

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

Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python

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

Q&A

1回答

617閲覧

forの値でlistにappendすると、中身がすべて最後の値になる

hiragiuni

総合スコア0

Python 3.x

Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python

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

0グッド

0クリップ

投稿2022/11/23 06:44

問題

id、x、y 座標を持った辞書型を羅列した リストを作りたいと思っています。
以下のようなレイアウトの dict を羅列したリスト items を作成したいです。
そして、それをjsonで保存したいです。

x, y は連番、 id は x*10 + y とします。

python

1template = { 2 'id': id, # ID (x*10 + y) が入る 3 'position': { 4 'x': x, # x 座標が入る 5 'y': y # y 座標が入る 6 } 7}

実現したいこと

json

1[ 2 { 3 "id": 00, 4 "position": { 5 "x": 0, 6 "y": 0 7 } 8 }, 9 { 10 "id": 01, 11 "position": { 12 "x": 0, 13 "y": 1 14 } 15 }, 16...

発生している問題

x, y を 4までの連番とした場合、次のようになってしまいます。
ループ終了時の値がリストのすべての要素にappendされてしまっているように見えます。

json

1[ 2 { 3 "id": 44, 4 "position": { 5 "x": 4, 6 "y": 4 7 } 8 }, 9 { 10 "id": 44, 11 "position": { 12 "x": 4, 13 "y": 4 14 } 15 }, 16...

該当のソースコード

上の問題が発生するソースコードが次のとおりです。

python

1import json 2 3template = { 4 'id': '', 5 'position': {} 6} 7 8items = [] 9 10for i in range(4): 11 for j in range(4): 12 id = i*10 + j 13 x = i 14 y = j 15 elm = template 16 elm['id'] = id 17 elm['position'].update({'x': x}) 18 elm['position'].update({'y': y}) 19 items.append(elm) 20 21with open('export.json', mode='w', encoding='utf-8') as f: 22 json.dump(items, fp=f)

試したこと

変数 の id を意識して書き換えた
appendした後に中身が変わってしまったのかと思い、elm = dict(elm) 、ループの最後にdel elmとしたりしてみましたが変化がありませんでした。

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

Python 3.10.6

ご教示よろしくお願い致します。

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

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

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

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

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

melian

2022/11/23 06:53

elm = template が shallow copy になっていますので、 from copy import deepcopy としておいて、 elm = deepcopy(template) とするとよいかと。
hiragiuni

2022/11/23 10:44

ご回答ありがとうございます。 その指摘をBAとさせて頂きたいため、解答欄に再度記述していただけますでしょうか。 よろしくお願い致します。
melian

2022/11/23 11:29

まぁ、既に回答とその後のコメントでのやり取りがありますので、BA は Cocode さんで良いと思いますよ。
guest

回答1

0

不具合の原因

templateという変数を繰り返しappendしているだけなので、全て同一のものです。
forの中で、templateという変数を利用して 新しい辞書型オブジェクトを作成する 必要があります。
アンパック ** を使いましょう!

修正箇所

1ヶ所修正するだけです。アンパックで新しいオブジェクトを作成します。

diff

1- elm = template 2+ elm = {**template}

投稿2022/11/23 06:59

編集2022/11/23 07:00
Cocode

総合スコア2314

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

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

hiragiuni

2022/11/23 07:18

初めて教えて頂いた書き方で、非常にためになりました。 ありがとうございます。 Cocodeさんのお答えで id は変わりましたが、 x, y の値が3, 3のまま変わりませんでした。 その部分は、質問へのコメントで答えをくださった、melianさんのアドバイスにより対処できました。 お二人とも、迅速なご回答に感謝致します。
Cocode

2022/11/23 07:33

BA選んでいただいてありがとうございます!でも私の回答が間違っていて申し訳ないので、BA取り消していただいて… melianさんに解答欄に投稿していただくようにお願いするか、 ご自身で投稿されてそれをBAに選択し、「自己解決」にされたらいいと思います! 今後同じようなお悩みをもってこのページにこられた方の参考になると思います。
hiragiuni

2022/11/23 10:42

わかりました、ではそのようにさせて頂きます。 ご回答感謝いたします。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問