[已解决]Pygame

歌者文明清理员

（因为 GPT 脚本的问题，先不发求助区）

我的问题：



右上角和右下角车不显示。这很奇怪，代码也检查不出啥错的。

Debug 过程：在第 138 和 139 行之间添加了一行 print(1) 用于 Debug，发现输出了两次 1，应该是重复放置的问题

另外，第 224 行的输出：

1. pygame 2.5.0 (SDL 2.28.0, Python 3.11.4)
   
2. Hello from the pygame community. https://www.pygame.org/contribute.html
   
3. [ br ,  bn ,  bb ,  bq ,  bk ,  bb ,  bn , None, ]
   
4. [ bp ,  bp ,  bp ,  bp ,  bp ,  bp ,  bp ,  bp , ]
   
5. [None, None, None, None, None, None, None, None, ]
   
6. [None, None, None, None, None, None, None, None, ]
   
7. [None, None, None, None, None, None, None, None, ]
   
8. [None, None, None, None, None, None, None, None, ]
   
9. [ wp ,  wp ,  wp ,  wp ,  wp ,  wp ,  wp ,  wp , ]
   
10. [ wr ,  wn ,  wb ,  wq ,  wk ,  wb ,  wn , None, ]

复制代码

question.zip (21.26 KB, 下载次数: 10)

2023-8-11 00:15 上传

点击文件名下载附件

最佳答案

月排行榜 / 总排行榜

鱼cpython学习者

2023-8-12 08:51:31

歌者文明清理员 发表于 2023-8-11 22:41
抱歉来晚了

把col.setter里的判断语句改成

1. if self._row != -1 and old != -1:
   

复制代码

其实，我推荐你加个move函数来代替setter实现移动位置的功能

跳转到最佳答案楼层

cjjJasonchen

消失的它

歌者文明清理员

cjjJasonchen 发表于 2023-8-11 09:08
消失的它

“下崽次数: 4”

cjjJasonchen

呐，问题我好像差不多看出来了

1. print_chessboard()
   
2.         print("")

复制代码

你把这个加到place_chess的最后，每帧都打印一下，看看返回值

cjjJasonchen

你所有的棋子出现的时候，右上角和右下角的棋子都在变，导致了错误，

但是我还没有成功找到bug的具体位置，应该是生成棋子的算法问题，

代码是你写的，到这里说不定可以直接看出问题。。。所有我觉定先发出来

cjjJasonchen

解决方法：
        第114行改为：

1. for name in 'kqbbnnpppppppprr':

复制代码

        让车最后生成

bug原因：
        不知道为啥，棋子生成是，会顶掉右上角和右下角的格子


注意！这只是暂时修复了生成棋子时的bug，具体原因还没有找到

歌者文明清理员

cjjJasonchen 发表于 2023-8-11 09:38
解决方法：
        第114行改为：
       

稍等，我先试试id

歌者文明清理员

cjjJasonchen 发表于 2023-8-11 09:38
解决方法：
        第114行改为：
       

问题找到了，id 是一样的（后四位）

只是不知道为什么每次都在变

cjjJasonchen

歌者文明清理员 发表于 2023-8-11 09:45
问题找到了，id 是一样的（后四位）

只是不知道为什么每次都在变

因该是因为col的默认值是-1

cjjJasonchen

可以试试把棋子生成时的xy轴位置改成一个没用的格子，比如3，1啊，4，3啊之类的没有东西的格子

歌者文明清理员

cjjJasonchen 发表于 2023-8-11 09:53
可以试试把棋子生成时的xy轴位置改成一个没用的格子，比如3，1啊，4，3啊之类的没有东西的格子

有思路了，None……？

歌者文明清理员

cjjJasonchen 发表于 2023-8-11 09:53
可以试试把棋子生成时的xy轴位置改成一个没用的格子，比如3，1啊，4，3啊之类的没有东西的格子

1. class Chessman(pygame.sprite.Sprite):
   
2.     def __init__(self, color, name):
   
3.         # color: b黑色 / w白色
   
4.         # name: k王 / q后 / r车 / b象 / n马 / p兵
   
5.         # row for y, col for x
   
6.         self.color = color
   
7.         self.name = name
   
8.         self.image = pygame.image.load(f'images/{color}{name}.png')
   
9.         self.back = pygame.Surface((GRID, GRID)).convert_alpha()
   
10.         self.back.fill((0, 0, 0, 0))
    
11.         pygame.draw.circle(self.back, 'BurlyWood', self.back.get_rect().center, GRID / 2)
    
