对于象攻击的边界处理疑似有问题 + 无法解析pgn中部分兵的走法

[PikaCat-OuO]

python-chinese-chess/cchess/__init__.py

Line 391 in 5a59068

if not (0 <= sq <= 90) or square_distance(sq, square) > 2:

这里的90应该是 < 而不是 <=

此外，对于以下pgn，无法解析：
test.zip

对于诸如”三兵平x“，貌似无法解析：
https://www.xqbase.com/protocol/cchess_move.htm

会抛出下面这个异常

python-chinese-chess/cchess/__init__.py

Line 2247 in 5a59068

raise ValueError("记号存在歧义(未指明兵(卒)所在列)")

[windshadow233]

感谢！您提出的问题，我来改改～

[windshadow233]

对于您提到的三兵进一、三兵平六等走法报错，是因为我代码里的解析规则采用了“旧的记谱方式”，可见维基百科 - 象棋 - 记谱方法中第2点第三小点的说明，按此记谱法，原谱中的“三兵进一”、“三兵平六”，应改为“前七进一”、“前七平六”。如有额外的解析需求，等我后面空了再看看怎么添加。

[PikaCat-OuO]

应该不难，可以把这个

python-chinese-chess/cchess/__init__.py

Lines 2246 to 2247 in 5a59068

	if multi_pawns_col_number > 1 and pawn_column is None:
	raise ValueError("记号存在歧义(未指明兵(卒)所在列)")

改成这个看看，同时兼容新旧两种记法

if multi_pawns_col_number > 1 and pawn_column is None:
    # 可能是新的兵(卒)记法
    count = ['一', '二', '三', '四', '五'].index(rank_notation)
    for i, num in enumerate(reversed(pawn_nums) if color else pawn_nums):
        if num >= 2:
            if count >= num:
                count -= num
            else:
                pawn_column = 8 - i if color else i
                rank_notation = (['前', '后'] if num == 2 else ['前', '中', '后'])[count]
                break
    else:
        raise ValueError("旧记法:记号存在歧义(未指明兵(卒)所在列) 或 新记法:记号中兵(卒)的数量超出实际兵(卒)的数量")

[windshadow233]

太感谢了！我最近也是忙得很 没啥时间细看以前写的屎山代码。过会我看看更新一下您提供的代码在 2024年2月3日，15:55，PikaCat ***@***.***> 写道： 应该不难，可以把这个 https://github.com/windshadow233/python-chinese-chess/blob/5a59068df50059bb5d18c648e8ca3a3b41b974de/cchess/__init__.py#L2246-L2247 改成这个看看，同时兼容新旧两种记法 if multi_pawns_col_number > 1 and pawn_column is None: # 可能是新的兵(卒)记法 if multi_pawns_col_number == 2: count = ['一', '二', '三', '四', '五'].index(rank_notation) for i, num in enumerate(reversed(pawn_nums) if color else pawn_nums): if num >= 2: if count >= num: count -= num else: pawn_column = 8 - i if color else i rank_notation = (['前', '后'] if num == 2 else ['前', '中', '后'])[count] break else: raise ValueError("记号存在歧义(未指明兵(卒)所在列)") —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>

[PikaCat-OuO]

我觉得，你还需要修改:

python-chinese-chess/cchess/__init__.py

Line 2189 in 8cc3651

"(?:[前中后二三四五][兵卒1-9一二三四五六七八九]))"

python-chinese-chess/cchess/__init__.py

Line 1955 in 8cc3651

elif piece_notation[0] in ['中', '二', '三', '四', '五']:

以识别”一兵平x“的notation

[windshadow233]

甚是有理！

…

2024年2月4日 21:02，PikaCat ***@***.***> 写道： 我觉得，你还需要修改: https://github.com/windshadow233/python-chinese-chess/blob/8cc3651939e0ee72710affd3999321dd88463025/cchess/__init__.py#L2189 <https://github.com/windshadow233/python-chinese-chess/blob/8cc3651939e0ee72710affd3999321dd88463025/cchess/__init__.py#L2189> https://github.com/windshadow233/python-chinese-chess/blob/8cc3651939e0ee72710affd3999321dd88463025/cchess/__init__.py#L1955 <https://github.com/windshadow233/python-chinese-chess/blob/8cc3651939e0ee72710affd3999321dd88463025/cchess/__init__.py#L1955> 以识别”一兵平x“的notation — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AMCSSQAX4XTOM7DONJIF5WLYR6BG3AVCNFSM6AAAAABCXU6WA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVG42DCNRZHA>. You are receiving this because you commented.