二重ループの使い方、ブロック配置

Question

二重ループを用いて8×4のブロック配置をしたいのですがうまくできないです。

コード

import pygame
from pygame.locals import QUIT
from pygame.locals import Rect
pygame.init()
SURFACE = pygame.display.set_mode((400,300))
pygame.display.set_caption("Game Window")
class Block:
    def __init__(self, color, rect):
        self.color = color
        self.rect = rect
    def draw(self):
        pygame.draw.rect(SURFACE, self.color, self.rect)
left=6
top=30
width=45
height=20
color=(200, 50, 200)
blocks = []
for j in range(2):
    for i in range(8):
        rect = Rect(left, top, width, height)
        blocks.append(Block(color, rect))
        left += 49
while True:
    SURFACE.fill((0,0,0))
    for event in pygame.event.get():
        if event.type == QUIT:
            pygame.quit()
            sys.exit()
    for block in blocks:
        block.draw()
    pygame.display.update()

エラーメッセージ

### 該当のソースコード

```ここに言語名を入力
ソースコード

試したこと

ここに問題に対して試したことを記載してください。

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

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

Comments

[1T2R3M4]

https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q10253060555

以下ご対応ください。
https://teratail.com/help#posted-otherservice

Answer 1

どのようにブロックを配置したいのかわかりませんが、すべてのブロックに同じleft と top を設定しているのが原因でしょう。

left=6
top=30
width=45
height=20
color=(200, 50, 200)
blocks = []
for j in range(2):
    for i in range(8):
        rect = Rect(left, top, width, height)
        blocks.append(Block(color, rect))
        left += 49

left と top の値は、 i と j の値によって変更される必要があるんじゃないかと思います。
たとえばこんな感じ。、

for j in range(2):
    button_top = top + (height + 2) * j 
    for i in range(8):
        button_left = left + (width + 2) * i
        rect = Rect(button_left, button_top, width, height)
        blocks.append(Block(color, rect))
        left += 49

Comments

[teslabee]

Traceback (most recent call last):
File "j11_kadai2.py", line 21, in <module>
button_top = top + (height + 2) * i
NameError: name 'i' is not defined
こういうエラーが出てきました。

[TakaiY]

i と j が逆でしたね。

直しました。

[teslabee]

ありがとうございます。