Python实现安卓iOS风格StopWatch（含Lap功能）代码优化求助

1. 状态管理合规化 #

你当前的代码没有显式标记状态，完全靠几个时间变量的取值隐式判断，不符合需求中「仅三种状态、非法调用无效果」的要求，存在多个逻辑漏洞：

• 初始状态下调用stop会给__EndTime赋值，导致getElapsedTime返回远大于0的错误结果，违反初始状态定义
• 停止状态下调用lap、stop仍会执行逻辑，不符合「其余方法调用无效果」的要求
  优化方案：
• 引入枚举明确标记三种状态：INIT（初始）、RUNNING（运行）、STOPPED（停止）
• 所有方法执行前先判断当前状态，不在允许的状态范围内直接返回，不执行任何逻辑

from enum import Enum
class StopWatchStatus(Enum):
    INIT = 0
    RUNNING = 1
    STOPPED = 2

在__init__中初始化self._status = StopWatchStatus.INIT，每个方法入口先做状态校验，比如stop方法只在状态为RUNNING时执行。 #

2. 封装与命名优化 #

• 移除不必要的Java风格getter/setter：Python不推荐对外暴露内部属性的读写接口，你当前的setStartTime等方法会允许外部随意修改内部时间变量，直接破坏秒表的状态一致性，这些接口完全可以删掉，内部属性用单下划线前缀标记为私有即可
• 替换无意义的变量名：__Time1、__Time2完全没有语义，建议改为_last_lap_time（上一次计次的时间）、_elapsed_offset（累计计时偏移量，用于暂停后继续计时的场景），可读性大幅提升
• 新增计次记录存储：添加_lap_records列表存储所有历史计次数据，符合移动端秒表可以查看所有计次记录的使用习惯，而不是只能拿到当前一次的计次结果

3. 核心逻辑简化 #

你当前start方法里的时间计算逻辑过于晦涩，容易出边界bug，建议改用「累计偏移量+基准时间」的逻辑实现，清晰不易错：

• 初始状态：_elapsed_offset = 0，无基准时间
• 启动时：如果是初始状态第一次启动，记录当前时间为_base_time，切换为运行状态；如果是停止后继续启动，更新_base_time为当前时间，保留之前的_elapsed_offset
• 停止时：计算当前时间和_base_time的差值，加到_elapsed_offset里，切换为停止状态
• 计算耗时：运行状态返回_elapsed_offset + time.time() - _base_time，停止状态直接返回_elapsed_offset，初始状态返回0
  优化后的getElapsedTime逻辑不到5行，完全没有冗余计算。

4. 功能体验优化 #

• 计次功能约束：lap方法只允许在运行状态调用，初始/停止状态调用直接返回None，符合需求要求
• 时间格式化优化：convert方法可以放到StopWatch类中作为实例方法，同时补全时间前导零，输出格式改为HH:mm:ss.xx（比如00:02:35.12），更符合秒表的通用显示习惯
• 全局变量冲突修复：main函数里的变量名不要用time，会覆盖你导入的time模块，引发不可预知的错误

核心优化后代码示例 #

import time
from enum import Enum

class StopWatchStatus(Enum):
    INIT = 0
    RUNNING = 1
    STOPPED = 2

class StopWatch:
    def __init__(self):
        self._status = StopWatchStatus.INIT
        self._base_time = 0
        self._elapsed_offset = 0
        self._last_lap_time = 0
        self.lap_records = []
    
    def start(self):
        if self._status == StopWatchStatus.RUNNING:
            return
        if self._status == StopWatchStatus.INIT:
            self._base_time = time.time()
            self._last_lap_time = self._base_time
        elif self._status == StopWatchStatus.STOPPED:
            self._base_time = time.time()
            self._last_lap_time = time.time() - (self._last_lap_time - self._base_time)
        self._status = StopWatchStatus.RUNNING
    
    def stop(self):
        if self._status != StopWatchStatus.RUNNING:
            return
        self._elapsed_offset += time.time() - self._base_time
        self._status = StopWatchStatus.STOPPED
    
    def get_elapsed_time(self):
        if self._status == StopWatchStatus.INIT:
            return 0
        elif self._status == StopWatchStatus.STOPPED:
            return self._elapsed_offset
        else:
            return self._elapsed_offset + time.time() - self._base_time
    
    def lap(self):
        if self._status != StopWatchStatus.RUNNING:
            return None
        current_time = time.time()
        lap_time = current_time - self._last_lap_time
        self.lap_records.append(lap_time)
        self._last_lap_time = current_time
        return lap_time
    
    def reset(self):
        if self._status != StopWatchStatus.STOPPED:
            return
        self._status = StopWatchStatus.INIT
        self._base_time = 0
        self._elapsed_offset = 0
        self._last_lap_time = 0
        self.lap_records.clear()
    
    def format_time(self, sec):
        hours = int(sec // 3600)
        mins = int((sec % 3600) // 60)
        sec = sec % 60
        return f"{hours:02d}:{mins:02d}:{sec:.2f}"

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