Q&A
いくつか質問させてください。「連携していない売上情報」の「連携していない」というのはどういう意味でしょうか？コードからは Django の `ForeignKey` を使っていないという意味なのかなと推測できるのですが、この理解で正しいですか？
POSと連携したシステムを独学で作っているのですが、うまく情報を引き出せずにいます。
システムは、python3.6 django2.1を使用しています。
目標：
売上データを表示するリストに売れた店名を表示させたい
現状：
リレーションで関連づけていないため、数字のコードの表記のみ
外部システムからAPIで売上データが飛んできます。
売上情報TransactionHeadとTransactionDetailの二つがあり、この二つはリレーションしていません。
この二つは、データの構造を変えずにデータベースへ保存しています。
保存した売上データをDjango内でリスト表示するために、下記のviewで一覧表示を出しています。
django
1class TransactionListView(ListView): 2 def get_context_data(self, **kwargs): 3 user_id = self.request.user.id 4 context = super(TransactionListView, self).get_context_data(**kwargs) 5 context["object_list"] = [] 6 for detail in TransactionDetail.objects.filter(user_id=user_id).order_by('-dateTime'): 7 head = TransactionHead.objects.filter( 8 transactionHeadId=detail.transactionHeadId).first() 9 10 tmp = {} 11 for d in head.__dict__: 12 tmp[d] = head.__dict__[d] 13 for d in detail.__dict__: 14 tmp[d] = detail.__dict__[d] 15 16 context["object_list"].append(tmp) 17 return context
TransactionHeadの中にstoreCodeという情報を持っていて４桁の数字が入っています。
storeというモデルがあり、id,store_code,store_nameのカラムを持っています。
model
1class TransactionHead(models.Model): 2 transactionHeadId = models.BigIntegerField(verbose_name="取引ID", null=True) 3 transactionDateTime = models.DateTimeField(verbose_name="取引日時", null=True) 4 subtotal = models.DecimalField(max_digits=9, decimal_places=0, verbose_name="小計", null=True) 5 total = models.DecimalField(max_digits=9, decimal_places=0, verbose_name="合計", null=True) 6 storeCode = models.CharField(max_length=20, verbose_name="店舗コード", null=True) 7 8class TransactionDetail(models.Model): 9 transactionHeadId = models.BigIntegerField(verbose_name="取引ID", null=True) 10 productId = models.BigIntegerField(verbose_name="商品ID", null=True) 11 productCode = models.CharField(max_length=20, verbose_name="商品コード", null=True) 12 13class Store(models.Model): 14 id = models.BigIntegerField(verbose_name="ID") 15 store_name = models.CharField(max_length=255, verbose_name="店名") 16 store_code = models.CharField(max_length=255, verbose_name="ストアコード", blank=True) 17
storeCodeに対応するstore_nameを紐ずけてテンプレートのリストに表示したいのですが、どの様に記述すれば解決出来るかお知恵をいただきたいです。
テンプレート上で{% for object in object_list %}の{{object.store_name}}で表示出来るのを目指しています。
（尚、運用は開始しておりデータがどんどん溜まってきます。よってリレーションで繋げられていません。）
何卒よろしくお願いします。
「この中にstoreCodeというのが情報として持っていて」の「この」とは何を指すのでしょう？また「 storeCode 」というのがご提示されたコードの中に無いように思います。これが何を意味するのかを説明されると議論がしやすくなると思います。
「別のもmodelでstoreがあり、id/store_code/store_nameのカラムを持っています。」の「別のも」とは何を指すのでしょう？（「 id/store_code/store_nameのカラムを持っています 」の主語は何でしょうか？）
未定義の変数 `group_code` がありますが、これはどこから来たものでしょう？ おそらく、「 storeCodeに合ったstore_nameを紐ずけてテンプレートに表示したい 」を実現する前に、現状ご提示されているコードだと動かないのでは・・・という気がしますが現状はどのような状態でしょう。というのと、 `TransactionListView` の `get_context_data` メソッド以外に関連するコード（ `TransactionListView` のアトリビュート・その他のメソッド、 urlconf 、 model 、テンプレート）をもう少し広い範囲でご提示された方が具体的なポイントを付いて回答をしてもらえる可能性が高まるものと思います。
ご指摘ありがとうございます。
＞＞「連携していない売上情報」の「連携していない」というのはどういう意味でしょうか？コードからは Django の `ForeignKey` を使っていないという意味なのかなと推測できるのですが、この理解で正しいですか？
はい、リレーションがかかっていないと言う意味なのでご理解通りです。
＞＞「この中にstoreCodeというのが情報として持っていて」の「この」とは何を指すのでしょう？
売上情報(TransactionHead,TransactionDetail)のTransactionHeadを「この」と指しています。
＞＞「別のも」と言うのはタイプミスで、別で「store」と言うモデルがあり、データベースにid,store_code,store_nameの三つの情報を持っています。
＞＞こちらへの記述が間違っていました。修正致します。
ご指摘いただき、どれだけ情報が少ないか理解致しました。
表記を修正して、ご指摘いただけたら幸いです。
回答1件
0
ベストアンサー
ご回答いただきありがとうございます。更新された内容でとてもわかりやすくなりました。
現状「店名」の表示以外の部分には問題が無いという前提で、課題は
「店名」を
{{ object.store_name }}で表示できるようにしたい
との想定で、以下回答させていただきます。
早速結論ですが、次のとおりに変更すればよいのではないかと思います。
patch
1class TransactionListView(ListView): 2 def get_context_data(self, **kwargs): 3 user_id = self.request.user.id 4 context = super(TransactionListView, self).get_context_data(**kwargs) 5 context["object_list"] = [] 6 for detail in TransactionDetail.objects.filter(user_id=user_id).order_by('-dateTime'): 7 head = TransactionHead.objects.filter( 8 transactionHeadId=detail.transactionHeadId).first() 9+ store = Store.objects.get(store_code=head.storeCode) 10 11 tmp = {} 12 for d in head.__dict__: 13 tmp[d] = head.__dict__[d] 14 for d in detail.__dict__: 15 tmp[d] = detail.__dict__[d] 16+ tmp['store_name'] = store.store_name 17 18 context["object_list"].append(tmp) 19 return context
（私の手元で動作確認していないので、アイデアだけ参考にしてください）
すでにお書きになっているコードを読ませていただくかぎりでは、この 2 行を追加することは takaomi_okuda さんにとって特に難しいことではないのではないか・・・と思いますが、このような回答でお求めの答えになっていますか？もう少し別のことをお尋ねになりたいのではないかなと思っています。
...
以下余談です。
余談 1: detail に対応する head が存在することが保証されているなら、次のようにしてもよいと思います。
patch
1- head = TransactionHead.objects.filter( 2- transactionHeadId=detail.transactionHeadId).first() 3+ head = TransactionHead.objects.get( 4+ transactionHeadId=detail.transactionHeadId)
余談 2: Python 3 をお使いなら次のように短く書いてもよいかと思います。
patch
1- context = super(TransactionListView, self).get_context_data(**kwargs) 2+ context = super().get_context_data(**kwargs)
余談 3: 例えば TransactionHead と Store は Django の ForeignKey でひもづいていなくても、それらが論理的に関係しているのであれば model の方でその関連を辿れるメソッドを定義した方がスッキリするのではないかと思います。つまり、例えばモデルに次のようなメソッドを追加します:
python
1class TransactionHead(models.Model): 2 # （省略） 3 4 @property 5 def store(self): 6 return Store.objects.get(store_code=self.storeCode)
すると、 view で head から store_name を辿るのは、次のようにかんたんに書けるようになります:
python
1store_name = head.store.store_name
ご参考になれば幸いです :)
投稿2018/12/19 09:18
編集2018/12/19 09:29総合スコア1407
ご丁寧にありがとうございました。
ご指摘いただいた事で、質問する際にしっかり考えをまとめるように心がけます。
また、質問の先のご提案までくださりとても勉強になりました。感謝です。
重ねての質問で恐縮なんですが、こちらでページネーションを実装しようとpaginate = 10 を追記すると、上手くページネーションができなかったんですが、原因はどこにあるんでしょうか？
（ちなみに同じ書き方をして別のリスト表示では動作していたので、書き方が間違っていることはないと思うんですが。。。
端的にだけ回答させていただきますね。
まず、 `paginate = 10` は正しくは `paginate_by = 10` なのかなと思いますが、いかがでしょうか。というのと、 `ListView` を継承して利用する場合は、 `ListView` が何をやっているのかを把握した上で利用されるのがよいかと思います。一度 `ListView` のコードをご覧になってみてください:
https://github.com/django/django/blob/2.1.4/django/views/generic/list.py
コードをご覧になるとおそらく解決するのではないかと思いますが、厳しい場合は新たに質問を立ててみてください :D
あなたの回答
tips
プレビュー