12.         self.rect = self.image.get_rect()
    
13.         # 不显示棋子
    
14.         self.rect.topleft = (-GRID, -GRID)
    
15.         self._row = -1
    
16.         self._col = -1
    
17. 
    
18.     @property
    
19.     def row(self):
    
20.         return self._row
    
21.    
    
22.     @row.setter
    
23.     def row(self, row):
    
24.         old = self._row
    
25.         self._row = row
    
26.         self.rect.top = row * GRID
    
27.         if old > -1:
    
28.             chessboard[old][self._col] = None
    
29.             chessboard[row][self._col] = self
    
30. 
    
31.     @property
    
32.     def col(self):
    
33.         return self._col
    
34. 
    
35.     @col.setter
    
36.     def col(self, col):
    
37.         old = self._col
    
38.         self._col = col
    
39.         self.rect.left = col * GRID
    
40.         if old > -1:
    
41.             chessboard[self._row][old] = None
    
42.             chessboard[self._row][col] = self

复制代码

还是一样

cjjJasonchen

enmmm,还是最简单的方法吧，把车挪到最后生成
114行改成：

1. for name in 'kqbbnnpppppppprr':

复制代码

歌者文明清理员

cjjJasonchen 发表于 2023-8-11 10:06
enmmm,还是最简单的方法吧，把车挪到最后生成
114行改成：

真的不能解决吗

cjjJasonchen

歌者文明清理员 发表于 2023-8-11 18:35
真的不能解决吗

我真的没看明白哪里出了问题。。。。

对不起

鱼cpython学习者

在34行，row.setter装饰的函数里面加一个判断语句：

1. @row.setter
   
2. def row(self, row):
   
3.     old = self._row
   
4.     self._row = row
   
5.     self.rect.top = row * GRID
   
6.     if self._row != -1 and self._col != -1:
   
7.         chessboard[old][self._col] = None
   
8.         chessboard[row][self._col] = self

复制代码

你用这个setter来实现移动棋子到另一个格子，并把原先的格子清空。但是在初始化阶段，你在初始化棋子的行数chessman.row时，这个setter会顺带把棋盘(0, -1)或者(7, -1)，也就是右上或右下清空。
这也是把车放在最后生成能解决问题的原因

歌者文明清理员

鱼cpython学习者 发表于 2023-8-11 19:20
在34行，row.setter装饰的函数里面加一个判断语句：

你用这个setter来实现移动棋子到另一个格子，并把原 ...

抱歉来晚了

1.     @row.setter
   
2.     def row(self, row):
   
3.         old = self._row
   
4.         self._row = row
   
5.         self.rect.top = row * GRID
   
6.         if self._row != -1 and self._col != -1:
   
7.             chessboard[old][self._col] = None
   
8.             chessboard[row][self._col] = self
   
9. 
   
10.     @property
    
11.     def col(self):
    
12.         return self._col
    
13. 
    
14.     @col.setter
    
15.     def col(self, col):
    
16.         old = self._col
    
17.         self._col = col
    
18.         self.rect.left = col * GRID
    
19.         if self._row != -1 and self._col != -1:
    
20.             chessboard[self._row][old] = None
    
21.             chessboard[self._row][col] = self

复制代码

还是一样捏……

鱼cpython学习者

歌者文明清理员 发表于 2023-8-11 22:41
抱歉来晚了

把col.setter里的判断语句改成

1. if self._row != -1 and old != -1:
   

复制代码

其实，我推荐你加个move函数来代替setter实现移动位置的功能

歌者文明清理员

鱼cpython学习者 发表于 2023-8-12 08:51
把col.setter里的判断语句改成

其实，我推荐你加个move函数来代替setter实现移动位置的功能

解决了，方法已经被采纳

1.     @row.setter
   
2.     def row(self, row):
   
3.         old = self._row
   
4.         self._row = row
   
5.         self.rect.top = row * GRID
   
6.         if old != -1 and self._col != -1:
   
7.             chessboard[old][self._col] = None
   
8.             chessboard[row][self._col] = self
   
9. 
   
10.     @property
    
11.     def col(self):
    
12.         return self._col
    
13. 
    
14.     @col.setter
    
15.     def col(self, col):
    
16.         old = self._col
    
17.         self._col = col
    
18.         self.rect.left = col * GRID
    
19.         if self._row != -1 and old != -1:
    
20.             chessboard[self._row][old] = None
    
21.             chessboard[self._row][col] = self

复制代码

Ewan-Ahiouy