回答編集履歴

リストをタプルに変更

2022/03/04 16:21

投稿

スコア5406

answer CHANGED Viewed

@@ -19,10 +19,10 @@
             names.append(name)
             counts += count
             if counts >= limit:
-                yield [store, " ".join(names)]
+                yield store, " ".join(names)
                 names, counts = [], 0
         if names:
-            yield [store, " ".join(names)]
+            yield store, " ".join(names)
 for item in convert():
     print(item)

処理簡略化

2022/03/04 16:08

投稿

スコア5406

answer CHANGED Viewed

@@ -14,15 +14,13 @@
 def convert(limit=5):
     for store, products in stores.items():
-        names = []
+        names, counts = [], 0
-        counts = 0
         for name, count in products.items():
             names.append(name)
             counts += count
             if counts >= limit:
                 yield [store, " ".join(names)]
-                counts = 0
+                names, counts = [], 0
-                names = []
         if names:
             yield [store, " ".join(names)]

処理簡略化

2022/03/04 16:06

投稿

スコア5406

answer CHANGED Viewed

@@ -14,22 +14,17 @@
 def convert(limit=5):
     for store, products in stores.items():
-        items = iter(products.items())
+        names = []
+        counts = 0
-        for name, counts in items:
+        for name, count in products.items():
+            names.append(name)
+            counts += count
             if counts >= limit:
-                yield [store, name]
-                continue
-            names = [name]
-            for name, count in items:
-                names.append(name)
-                counts += count
-                if counts >= limit:
-                    yield [store, " ".join(names)]
-                    counts = 0
-                    names = []
-            if names:
                 yield [store, " ".join(names)]
+                counts = 0
+                names = []
-            break
+        if names:
+            yield [store, " ".join(names)]
 for item in convert():
     print(item)