Python类实现SMS存储模拟程序的技术求助

看起来你在面向对象的类设计和实例调用上有点绕晕了，咱们先把核心问题理清楚：原代码里把类的实例方法和全局操作混在一起了，而且SMSMessage类的hasBeenRead属性没正确初始化，调用方法的方式也不对（比如直接用类名调用实例方法，或者把列表当成类来调用方法）。

我帮你重构了代码，既符合面向对象的设计，又能实现你需要的功能：

# 定义单个SMS消息的类
class SMSMessage:
    def __init__(self, from_number, message_body):
        self.from_number = from_number
        self.message_body = message_body
        self.has_been_read = False  # 默认未读状态

    def mark_as_read(self):
        self.has_been_read = True

# 定义管理所有SMS消息的类
class SMSStore:
    def __init__(self):
        self.messages = []  # 用实例内部列表存储消息，替代全局变量

    def add_sms(self):
        source_number = input("请输入来源号码：")
        message_body = input("请输入消息内容：")
        new_message = SMSMessage(source_number, message_body)
        self.messages.append(new_message)
        print("消息已成功添加！")

    def get_count(self):
        return len(self.messages)

    def get_all_messages(self):
        # 展示所有消息的详细信息
        for idx, msg in enumerate(self.messages):
            read_status = "已读" if msg.has_been_read else "未读"
            print(f"[{idx+1}] 发送方：{msg.from_number} | 状态：{read_status} | 内容：{msg.message_body}")

    def get_unread_messages(self):
        # 筛选并展示所有未读消息
        unread_messages = [msg for msg in self.messages if not msg.has_been_read]
        if not unread_messages:
            print("当前没有未读消息！")
            return
        print("=== 未读消息列表 ===")
        for idx, msg in enumerate(unread_messages):
            print(f"[{idx+1}] 发送方：{msg.from_number} | 内容：{msg.message_body}")

    def mark_message_as_read(self, message_index):
        # 根据索引标记消息为已读（注意列表索引从0开始）
        if 0 <= message_index < len(self.messages):
            self.messages[message_index].mark_as_read()
            print("消息已标记为已读！")
        else:
            print("错误：输入的消息序号无效！")

    def remove_sms(self, message_index):
        if 0 <= message_index < len(self.messages):
            del self.messages[message_index]
            print("消息已成功删除！")
        else:
            print("错误：输入的消息序号无效！")

# 主程序交互逻辑
def main():
    sms_manager = SMSStore()  # 创建SMSStore的实例，所有操作通过这个实例完成
    while True:
        user_choice = input("\n请选择操作：read（读取消息）/ send（发送消息）/ quit（退出）：").strip().lower()
        if user_choice == "send":
            sms_manager.add_sms()
            # 如果需要批量添加，可以改成循环：比如让用户输入要添加的数量，然后循环调用add_sms
        elif user_choice == "read":
            total_messages = sms_manager.get_count()
            if total_messages == 0:
                print("当前没有任何消息！")
                continue
            # 先展示所有消息
            sms_manager.get_all_messages()
            # 让用户选择要标记为已读的消息
            try:
                selected_idx = int(input("\n请输入要标记为已读的消息序号（输入0跳过）：")) - 1
                if selected_idx != -1:
                    sms_manager.mark_message_as_read(selected_idx)
            except ValueError:
                print("请输入有效的数字！")
        elif user_choice == "quit":
            print("再见！")
            break
        else:
            print("输入错误，请重新选择！")

if __name__ == "__main__":
    main()

核心修正与说明 #

• 类职责拆分：

  • SMSMessage只负责单个消息的属性和标记已读的行为，职责单一清晰；
  • SMSStore负责管理所有消息的集合，包含添加、查询、删除、标记等操作，避免了全局变量的混乱。

• 正确的实例化与方法调用：

  • 首先创建SMSStore的实例：sms_manager = SMSStore()，所有操作都通过这个实例调用方法（比如sms_manager.add_sms()）；
  • 每个新消息都是SMSMessage的实例：new_message = SMSMessage(source_number, message_body)，自带has_been_read属性和mark_as_read方法。

• 已读/未读标记的实现：

  • 在SMSMessage初始化时默认设置has_been_read = False；
  • 标记已读时，通过调用消息实例的mark_as_read()方法修改属性，查询未读消息时通过筛选属性值实现。

• 避免全局变量：

  • 把消息列表放到SMSStore类的实例内部（self.messages），符合面向对象的封装原则，数据更安全。

内容的提问来源于stack exchange，提问作者Burger Du Plessis