istringstream不尊重基地？

Question

由于使用了atoi和atof，我试图修复一些关于污染值的Coverity调查结果。我切换到istringstream，但它没有为10以外的基数产生预期的结果。

如果我切换到16位，输入0xa并避开iss.ignore(2);，则结果为0：

$ ./tt.exe 0xa
X: 0

如果我切换到16位，输入0xa并使用iss.ignore(2);，则结果是例外：

$ ./tt.exe 0xa
'0xa' is not a value

我按照@πάντα的建议访问了CPP Reference on istringstream，但没有讨论此背景下的限制。

任何想法我做错了什么？或者，我怎样才能使其按预期工作？

$ cat tt.cxx
#include <iostream>
#include <sstream>
#include <iomanip>
#include <stdexcept>
using namespace std;

template <class T>
T StringToValue(const std::string& str) {
  std::istringstream iss(str);
  T value;

  if (str.length() >= 2) {
    if (str[0] == '0' && (str[1] =='X' || str[1] =='x'))
      {
        iss.setf(std::ios_base::hex);
        iss.ignore(2);
      }
  }
  iss >> value;

  if (iss.fail())
    throw runtime_error("'" + str +"' is not a value");

  return value;
}

int main(int argc, char* argv[])
{
  try
    {
      int x = StringToValue<int>(argc >= 2 ? argv[1] : "ZZZ...");
      cout << "X: " << x << endl;
    }
  catch(const runtime_error& ex)
    {
      cerr << ex.what() << endl;
      return 1;
    }

  return 0;
}

Answer 1

你完全过分思考这个问题。以十六进制表示法读取值很容易。

#include <sstream>
#include <iomanip>
#include <cassert>

int main()
{
   {
      int x = 0;
      std::istringstream is("0xa");
      is >> std::hex >> x;
      assert(x == 10);
   }
   {
      int x = 0;
      std::istringstream is("a");
      is >> std::hex >> x;
      assert(x == 10);
   }
}

（live demo）

无论如何，我可以确定您的代码存在两个问题。

1。 std::ios_base::setf

的使用不正确

您正在使用 std::ios_base::hex替换流的整个标记集。标志不仅仅用于数字基础。要做到这一点，你需要掩盖其他一切，以防止取消其他，无关，但必要的标志（我不知道它们是什么）：

iss.setf(std::ios_base::hex, std::ios::basefield);

这就是iss >> std::hex更容易的原因。

这也是为什么你应该在发布之前构建一个由组成的最小测试用例，而不是iss.setf 的测试！

2。破碎的逻辑

输入只是“a”的情况完全跳过该语句，因为条件是前两个字符为“0x”。

将它移到iss >> value之前，我愿意。

你可以看到你的固定代码here但是，正如我在回答的开头所探讨的那样，整个切换领先 - "0x"是不必要的，所以你的大多数代码都可以除去。