Это равноценно?

[Saturn.65]

Например: на форме три кнопки. Две из них выполняют свои команды, а третья кнопка выполняет команды первых двух кнопок. Обязательно ли вписывать в команду третьей кнопки команды первых двух?

Код: Выделить всё

Private Sub Command1_Click()
    Команда1
    Команда2
    Команда3
End Sub
Private Sub Command2_Click()
    Команда11
    Команда22
    Команда33
End Sub
Private Sub Command3_Click()
    Команда1
    Команда2
    Команда3
    Команда11
    Команда22
    Команда33
End Sub


Или можно упростить?

Код: Выделить всё

Private Sub Command3_Click()
    Command1_Click
    Command2_Click
End Sub


Работать то будет, а вот как по функциональности? Хуже или без разницы?

[tyomitch]

Второй вариант существенно лучше.

Если в программе нечто повторяется дважды, это показатель, что она плохо написана. (См. тж. viewtopic.php?f=27&t=27324)

[Александр Дмитриев]

ИМХО Полностью равноценно, так что более приемлем второй вариант. Происходит простой вызов функции. Но, может быть, лучше эти действия (Команда1, Команда2, Команда3, Команда11, Команда22, Команда33) выделить в две отдельных процедуры и вызывать их из обработчиков событий?

[Saturn.65]

Спасибо за ответы. Я именно так и писал, как во втором варианте, но вот сомневался, правильно ли делаю.

[alibek]

Если в программе нечто повторяется дважды, это показатель, что она плохо написана. (См. тж. viewtopic.php?f=27&t=27324)

Не обязательно.
Задачи всякие бывают, и в некоторых производительность гораздо важнее, чем изящность кода.
В SQL такое повсюду.

[jangle]

Работать то будет, а вот как по функциональности? Хуже или без разницы?

Зависит от задачи, если у тебя в цикле десятки тысяч раз вызывается Command3_Click (расчет какой-нибудь трехмерки например). То лучше, вызывать процедуры так:

Код: Выделить всё

    Команда1
    Команда2
    Команда3
    Команда11
    Команда22
    Команда33

выиграешь несколько процентов производительности

[tyomitch]

Несколько процентов на вызов процедуры?
Не верю. Если внутри неё самой есть хоть что-нибудь осмысленное, разница должна быть в доли процента.