Question

我编写了一个函数，该函数将由“，”分隔的数字字符串转换为整数数组并打印出来，因此该字符串为：

string str = "1,-20,-50,-70,6";

成为：

int a[5] = {1,-20,-50,-70,6};

一切似乎都很好，但是对于最后一个数字，如果没有正确打印，对于上面的示例，新数组为： 1，-50，-20，-70,1999882784

这是我现在拥有的：

void convert_str_to_int(string str){
    size_t stringLen = str.length(); //find string length
    int arr[stringLen]; //new array to save int values
    string holder = ""; //temporary variable
    int counter = 0;
    for (size_t i = 0;i<stringLen;i++) { 
        if (str[i] != ',')    
            holder = holder+str[i];
        else {
            arr[counter] = atoi(holder.c_str());
            holder = "";
            counter++;
        }
    }
    for (int i = 0;i<=counter;i++){
           cout<<arr[i]<<endl; //to print the array
    }
}

可能是什么问题？

Answer 1

好吧，我将为您提供完整的解决方案，但是我正在使用sstream库。请在http://www.cplusplus.com/reference/sstream/istringstream/

上阅读有关内容

std::vector<int> from_string_to_int_vector(const std::string& arr) {
   std::vector<int> numbers;
   std::string token;
   std::istringstream tokenStream(arr);
   while (std::getline(tokenStream, token, ',')) {
      numbers.push_back(atoi(token.c_str()));
   }
   return numbers;
}

Answer 2

这是因为您在counter中arr的位置没有任何元素，即arr[counter]是垃圾值。

仔细看，

//Suppose now counter = 4
arr[counter] = atoi(holder.c_str()); // so this is arr[4]
holder = ""; 
counter++; // now counter is 5

很明显，索引5不在数组大小的范围内。

所以，当您这样做

// i will have values from 0 to 5 because of i <= counter
for (int i = 0;i<=counter;i++) { 
    cout<<arr[i]<<endl; // when i == counter , it will print garbage value
}

您可能要删除=符号，而要这样做

for (int i = 0;i<counter;i++) { 
    cout<<arr[i]<<endl;
}

但是由于数组末尾没有,，因此您的代码仍然比预期少了一个元素。因此，循环结束后，您需要做

arr[counter] = atoi(holder.c_str());
counter++;

此外，计算字符串的长度并声明该长度的数组是错误的。而是这样做：

int *arr = new int[stringLen];
// if you do this, don't forget to free up the memory by using the following line whenever you no longer require arr
delete arr;

或者简单地使用矢量

vector<int> arr(stringLen);

下面是整个代码

void convert_str_to_int(string str){ 
    size_t stringLen = str.length(); 
    vector<int> arr(stringLen);
    string holder = ""; 
    int counter = 0; 
    for (size_t i = 0; i<stringLen; i++) { 
        if (str[i] != ',') {
            holder = holder+str[i]; 
        } else { 
             arr[counter] = atoi(holder.c_str()); 
             holder = ""; 
             counter++; 
        } 
    } 
    arr[counter] = atoi(holder.c_str()); 
    counter++; 
    for (int i = 0;i<=counter;i++){ 
          cout<<arr[i]<<endl; 
    }
}

Answer 3

对于您的问题，for(int i = 0; **i < counter;** ++i)。

我建议使用一些现成的c / c ++功能。

scanf，sscanf，fscanf函数具有一个格式化程序%n，该格式化程序返回当前位置。使用它。

#include <cstdio>
#include <vector>
#include <string>
int main()
{
    std::string str = "1,-20,-50,-70,6";
    std::vector<int> ints;

    const char* c_str = str.c_str();
    int pos = 0, val;

    // sscanf return number of success read variables, except %n.
    while (sscanf(c_str, "%d,%n", &val, &pos) == 1 ) 
    {
        ints.push_back(val);
        c_str += pos;
    }

    // print result array
    for (int i = 0; i < (int)ints.size(); ++i)
         printf("%d ", ints[i]);
    printf("\n");

    return 0;
}

为什么我的代码无法将字符串正确转换为int数组？

3 个答案: