从Perl转Python：类方法return语句语法错误求助

刚从Perl转Python确实容易踩语法和习惯的坑，我帮你梳理下代码里的问题，以及修正后的版本：

核心语法错误点 #

• 类代码块未缩进：Python靠缩进划分代码块，类定义后的所有属性、方法都必须缩进（通常4个空格），你的代码里类内的内容全是顶格写的，这是最大的语法错误来源。
• 列表与字典方法调用错误：列表没有.get[]这种用法，直接用方括号[]索引即可；字典的get是方法，必须用括号调用（比如self.ByName.get(bookname)），不是方括号。
• None判断逻辑错误：判断变量是否为None应该用if idx is not None，而不是和字符串"NoneType"比较——type(idx)返回的是类型对象，和字符串完全不匹配。
• 字典取值错误：self.ByNum.booknum是错误写法，字典要通过键取值，应该用self.ByNum.get(booknum)。
• 括号不匹配：bynum方法里的self.BookSum[self.ByNum.get[int(bnum)]少了一个右括号，且调用get的方式完全错误。
• 类属性vs实例属性混淆：你直接在类里定义bookptr = 0等属性，这些是类属性，会被所有实例共享，应该在__init__方法里初始化实例属性，保证每个BookSumIndex实例独立。

修正后的代码 #

# 先假设你已经定义了BookSumObject类（如果需要存传入的thisbook，可修改addbook逻辑）
class BookSumObject:
    def __init__(self):
        # 这里可以添加BookSumObject的初始化逻辑
        pass

class BookSumIndex():
    def __init__(self):
        # 初始化实例属性，每个实例独立拥有这些数据
        self.bookptr = 0
        self.BookSum = []  # 对象容器
        self.ByName = {}   # 名称到容器索引的映射
        self.ByNum = {}    # 编号到容器索引的映射

    def addbook(self, thisbook, bookname, booknum):
        # 若要存入传入的thisbook，可替换为 self.BookSum.append(thisbook)
        self.BookSum.append(BookSumObject())
        # 直接赋值更新映射，比创建临时字典再update更简洁
        self.ByName[bookname] = self.bookptr
        self.ByNum[booknum] = self.bookptr
        self.bookptr += 1

    def byname(self, bookname):
        idx = self.ByName.get(bookname)
        if idx is not None:
            return self.BookSum[idx]
        # 未找到时返回None，也可根据需求抛出异常
        return None

    def bynum(self, booknum):
        idx = self.ByNum.get(booknum)
        if idx is not None:
            return self.BookSum[idx]
        return None

额外优化建议 #

• addbook方法里的thisbook参数看起来没被使用，如果是要把传入的thisbook存入容器，直接改成self.BookSum.append(thisbook)即可，无需新建BookSumObject()。
• 如果希望找不到对应名称/编号时抛出明确错误，可直接用self.ByName[bookname]取值（找不到会触发KeyError），或者主动抛出自定义异常，方便调用方处理错误场景。

内容的提问来源于stack exchange，提问作者James Aanderson