减少代码行而不影响性能

Question

我还在学习Python（3.5），我有一段代码，它需要为一些对象分配很多值，所以我想知道什么是最优化的方法。我问这个问题是因为我缺乏知识来理解循环中“更少的代码行”是否比不使用循环需要更多的时间（文件大小更小，但处理过程增加了。。。也许？）。你知道吗

方法1:

    # 01
    self.lineReferencia1.setText(utilidades.ensure_its_str(registro[16]))
    self.lineConcepto1.setText(utilidades.ensure_its_str(registro[17]))
    self.lineCantidad1.setText(utilidades.ensure_its_str(registro[18]))
    self.linePrecio1.setText(utilidades.ensure_its_str(registro[19]))
    self.lineDescuento1.setText(utilidades.ensure_its_str(registro[20]))
    # 02
    self.lineReferencia2.setText(utilidades.ensure_its_str(registro[23]))
    self.lineConcepto2.setText(utilidades.ensure_its_str(registro[24]))
    self.lineCantidad2.setText(utilidades.ensure_its_str(registro[25]))
    self.linePrecio2.setText(utilidades.ensure_its_str(registro[26]))
    self.lineDescuento2.setText(utilidades.ensure_its_str(registro[27]))
    # 03
    self.lineReferencia3.setText(utilidades.ensure_its_str(registro[30]))
    self.lineConcepto3.setText(utilidades.ensure_its_str(registro[31]))
    self.lineCantidad3.setText(utilidades.ensure_its_str(registro[32]))
    self.linePrecio3.setText(utilidades.ensure_its_str(registro[33]))
    self.lineDescuento3.setText(utilidades.ensure_its_str(registro[34]))
    # 04
    self.lineReferencia4.setText(utilidades.ensure_its_str(registro[37]))
    self.lineConcepto4.setText(utilidades.ensure_its_str(registro[38]))
    self.lineCantidad4.setText(utilidades.ensure_its_str(registro[39]))
    self.linePrecio4.setText(utilidades.ensure_its_str(registro[40]))
    self.lineDescuento4.setText(utilidades.ensure_its_str(registro[41]))

方法2:

    items = (
    (self.lineReferencia1, registro[16]), (self.lineConcepto1, registro[17]), (self.lineCantidad1, registro[18]),
    (self.linePrecio1, registro[19]), (self.lineDescuento1, registro[20]), (self.lineReferencia2, registro[23]),
    (self.lineConcepto2, registro[24]), (self.lineCantidad2, registro[25]), (self.linePrecio2, registro[26]),
    (self.lineDescuento2, registro[27]), (self.lineReferencia3, registro[30]), (self.lineConcepto3, registro[31]),
    (self.lineCantidad3, registro[32]), (self.linePrecio3, registro[33]), (self.lineDescuento3, registro[34]),
    (self.lineReferencia4, registro[37]), (self.lineConcepto4, registro[38]), (self.lineCantidad4, registro[39]),
    (self.linePrecio4, registro[40]), (self.lineDescuento4, registro[41]))
    for i in items:
        i[0].setText(utilidades.ensure_its_str(i[1]))

注意：使用这种方法，我将20行代码缩减为10行，但真正的代码并不是只有4个部分，而是27个部分，因此目标是缩减135行代码。你知道吗

方法3:

使用dict来存储数据，而不是元组的元组，在处理过程中会更有效吗？你知道吗

方法4:

我应该为每个项目使用方法/函数而不是循环吗？你知道吗

真正的问题是：有没有其他方法可以在不影响性能或可读性的情况下减少文件中的代码行？怎样？为什么？你知道吗

Answer 1

代码本身看起来像是开发得很糟糕的接口，但是假设您必须使用它，那么您可以通过编程获得带有getattr的对象的属性

所以，组id，索引引用点和属性，并使smth类似

groups = (1, 2, 3, 4)
attrs = ('Referencia', 'Concepto', 'Cantidad', 'Precio', 'Descuento')
idx = (15, 22, 29, 38)

for group, index in zip(groups, idx):
    for i, attr in enumerate(attrs):
        getattr(self, 'line' + attr + str(i)).setText(utilidades.ensure_its_str(registro[i+index]))

关于整体代码质量：

• 这样的命名（用连续的名字）是非常糟糕的设计。如果您有序列-您可能需要按序列存储此数据-将lineReferencia存储为对象列表
• 您可能会在某个时候遇到这种情况，也可能不会，但请尝试为代码保留英文名称-这使它更易于广泛的人阅读。有时它可能成为代码库支持的关键

Answer 2

好方法：面向对象

应该用Referncia，Concepto。。。你的主类包含一个行列表。这将与您的扩展版本一样快。你知道吗

Class Line:
    def __init__(self, ......):
         self.referencia = .....

Class Polygon:
    def __init__(self):
         self.lines = ()
         line1 = Line(.......)
         self.lines.append(line1)
         line2 = 

快速方法：对于解释语言

member_names = ("lineReferencia", "lineConcepto", "lineCantidad", "linePrecio", "lineDescuento")
for i in range(4):
   for member_name in member_names:
       text_widdget = self.__dict__["lineReferencia"  + str(i)]
       i_register = 16 + member_names.index(member_name) + 7 * i
       text_widget.setText(utilidades.ensure_its_str(registro[i_register]))

注意self.__dict__的用法，它是一个包含当前对象成员变量的dict，它们的名称作为键。你知道吗