验证用户输入是否有效 [从 'char' 到 'char*' 的转换无效]

如果你想练习复杂的算法，解析数字是一个很好的练习。但是，如果您的目标是编写有用，简单的程序，那么您就走错了路。在C++中，许多常见任务已经由C++标准库解决，您只需要使用它们即可。

#include <iostream>
#include <sstream>
#include <string>
int main() {
    std::string line;
    if (!std::getline(std::cin, line)) {
        std::cerr << "error reading the linen";
        return 1;
    }
    std::istringstream in{line};
    double num;
    if (in >> num && in.peek() == EOF) {
        std::cout << "it's a number, " << num << "n";
    } else {
        std::cout << "it's not a numbern";
    }
}

上面的代码比你的代码读起来更高级。最重要的是，它可以处理任意的长行而不会使程序崩溃。

我对C++标题不太熟悉，所以我可能忘记了包含其他一些标题。但是其余的代码应该没问题，即使我没有测试它。

函数isNumber适合您。

• 在这里，我使用动态字符序列std::string它使我们能够输入任何小于std::string::max_size的大小字符串。

• 我们可以通过以下方式检查给定字符是否为数字 std::isdigit .

• 没有额外的副本和对象创建将显示出良好的性能。

• 输入字符串的左侧和右侧不允许使用空格字符。

我还编写了迭代器的显式类型并避免使用auto，因为您要标记C++98：

#include <string>
#include <cctype>
bool isNumber(const std::string& s)
{
    // this also validates the following access to s[0]
    if(s.empty()){
        return false;
    }
    const std::size_t offset = (s[0] == '+' || s[0] == '-') ? 1 : 0;
    std::string::const_iterator begin = s.begin() + offset;
    // this also validates the following dereferencing begin
    if(begin == s.end()){
        return false; // false if just a sign "+" or "-"
    }
    if(!std::isdigit(static_cast<unsigned char>(*begin))){
        return false; // e.g. "+.123"
    }
    bool isdecimal = false;
    for(std::string::const_iterator it = ++begin; it != s.end(); ++it) 
    {
        if (!std::isdigit(static_cast<unsigned char>(*it)))
        {
            if(!isdecimal && (*it == '.'))
            {
                isdecimal = true;
                if((it+1) == s.end()){
                    return false; // e.g. "+1."
                }
            }
            else{
                return false;
            }
        }
    }
    return true;
}

现在实现 main 函数变得简单明了：

演示

#include <iostream>
int main()
{
    std::string s;
    std::cout << "Enter something: ";
    std::getline(std::cin, s);
    std::cout << std::endl;
    std::cout 
        << s << " is a " 
        << (isNumber(s) ? "number." : "character string.");
    return 0;
}

你去吧，我已经评论了我改变的东西

#include <iostream>             
bool isNum(char * p) {
    if (NULL == p || *p == '') {
        return false;
    }
    int dot = 0;
    char a = *p;
    if (a<'0' || a>'9') {
        if (a != '-' && a != '+') { return false; }
        else p++;
    }

    if (*p<'0' || *p>'9') return false;
    p++;
    while (*p != '') {
        a = *p;
        switch (a)
        {
            //Only allows 1 dot
        case '.':
            if (++dot > 1) {
                return false;
            }
            p++;
            if (*p == '') return false;
            break;
        default:
            if (a < '0' || a > '9') {
                return false;
            }
            p++;
            break;
        }
    }
    return true;
}
    int main() {
        //char array of size 1024
        char c[1024];
        std::cout << "Enter something: ";
        std::cin >> c;
        // you don't need to loop through every character just pass your array of characters & your function is looping through it
        if (!isNum(c)) {
            std::cout << c << " is a character string";
        }
        else {
            std::cout << c << " is a number";
        }
    }