Как описать условие в более приемлемом виде в c++?

Question

[tj57]

Есть условие :

for ( int i = 0; i < dlina; i++ )
	{

if ( ( a[i] == '0' ) || ( a[i] == '1' ) || ( a[i] == '2' ) || ( a[i] == '3' ) || ( a[i] == '4' ) || ( a[i] == '5' ) || ( a[i] == '6' ) || ( a[i] == '7' ) || ( a[i] == '8' ) || ( a[i] == '9' ) || ( a[i] == ',' ) || ( a[i] == '#' ) )
... 
        }

Можно ли сократить его ?

Answer 1

[Максим Мосейчук]

if ( (( a[i] >= '0' ) && ( a[i] <= '9' )) || ( a[i] == ',' ) || ( a[i] == '#' ) )

Comments

[jcmvbkbc]

Можно ещё чуток сократить:

#include <ctype.h>
...
if (isdigit(a[i]) || a[i] == ',' || a[i] == '#')

Answer 2

[0xD34F]

Перед циклом так:

string s = "01234567890,#";

Проверять так:

if (s.find(a[i]) != string::npos) {

Comments

[devalone]

Хреновый способ, так код будет намного менее эффективный, особенно если тс решит добавить туда ещё a-z и A-Z

[0xD34F]

devalone, согласитесь, что эта проблема не моя. Автор вопроса хотел сократить код - я предложил вариант.

[devalone]

согласитесь, что эта проблема не моя

0xD34F, то, что ты предлагаешь не эффективный код, проблема только твоя.

[0xD34F]

devalone, да вы с ума сошли. Каким образом для меня может быть проблемой предложение неэффективного кода - совершенно непонятно. Кроме того, автор вопроса никоим образом не указал, какую задачу он решает - так что возможно, мой вариант вполне приемлем по скорости, используемой памяти и т.д.

[devalone]

Каким образом для меня может быть проблемой предложение неэффективного кода - совершенно непонятно

0xD34F, ну, если ты не программист, тогде да, не может. Выше привели эффективное решение, а твой вариант не может быть эффективнее ни по скорости, ни по используемой памяти.

[0xD34F]

devalone, точно, вы спятили.

Я не утверждал, что мой вариант может быть эффективнее - так к чему ваше утверждение, что он... чего-то там? Ещё разочек, вдруг до вас дойдёт: автор вопроса никоим образом не указал, какую задачу он решает - так что возможно, мой вариант вполне приемлем по скорости, используемой памяти и т.д. Не улавливаете разницу между "приемлем" и "эффективнее"?

Или, постойте... Так вы телепат? Знаете, что именно нужно автору и потому с такой наглой уверенностью утверждаете, что предложенный мной вариант "хреновый"? Завидую чёрной завистью вашим сверхъестественным способностям.

[devalone]

0xD34F, написал хреновый код и оправдывается.. У автора указано, что нужно упростить условие и сделать это можно неэффективно, как у тебя или же эффективно, как указано выше, так зачем делать хуже, если на написание кода потратится столько же времени?

[0xD34F]

devalone, юноша, возьмите себя в руки. Попейте водички, успокойтесь.

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

Лайк за решение на C++, а не на Си. `string` можно и глобально объявить, если на то пошло.