Question

我在尝试使用函数创建将十进制数转换为二进制数的代码时遇到了问题。起初我只使用main函数创建了代码并且工作正常，但决定将其修改为使用函数。我相信代码是正确的，但是当我尝试cout我的答案时，我会得到一个像115120160758866453687091316369641637416这样的大数字。

这是代码

  #include <iostream>
    #include <math.h>

    using namespace std;
int* unsigned_dec(int dec_M) { //function for converting absolute part of          numbers

    int bin[8] = { 0,0,0,0,0,0,0,0 };
    int ind = 7;
    int arr_ind = 0;
    for (int base = (int)abs(dec_M); base > 0; base = base / 2) {

        if (base % 2 == 0) {

            bin[arr_ind] = 0;
            ind--;
        }
        else {
            bin[arr_ind] = 1;
            ind--;
        }
        arr_ind++;
    }
    return bin;

}


int main() {// main function
    int dec_N;
    cin >> dec_N;
    int* bin_main = unsigned_dec(dec_N); //we are not sure if we are     assigning the returned value of function to array in correct  

for (int i = 0; i <= 7; i++) {

        cout << bin_main[i];
}
cout << endl;

return 0;
}

然后我尝试将cout代码更改为

cout << bin_main[0] << bin_main[1] << bin_main[2] << bin_main[3] << bin_main[4] << bin_main[5] << bin_main[6] << bin_main[7] << endl;

这很好用。然后我以其他方式写了相同的第二个cout变种

cout << bin_main[0];
cout << bin_main[1];
cout << bin_main[2];
cout << bin_main[3];
cout << bin_main[4];
cout << bin_main[5];
cout << bin_main[6];
cout << bin_main[7];
cout << endl;

我的代码开始cout同样奇怪的数字。我认为couts的所有3种方式几乎都是相同的（特别是2和3），但不明白是什么让它无法工作。

Answer 1

int bin[8] = { 0,0,0,0,0,0,0,0 };

在stack上分配。你应该在堆上分配bin

auto bin = std::unique_ptr<int, std::default_deleter<int[]>>(new int[8]);

甚至更好，请使用std::vector

Answer 2

您正在返回指向int unsigned_dec bin []的本地数组的指针。一旦来自main的另一个函数被调用，即cout运算符，该函数unsigned_dec的数组就会失效。

Answer 3

正如其他人已经提到的：函数永远不应该返回指向局部变量的指针。函数返回时局部变量无效。

更好的方法是使用向量，只需使函数返回向量。

类似的东西：

#include <iostream>
#include <math.h>

using namespace std;

//function for converting absolute part of numbers
vector<int> unsigned_dec(int dec_M) {
    vector<int> bin;   // Create a vector
    bin.resize(8, 0);  // Fill it with 8 elements initialized to zero

    int arr_ind = 0;
    // Note: added check for arr_ind being valid
    for (int base = (int)abs(dec_M); base > 0 && arr_ind < 8; base = base / 2) {
        if (base % 2 == 0) {
            bin[arr_ind] = 0;
        }
        else {
            bin[arr_ind] = 1;
        }
        arr_ind++;
    }
    return bin; // Return the vector
}


int main() {
    int dec_N;
    cin >> dec_N;
    vector<int> bin_main = unsigned_dec(dec_N);
    for (int i = 0; i < bin_main.size(); i++) {
        cout << bin_main[i];
    }
    cout << endl;

    return 0;
}

使用函数

3 个答案: