Ответ 1
Выглядит хорошо, но вы можете использовать isalnum(c)
вместо isalpha
и isdigit
.
Я пишу функцию, которая определяет, содержит ли строка только буквенно-цифровые символы и пробелы. Я эффективно проверяю, соответствует ли оно регулярному выражению ^[[:alnum:] ]+$
, но не использует регулярные выражения. Это то, что у меня есть до сих пор:
#include <algorithm>
static inline bool is_not_alnum_space(char c)
{
return !(isalpha(c) || isdigit(c) || (c == ' '));
}
bool string_is_valid(const std::string &str)
{
return find_if(str.begin(), str.end(), is_not_alnum_space) == str.end();
}
Есть ли лучшее решение или способ "больше С++" для этого?
Выглядит хорошо, но вы можете использовать isalnum(c)
вместо isalpha
и isdigit
.
И с нетерпением жду С++ 0x, вы сможете использовать лямбда-функции (вы можете попробовать это с помощью gcc 4.5 или VS2010):
bool string_is_valid(const std::string &str)
{
return find_if(str.begin(), str.end(),
[](char c) { return !(isalnum(c) || (c == ' ')); }) == str.end();
}
Вы также можете сделать это со связующими, чтобы вы могли отказаться от вспомогательной функции. Я бы рекомендовал Boost Binders, поскольку они намного проще в использовании, чем стандартные связующие библиотеки:
bool string_is_valid(const std::string &str)
{
return find_if(str.begin(), str.end(),
!boost::bind(isalnum, _1) || boost::bind(std::not_equal_to<char>, _1, ' ')) == str.end();
}
Незначительные точки, но если вы хотите, чтобы is_not_alnum_space() была вспомогательной функцией, которая видна только в этом конкретном блоке компиляции, вы должны поместить ее в анонимное пространство имен, а не статично:
namespace {
bool is_not_alnum_space(char c)
{
return !(isalpha(c) || isdigit(c) || (c == ' '));
}
}
...etc