求 Python 大佬看看哪个版本的代码比较好，类化争议

近期一个项目要写一个类，跟同学争执不下，特来请教各位大佬

import time

class SchoolDate(object):

    def __init__(self, school_time):
        self.weeks = 1
        self.what_day = 1
        self.school_time = time.strptime(school_time, "%Y-%m-%d")

    def get_term(self):
        timestamp = time.mktime(self.school_time)
        #计算开学前周日的时间戳
        first_week_timestamp=timestamp- 86400* int(time.strftime("%w",self.school_time))
        now_timestamp = time.time()
        self.what_day = time.strftime("%w",time.localtime(now_timestamp))
        #周数 = (当前时间戳 - 开学周日的时间戳) / 7 天 +1
        self.weeks = int((time.time() -first_week_timestamp)/ (86400*7)) + 1
        if self.weeks < 1:
            self.what_day = 1
            self.weeks = 1
        return self

    def weeks(self):
        return self.weeks

    def day(self):
        return self.what_day

版本 2


import time
from datetime import datetime


class SchoolDate(object):
    __get = False

    def __init__(self, starting_date):
        """
        :param starting_date: 开学日期
        """
        self.now_weeks = 1
        self.what_day = 1
        self.starting_date = time.strptime(starting_date, "%Y-%m-%d")

    def _get_term(self):
        self.__get = True
        timestamp = time.mktime(self.starting_date)
        # 计算开学前周日的时间戳
        first_week_timestamp = timestamp - 86400 * int(time.strftime("%w", self.starting_date))
        now_timestamp = time.time()
        self.what_day = time.strftime("%w", time.localtime(now_timestamp))
        # 周数 = (当前时间戳 - 开学周日的时间戳) / 7 天 +1
        self.now_weeks = int((time.time() - first_week_timestamp) / (86400*7)) + 1
        if self.weeks < 1:
            self.what_day = 1
            self.now_weeks = 1

    @property
    def weeks(self):
        """开学第几周"""
        if not self.__get:
            self._get_term()
        return self.now_weeks

    @property
    def week_day(self):
        """周几"""
        if not self.__get:
            self._get_term()
        return self.what_day

    @property
    def school_year(self):
        """获取当前学年"""
        now = datetime.now()
        if 2 <= now.month <= 7:
            return "%d-%d" % ((now.year - 1), now.year)
        else:
            return "%d-%d" % (now.year, (now.year + 1))

初级写法，敬请吐槽！！

ipwx

2018-02-11 11:58:35 +08:00

我觉得楼主和你的小伙伴最好注意一下“副作用”有关的各种约定。

面向对象设计中，为了让调用更安全，函数语义上最好隔离“读取”和“写入”。换句话说，叫做 get_xxx 的，语义上应该只返回状态而不更改状态，叫做 set_xxx 的则只更改状态而不返回状态（返回 self 例外，这是链式调用）。

缓存策略是另一回事。它应该对调用者透明。换句话说，你要是不高兴现在的缓存策略，可以随时更换，而不影响接口语义。所以你第一个版本是不对的，因为调用者需要明确知道 get_term 引起一次缓存，这个接口设计是不合理的。