相关变量和操作的分组？

Question

在Code Complete，第10章中，建议对相关语句进行分组，并给出了以下示例：

void SummarizeData(...) {
    ...
    GetOldData( oldData, &numOldData );
    GetNewData( newData, &numNewData );
    totalOldData = Sum( oldData, numOldData );
    totalNewData = Sum( newData, numNewData );
    PrintOldDataSummary( oldData, totalOldData, numOldData );
    PrintNewDataSummary( newData, totalNewData, numNewData );
    SaveOldDataSummary( totalOldData, numOldData );
    SaveNewDataSummary( totalNewData, numNewData );
    ...
}

据说这种分组和并发处理是糟糕的设计，而是给出了更分离的东西：

void SummarizeData(...) {
    GetOldData( oldData, &numOldData );
    totalOldData = Sum( oldData, numOldData );
    PrintOldDataSummary( oldData, totalOldData, numOldData );
    SaveOldDataSummary( totalOldData, numOldData );
    ...
    GetNewData( newData, &numNewData );
    totalNewData = Sum( newData, numNewData );
    PrintNewDataSummary( newData, totalNewData, numNewData );
    SaveNewDataSummary( totalNewData, numNewData );
    ...
}

我确实同意第二种方法是更容易阅读和理解，并提供看起来更清晰的代码，至少从我自己的角度来看是这样。那么，我的问题是，第二种方法有什么缺点吗？例如，我能想到的一个可能的问题是与数据库的临时连接，例如：

void SummarizeData(...) {
    ...
    externalDataStore.open();
    externalDataStore.save(oldData, numOldData);
    externalDataStore.save(newData, numNewData);
    externalDataStore.close();
    ...
}

第一种方法将在一个打开/关闭周期中完成两个保存操作。但是，使用第二种方法...

void SummarizeData(...) {
    ...
    externalDataStore.open();
    externalDataStore.save(oldData, numOldData);
    externalDataStore.close();
    ...
    externalDataStore.open();
    externalDataStore.save(newData, numNewData);
    externalDataStore.close();
    ...
}

您必须为每个操作打开和关闭连接。这看起来很浪费，但我不知道它如何影响实践中的性能。

抱歉问了一个不必要的长问题......

Answer 1

我还没有读到 Code Complete 中的第 10 章（应该再花几个晚上才能读到！），但我认为这里的要点是以逻辑且易于阅读的方式对代码行进行分组，没有影响程序功能。换句话说，尽可能地清理并重新排列它，但一旦它开始真正影响行为就停止。

在您的示例中，我们应该记住“过早优化是万恶之源”，但我认为我们仍然可以安全地假设，如果您要立即再次打开连接，则不应关闭连接，因为这两个动作实际上是相互抵消的。作为一般规则，为了简单起见，任何连接都应该仅在第一次需要它们之前打开，并在最后一次使用之后立即关闭。

Answer 2

我很无聊，所以我尝试使用 Sqlite 在 Python 中进行概念验证速度测试（我意识到这不是最好的方法）。

首先，进行 50,000 次迭代的基础测试，每次迭代后打开和关闭连接。

#!/usr/bin/env python

import sqlite3

class Creature(object):
    legs = 0

    eyes = 'monocular'

    kind = ''

conn = sqlite3.connect(':memory:')

c = conn.cursor()
c.execute('''create table testtable
        (date text, legs text, eyes text, kind text)''')
conn.commit()
c.close()

for i in range(50000):
    c = conn.cursor()

    creature1 = Creature()
    creature1.legs = 5
    creature1.eyes = 'monocular'
    creature1.kind = 'mungy'
    c.execute('insert into testtable values (?,?,?,?)', ('today', str(creature1.legs), creature1.eyes, creature1.kind))

    creature2 = Creature()
    creature2.legs = 3
    creature2.eyes = 'binocular'
    creature2.kind = 'thingy'
    c.execute('insert into testtable values (?,?,?,?)', ('today', str(creature2.legs), creature2.eyes, creature2.kind))

    creature3 = Creature()
    creature3.legs = 3
    creature3.eyes = 'monocular'
    creature3.kind = 'pungy'    
    c.execute('insert into testtable values (?,?,?,?)', ('today', str(creature3.legs), creature3.eyes, creature3.kind))

    conn.commit()
    c.close()

现在，迭代了 50,000 次，但没有关闭连接。

#!/usr/bin/env python

import sqlite3

class Creature(object):
    legs = 0

    eyes = 'monocular'

    kind = ''

conn = sqlite3.connect(':memory:')

c = conn.cursor()
c.execute('''create table testtable
        (date text, legs text, eyes text, kind text)''')
conn.commit()
c.close()

c = conn.cursor()
for i in range(50000):

    creature1 = Creature()
    creature1.legs = 5
    creature1.eyes = 'monocular'
    creature1.kind = 'mungy'

    creature2 = Creature()
    creature2.legs = 3
    creature2.eyes = 'binocular'
    creature2.kind = 'thingy'

    creature3 = Creature()
    creature3.legs = 3
    creature3.eyes = 'monocular'
    creature3.kind = 'pungy'

    c.execute('insert into testtable values (?,?,?,?)', ('today', str(creature1.legs), creature1.eyes, creature1.kind))
    c.execute('insert into testtable values (?,?,?,?)', ('today', str(creature2.legs), creature2.eyes, creature2.kind))
    c.execute('insert into testtable values (?,?,?,?)', ('today', str(creature3.legs), creature3.eyes, creature3.kind))

    conn.commit()
c.close()

结果？

First method: Average 2.264s
Second method: Average 2.157s

因此，它会产生影响，尽管影响可以忽略不计。

现在你就得到了它。

不过，我绝对同意斯蒂芬所说的。