致命的Pythonエラー：スタックオーバーフローから回復できません。フラッドフィル中
私は行き止まりになりました、そして過度の（そして失敗した）グーグルの後、私は助けが必要です。
それぞれがNoneに初期化された60x80の正方形のグリッドを配置する単純なPyQt4ウィジェットを構築しています。ユーザーがそのボックスをクリックすると、このリストで定義されている左クリックの回数に基づいて色が変わります。
_self.COLORS=[
(0, 0, 255), #WATER
(255, 210, 128), #SAND
(0, 128, 0), #GREEN
(255, 255, 0), #YELLOW
(255, 165, 0), #ORANGE
(255, 0, 0) #RED
]
_ユーザーが右クリックすると、一般的な再帰的フラッドフィルアルゴリズムを使用して、エリアがフラッドフィルされます。これは小さなスペースでは完全に機能しますが、スペースが十分に大きい場合、プログラムはエラー_Fatal Python error: Cannot recover from stack overflow._で失敗します。これを修正する方法がわかりません。おそらく、再帰的ではないフラッドフィルですか？
すべての正方形とそれに続くカラーコードは_self.cells_に格納されるため、self.cells[(y,x)]=1を設定すると、セル_(y,x)_がSandカラーに設定されます。
これがプログラム全体です。
_import sys
from PyQt4 import QtGui, QtCore
class Example(QtGui.QWidget):
def __init__(self, cell_size=10, swidth=800, sheight=600):
QtGui.QWidget.__init__(self)
self.resize(swidth,sheight)
self.cell_size = cell_size
self.height = sheight
self.width = swidth
self.columns = self.width // self.cell_size
self.rows = self.height // self.cell_size
self.COLORS=[
(0, 0, 255), #WATER
(255, 210, 128), #SAND
(0, 128, 0), #GREEN
(255, 255, 0), #YELLOW
(255, 165, 0), #ORANGE
(255, 0, 0) #RED
]
self.cells = {(x,y):None for x in range(1,self.columns+1) for y in range(1,self.rows+1)}
def translate(self,pixel_x, pixel_y):
"Translate pixel coordinates (pixel_x,pixel_y), into grid coordinates"
x = pixel_x * self.columns // self.width + 1
y = pixel_y * self.rows // self.height + 1
return x,y
def check_cell(self,x,y):
if self.cells[(x,y)] <= 0:
self.cells[(x,y)]=0
Elif self.cells[(x,y)] >= len(self.COLORS)-1:
self.cells[(x,y)]=len(self.COLORS)-1
else:
pass
def draw_cell(self, qp, col, row):
x1,y1 = (col-1) * self.cell_size, (row-1) * self.cell_size
x2,y2 = (col-1) * self.cell_size + self.cell_size, (row-1) * self.cell_size + self.cell_size
qp.drawRect(x1, y1, x2-x1, y2-y1)
def color_cell(self, qp, col, row):
qp.setBrush(QtGui.QColor(*self.COLORS[self.cells[(col,row)]]))
self.draw_cell(qp, col, row)
def draw_grid(self, qp):
qp.setPen(QtGui.QColor(128,128,128)) # gray
# Horizontal lines
for i in range(self.rows):
qp.drawLine(0, i * self.cell_size, self.width, i * self.cell_size)
# Vertical lines
for j in range(self.columns):
qp.drawLine(j * self.cell_size, 0, j * self.cell_size, self.height)
def set_all(self, type):
self.cells = {(x,y):type for x in range(1,self.columns+1) for y in range(1,self.rows+1)}
self.repaint()
def fill(self, x, y, type):
print(x,y)
if x < 1 or x >= self.columns+1 or y < 1 or y >= self.rows+1:
return
if self.cells[(x,y)] != None:
return
self.cells[(x,y)] = type
self.repaint()
self.fill(x+1, y, type)
self.fill(x-1, y, type)
self.fill(x, y+1, type)
self.fill(x, y-1, type)
def paintEvent(self, e):
qp = QtGui.QPainter()
qp.begin(self)
self.draw_grid(qp)
for row in range(1, self.rows+1):
for col in range(1, self.columns+1):
if self.cells[(col,row)] != None:
self.color_cell(qp, col, row)
qp.end()
def drawPoints(self, qp):
size = self.size()
for i in range(1000):
x = random.randint(1, size.width()-1)
y = random.randint(1, size.height()-1)
qp.drawPoint(x, y)
def mousePressEvent(self, e):
x,y = self.translate(e.pos().x(),e.pos().y())
if e.button() == QtCore.Qt.LeftButton:
if self.cells[(x,y)] == None:
self.cells[(x,y)]=0
else:
self.cells[(x,y)]+=1
self.check_cell(x,y)
Elif e.button() == QtCore.Qt.RightButton:
self.fill(x,y,0)
'''
if self.cells[(x,y)] == None:
self.cells[(x,y)]=0
else:
self.cells[(x,y)]-=1
self.check_cell(x,y)
'''
else: pass
self.repaint()
def save(self):
return self.cells
def open(self, new_cells):
self.cells=new_cells
self.repaint()
def main():
app = QtGui.QApplication(sys.argv)
ex = Example()
ex.show()
sys.exit(app.exec_())
if __name__ == '__main__':
main()
_誰かが問題の診断を手伝ったり、おそらくそれを修正する方向を指し示したりできますか？
スタックを大量に消費することが知られているスタックベースの森林火災アルゴリズムを使用しているため、回避することをお勧めします。
再帰を回避するための私の提案： 代替森林火災アルゴリズム
クラスオブジェクトを使用して実装しました。いくつかのASCIIアートと実際のコードでテストしたところ、大きなゾーンでも問題なく動作しました。
def fill(self, x, y, t):
if self.cells[(x,y)] == None: # cannot use not: there are 0 values
to_fill = [(x,y)]
while to_fill:
# pick a point from the queue
x,y = to_fill.pop()
# change color if possible
self.cells[(x,y)] = t
# now the neighbours x,y +- 1
for delta_x in range(-1,2):
xdx = x+delta_x
if xdx > 0 and xdx < self.columns+1:
for delta_y in range(-1,2):
ydy = y+delta_y
# avoid diagonals
if (delta_x == 0) ^ (delta_y == 0):
if ydy > 0 and ydy < self.rows+1:
# valid x+delta_x,y+delta_y
# Push in queue if no color
if self.cells[(xdx,ydy)] == None:
to_fill.append((xdx,ydy))
self.repaint()
ポイントを渡すと、それは満たされなければならないかどうかをチェックします。埋める必要がある場合は、キューに挿入してループを実行します。
ループは、キューからアイテムをポップし、その色を変更し、隣接するものに対して同じことを試みます。対角線ではなく、画像内にまだあり（x、y境界チェック）、隣接する色が定義されていない場合は、座標をキューに挿入します。
すべてのアイテムが処理されると、ループが停止します。しばらくすると、エッジに到達するか、塗りつぶされたポイントのみが検出されるため、余分なポイントはキューに入れられません。
このアプローチは、スタックではなく、使用可能なメモリのみに依存します。
それが機能することの証明：スタックオーバーフローなしで巨大なブルーゾーンを正常に埋めました。
