Замечания по паттерну MVC

[alexey-malov]

• Обработчик сигнала от отдельно взятых гармоник используется и при изменении всех гармоник. При этом нет возможности отписаться от отдельно взятой гармоники + увеличивается риск повторного добавления обработчика. Допустимый вариант - коллекция гармоник подписывается на изменения своих дочерних объектов и шлет одно уведомления когда что-то внутри нее изменяется.
• При использовании сигналов надо помнить о том, чтобы отписаться в нужный момент. Например, если мы удалили гармонику из коллекции гармоник, изменения, происходящие в ней не должны обрабатываться коллекцией.

[alexey-malov]

• CMainDlgPresenter должен хранить соединения с сигналами от модели/view и удалять их при своем удалении (либо хранить scoped connection-ы)

[alexey-malov]

	auto harmonic = std::make_shared<CHarmonic>(amplitude, type, frequency, phase);

	auto conn = harmonic->SetHandlerToUpdateHarmonic([=] {
		auto it = std::find_if(m_harmonics.begin(), m_harmonics.end(), [](auto h) {
			return h.first == harmonic;
		});
		if (it != m_harmonics.end())
		{
			size_t index = it - m_harmonics.begin();
		}
	});
	// std::vector<std::pair<IHarmonicPtr, sig::scoped_connection>> m_harmonics;
	m_harmonics.emplace_back(harmonic, conn);

Вот так, примерно, можно опеределить индекс гармоники при получении сигнала от нее

[alexey-malov]

• Вместо SetHandlerToFunctionTypeUpdate лучше использовать именование DoOnFunctionTypeUpdate()