SoundLoad: оптимизация кода

[lexu5]

Добрый день всем. На просторах интернета нашёл плагин, который при загрузке на сервер, проигрывает от 1 до 5 мелодий. Подскажите пожалуйста, можно ли как-нибудь оптимизировать данный код?

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

#include amxmodx

public plugin_precache() 
{
    precache_generic("sound/test/loading/1.mp3")
    precache_generic("sound/test/loading/2.mp3")
    precache_generic("sound/test/loading/3.mp3")
    precache_generic("sound/test/loading/4.mp3")
    precache_generic("sound/test/loading/5.mp3")
}

public client_connect(id) consound(100 + id)

public consound( timerid_id )
{
    new id = timerid_id - 100
    new Usertime = get_user_time(id, 0)
    
    if (Usertime>3) return
    
    new Sounds = random_num(1, 5)
    switch(Sounds)
    {
        case 1: client_cmd(id, "mp3 play ^"sound/test/loading/1.mp3^"")
        case 2: client_cmd(id, "mp3 play ^"sound/test/loading/2.mp3^"")
        case 3: client_cmd(id, "mp3 play ^"sound/test/loading/3.mp3^"")
        case 4: client_cmd(id, "mp3 play ^"sound/test/loading/4.mp3^"")
        case 5: client_cmd(id, "mp3 play ^"sound/test/loading/5.mp3^"")
    }
}

[Subb98]

Заменить plugin_precache:

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

public plugin_precache() {
    for(new i, j, sample[64]; i < 5; i++) {
        formatex(sample, charsmax(sample), "sound/test/loading/%d.mp3", ++j);
        precache_generic(sample);
    }
} 

Удалить switch, вместо него добавить:

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

client_cmd(id, "mp3 play ^"sound/test/loading/%d.mp3^"", random_num(1, 5)); 

П.с.: цикл в plugin_precache не даст оптимизации, это рефакторинг, чтобы избежать дублирования строк.

[артист666]

В плане цикла - это не совсем оптимизация, а наоборот...
Это просто в плане читабельности, краткости удобней.

[RevCrew]

артист666, он тоже самое написал, лол

[артист666]

Да, не заметил, сорьте.

[lexu5]

Всем спасибо за ответы