Как сократить запись?

Question

[HoShik]

Как можно сократить такую запись?
Через другие операции например

if((String[i]==' ') 
            || (String[i]==',') 
            || (String[i]=='!') 
            || (String[i]=='?') 
            || (String[i]=='.')

Answer 1

[Армянское Радио]

using test = std::set<char>;
static const test{' ', ',', '?', '.'};
if(test.find(String[i])!=test.cend())
{
}

Comments

[Антон Жилин]

В данном случае первоначальный код будет быстрее, чем std::set, т.к. раньше мы тягали из динамической памяти один символ, а теперь придётся проходиться по мапе

[Армянское Радио]

@Anton3
1. Требования по производительности заявлено не было
2. Такие утверждения следует подкреплять бенчмарком.

[res2001]

Фокс Йовович,

2. Такие утверждения следует подкреплять бенчмарком.

В принципе верно, но в данном случае излишне, т.к. по моему тут все достаточно очевидно.
Хотя в данном случае задержки по отношению к оригинальному коду будут минимальны, потому что множество маленькое, но они будут.
Поэтому я и предложил другой вариант, но он то же будет медленнее, чем оригинальный код.

HoShik, если производительность важна, то лучше оставить код так как есть - это самый быстрый вариант. Для краткости, можно вынести сравнения в #define или inline функцию и использовать ее в коде.
Многие парсеры (HTTP, JSON, XML), написанные на Си и используемые в проектах с большими нагрузками используют именно такой "многословный" подход и не пытаются экономить на строчках, т.к. скорость работы важнее красоты кода.

[Армянское Радио]

res2001, тогда еще быстрее будет переписать на switch, так как популярные компиляторы реализуют его при помощи бинарного поиска по ключам.

[res2001]

Фокс Йовович, согласен

Answer 2

[res2001]

if(std::strpbrk(String.c_str(), " ,.?") != nullptr_t)
{
}