是不是"OK"才开始给Skyfield对象添加属性?

Is it "OK" to just start adding attributes to Skyfield objects?

此处,“OK”当然意味着 AYOR(风险自负),但如果避免与现有属性名称发生明显冲突,则不会出现可预见的问题。

Skyfield 物体——尤其是行星——通常具有有限数量的属性。我经常编写简短的脚本来提取我保存为文本并在以后使用的数字数据。 这些本质上是 'disposable' 脚本,因为我很少使用它们超过一次或两次,而且 从来没有 分享过它们。

当我编写更持久的代码时,我当然会创建自己的容器对象。

我的问题:它似乎对我来说效果很好,所以在这个特定的上下文中,是否有任何可能出错的地方除了属性名称冲突

from skyfield.api import load
import numpy as np
import matplotlib.pyplot as plt
from mpl_toolkits.mplot3d import Axes3D

eph   = load('de421.bsp')
earth = eph['earth']
sun   = eph['sun']

ts = load.timescale()

t  = ts.utc(2016, 1, np.linspace(0, 366, 1000))

# SLOPPY WAY: just add them directly
earth.pos  = earth.at(t).position.km
sun.pos    = sun.at(t).position.km
earth.r    = np.sqrt(((earth.pos-sun.pos)**2).sum(axis=0))
earth.peri = earth.r.min()
earth.apo  = earth.r.max()

print earth.peri, earth.apo, earth.pos.shape


# BETTER WAY: tedious but more cautious
uhoh  = dict()
ep    = earth.at(t).position.km
sp    = sun.at(t).position.km
r     = np.sqrt(((ep-sp)**2).sum(axis=0))
uhoh['pos']  = ep
uhoh['r']    = r
uhoh['peri'] = r.min()
uhoh['apo']  = r.max()
earth.uhoh   = uhoh

print earth.uhoh['peri'], earth.uhoh['apo'], earth.uhoh['pos'].shape

returns:

147100175.99 152103762.948 (3, 1000)
147100175.99 152103762.948 (3, 1000)

这确实是一种在非正式 Python 代码中偶尔会遇到的模式。另一个可能出错的主要问题是,除了未来的属性名称冲突之外,库作者在想要更有效地创建数百万个对象的人们的怂恿下添加了一个 __slots__ 规范,你开始得到一个尝试添加其他属性时出错。

防御__slots__,如果出现的话,是使用你自己的子class你想使用的class。如果 subclass 也未能指定 __slots__ — 说至少 __slots__ = [] 即使它没有更多要添加的属性 — 那么 subclass 的实例是完全开放并且可以具有任何属性,因此您始终可以创建自己的子 class "unlocks" 实例并让它们具有任何属性。

请注意,像 earth 这样的对象有资格成为字典键,因此如果您有一条信息要与每个行星相关联,您可以这样说:

positions = {}
positions[earth] = ...

当您需要记住一组对象中的每一个对象的额外信息时,这是一种常见的模式。