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

我访问了istringstream上的CPP参考资料，该参考资料由@πάγτα推荐，但它没有讨论这种情况下的限制。

你知道我做错了什么吗？或者，我如何才能让它按预期工作？

---

$ 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);
   }
}

（现场演示）

---

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

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之前。

---

你可以在这里看到你的固定代码，但是，正如我在回答的开头所探讨的，整个打开前导"0x"的事情是不必要的，所以你的大部分代码都可以被删除。