Question

我正在研究一个c ++项目（我刚开始学习）并且无法理解为什么这个函数不起作用。我正在尝试使用变量first_name编写“Person”类，并使用函数set_first_name来设置名称。 Set_first_name需要调用一个函数（下面的函数）来检查名称中是否有任何数字。该函数总是返回false，我想知道为什么？此外，这是检查数字的最佳方式，还是有更好的方法？

   bool Person::contains_number(std::string c){ // checks if a string contains a number
        if (c.find('0') == std::string::npos || c.find('1') == std::string::npos || c.find('2') == std::string::npos || c.find('3') == std::string::npos
        || c.find('4') == std::string::npos || c.find('5') == std::string::npos || c.find('6') == std::string::npos || c.find('7') == std::string::npos
        || c.find('8') == std::string::npos || c.find('9') == std::string::npos){// checks if it contains number

        return false;
        }
        return true;
    }

Answer 1

将您的所有||更改为&&。

更好的是：

return std::find_if(s.begin(), s.end(), ::isdigit) != s.end();

或者，如果你有：

return std::any_of(s.begin(), s.end(), ::isdigit);

Answer 2

C ++ 11：

#include <algorithm>
#include <cctype>
#include <string>
#include <iostream>

bool has_any_digits(const std::string& s)
{
    return std::any_of(s.begin(), s.end(), ::isdigit);
}

int main()
{
    std::string query("H311o, W0r1d!");

    std::cout << query << ": has digits: "
              << std::boolalpha
              << has_any_digits(query)
              << std::endl;
    return 1;
}

输出：

H311o, W0r1d!: has digits: true

Answer 3

How to test if a string contains any digits in C++

应该这样做！

if (std::string::npos != s.find_first_of("0123456789"))
{
  std::cout << "digit(s)found!" << std::endl;
}

Answer 4

它总是返回false，因为你的逻辑是倒退的。您正在使用||运算符进行== npos检查。如果字符串中缺少任何一个特定数字，== npos评估为true并且||已满足，则返回false。如果任何检查评估为!= npos，则需要使用true检查然后返回true：

bool Person::contains_number(const std::string &c)
{
    if (c.find('0') != std::string::npos ||
        c.find('1') != std::string::npos ||
        c.find('2') != std::string::npos ||
        c.find('3') != std::string::npos ||
        c.find('4') != std::string::npos ||
        c.find('5') != std::string::npos ||
        c.find('6') != std::string::npos ||
        c.find('7') != std::string::npos ||
        c.find('8') != std::string::npos ||
        c.find('9') != std::string::npos)
    {
        return true;
    }

    return false;
}

或者：

bool Person::contains_number(const std::string &c)
{
    return (
        c.find('0') != std::string::npos ||
        c.find('1') != std::string::npos ||
        c.find('2') != std::string::npos ||
        c.find('3') != std::string::npos ||
        c.find('4') != std::string::npos ||
        c.find('5') != std::string::npos ||
        c.find('6') != std::string::npos ||
        c.find('7') != std::string::npos ||
        c.find('8') != std::string::npos ||
        c.find('9') != std::string::npos
    );
}

更简单的解决方案是使用find_first_of()代替find()：

bool Person::contains_number(const std::string &c)
{
    return (c.find_first_of("0123456789") != std::string::npos);
}

Answer 5

您正在使用||（或运算符）检查if语句中的多个条件。如果其中一个表达式为真，则运算符返回true（满足条件）。

or运算符首先计算左边的表达式：如果为真，那么它不会计算右边的表达式并返回true。如果左边的表达式为false，则计算右边的表达式，并返回结果作为||的结果操作

这是您的功能中发生的事情：

c包含'0'吗？如果不是（因为find（）中的std :: string :: npos表示未找到），则返回false
c包含'1'吗？如果没有则返回false
...

所以，用＆amp;＆amp;替换或运算符。（和运营商）。

用于检查字符串是否包含数字的函数

5 个答案: