将代码块优化为循环

Optimising a code block into a loop

本文关键字：循环 优化 代码      更新时间：2023-10-16

我对C++和编译语言很陌生，我有很强的解释背景，所以我试图绕过在编译时无法访问某些东西的限制。

目前我有一大块代码，看起来像这样：

//New note, note36
MemoryInputStream* input36 = new MemoryInputStream(BinaryData::C1hard1_wav, BinaryData::C1hard1_wavSize, false);
auto reader36 = audioFormatManager.createReaderFor(input36);
BigInteger note36;
note36.setRange(36, 1, true);
addSound(new SamplerSound("note36", *reader36, note36, 36, 0.001, 0.01, 26.0));
delete input36;
delete reader36;
//New note, note37
MemoryInputStream* input37 = new MemoryInputStream(BinaryData::Csharp1hard1_wav, BinaryData::Csharp1hard1_wavSize, false);
auto reader37 = audioFormatManager.createReaderFor(input37);
BigInteger note37;
note37.setRange(37, 1, true);
addSound(new SamplerSound("note37", *reader37, note37, 37, 0.001, 0.01, 26.0));
delete input37;
delete reader37;

此代码在此方法中重复 48 次，这不是好的做法。我如何在 PHP 中实现这一点的一个例子，解释型语言看起来像这样

$noteMap = array(36 => "C1", 37 => "Csharp1");
foreach($noteMap as $midiNumber => $note)
{
$name = $note . "hard1_wav";
$size = $note . "hard1_wavSize";
$input = new MemoryInputStream(BinaryData::$name, BinaryData::$size, false);
$reader = audioFormatManager->createReaderFor($input);
//I know this bit is bad PHP, no bigintegers in PHP but I can't think of a way to replicate it for arguments sake
$note = 0;
$note->setRange($midiNumber, 1, true);
addSound(new SamplerSound("note".$midiNumber, $reader36, $note, $midiNumber, 0.001, 0.01, 26.0));
}

这更易于管理和重用。我所做的更改不需要在整个文件中仔细重复，并且可以快速进行更改。我知道将变量作为函数名称传递不是编译语言中的事情，但即使考虑到这一点，也必须有一种方法将我的大代码块包装成一个整洁的循环，我只是找不到资源在那里向我解释它。

假设 BinaryData 是 JUCE 生成的资源文件，以下算法应该等效于您的 PHP 版本。它不使用动态变量名称，而是使用应出现在 JUCE 生成的代码中的BinaryData::getNamedResource函数。

std::map<int, std::string> note_map{{36, "C1"}, {37, "Csharp1"}};
for ( const auto& note : note_map ) {
BigInteger midi_bit;
midi_bit.setBit(note.first);
int size;
std::string note_resource = note.second + "hard1_wav";
auto data = BinaryData::getNamedResource(note_resource.c_str(), size);
auto input = new MemoryInputStream(data, size, false);
auto reader = audioFormatManager->createReaderFor(input);
auto note_name = "note" + std::to_string(note.first);
addSound(new SamplerSound(note_name.c_str(), *reader, midi_bit,
note.first, 0.001, 0.01, 26.0));
}

另一个(不好的)解决方案是使用 C 预处理器生成如下所示的代码：

#define note(num, name)                                                                                               
MemoryInputStream* input##num = new MemoryInputStream(BinaryData::name##_wav, BinaryData::name##_wavSize, false); 
auto reader##num = audioFormatManager.createReaderFor(input##num);                                                
BigInteger note##num;                                                                                             
note##num.setBit(num);                                                                                            
addSound(new SamplerSound("note" #num, *reader##num, note##num, num, 0.001, 0.01, 26.0));                         
delete input##num;                                                                                                
delete reader##num
note(36, C1);
note(37, Csharp1);

预处理阶段生成以下代码：

MemoryInputStream* input36 = new MemoryInputStream(BinaryData::C1_wav, BinaryData::C1_wavSize, false); auto reader36 = audioFormatManager.createReaderFor(input36); BigInteger note36; note36.setBit(36); addSound(new SamplerSound("note" "36", *reader36, note36, 36, 0.001, 0.01, 26.0)); delete input36; delete reader36;
MemoryInputStream* input37 = new MemoryInputStream(BinaryData::Csharp1_wav, BinaryData::Csharp1_wavSize, false); auto reader37 = audioFormatManager.createReaderFor(input37); BigInteger note37; note37.setBit(37); addSound(new SamplerSound("note" "37", *reader37, note37, 37, 0.001, 0.01, 26.0)); delete input37; delete reader37;

预处理器解决方案应该是与当前解决方案等效的代码，但一般来说，以这种方式使用宏生成大量变量并不是很好的做法。上述运行时解决方案应优先于使用预处理器。

你可以做这样的事情：

typedef void (FuncPtr)(int x);
void C1Func(int x);
void CSharpFunc( int x);
void someFunc() {
const Input input = new MemoryInputStream(name, size, false);
const Reader reader = audioFormatManager->createReaderFor(input);
// note: gxx syntax here, may not be supported in other versions...    
Note noteMap[48] = { [36]=Note(36,C1Func), [37]=Note(37,CSharpFunc) };
String name, size;
for (int i = 0; i < arraySizeOf(noteMap); i++) {
Note * note = noteMap[i];
if (!note->isInitialized())
continue;
name = note + "hard1_wav";
size = note + "hard1_wavSize";
note->setRange(i,1,true);
std::string name = "note"+i;
addSound(new SamplerSound(name, reader, note[i], i, 0.001, 0.01, 26.0));
}
delete input;
delete reader;
}

你需要一个带有构造函数(Note::Note(int idx,FuncPtr fn))的Note类。 此外，数组的初始化特定于 gnu，因此您可能需要使用std::map或等效的东西，(或将INVALID_NOTE定义为无效并在其中添加一堆INVALID_NOTE,以将其他数组成员标记为空)。

不要这样，浪费资源，你在每个音符上调用 new 并删除。

请改用数据结构，例如std::vector或std::map。

填充数据结构后，您可以迭代其元素。

相关文章：