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

Question

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

import sys
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("pygame window ")
class Block:
     def __init__(self, color, rect):
          self.color = color #ボールの色(R,G,B)
          self.rect = rect #初期位置と大きさ(Rect を利用)
   
     def draw(self):
          pygame.draw.rect(SURFACE, self.color, self.rect)

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

    for event in pygame.event.get():
         if event.type == QUIT:
              pygame.quit()
              sys.exit()

    SURFACE.fill((0,0,0))

    for block in blocks:
         block.draw()
    pygame.display.update()

Comments

[y_waiwai]

うまくできないとはどういうふうになるんでしょう

[8524ba23]

「ブロック配置」というのがよく分かりません。
具体的に得たいblocks変数（リスト）の結果を
リストの各要素が持つleft,top値が分かるようにして示してください。

Answer 1

blocks=[]
for y in range(4):
    for x in range(6):
        color = ( x*30+50, y*20+50, 100)
        rect = Rect(left+50*x, top+50*y, width, height)
        blocks.append(Block(color, rect))

Comments

[kkashi89sk]

ありがとうございます！参考になります。

Answer 2

単純に、Blockをforごとに2回定義しリストに入れているのが誤りじゃないでしょうか。

なので20行目～27行目を下記のように変えればいいと思います。

for i in range(4):
     for j in range(6):
          rect = Rect((j + 1) * 50, (i + 1) * 50, width, height)
          blocks.append(Block(color, rect))

Comments

[kkashi89sk]

ありがとうございます！参考にします。