C++：添加到输出中的不需要的数字

Question

我正在编写一个将十进制数转换为二进制和十六进制数的 C++ 程序。问题是，出于某种原因，它每次都将数字“1875954912”连接到两种表示形式。

我尝试了很多东西 - 主要是改变了程序计算 numArrayLength 的方式和我的 decToBase 函数中的 for 循环，但我还没有弄清楚为什么会这样。

顺便说一句，该程序尚未完成 - 它尚未将大于 9 的整数转换为十六进制表示的字母，但这不是我现在主要关心的问题。

这是我的代码：

#include <iostream>
#include <cmath>
using namespace std;

int howManyBitsNeeded(int someNum, int base) {
    int numOfDivisions = 0;
    while (someNum != 0) {
        someNum = floor(someNum / base);
        numOfDivisions += 1;
    }
    return numOfDivisions;
}


int decToBase(int someNum, int base) {
    int bitsNeeded = howManyBitsNeeded(someNum,base);
    int numArrayLength = bitsNeeded;
    int numArray[bitsNeeded];

    while (bitsNeeded > 0) {
        numArray[bitsNeeded] = (someNum % base);
        someNum = floor(someNum / base);
        bitsNeeded -= 1;
    }

    for (int k = (numArrayLength-1); k >= 0; --k) {
        cout << numArray[(numArrayLength - k)];
    }

}


int main() {
    int inpNum;

    cout << "Enter your number: ";
    cin >> inpNum;

    cout << "Binary representation: " << decToBase(inpNum,2) << endl;
    cout << "Hexadecimal representation: " << decToBase(inpNum,16);

    return 0;
}

输出如下：

Enter your number: 25
Binary representation: 110011875954912
Hexadecimal representation: 191875954912

如有任何帮助，我们将不胜感激！

Answer 1

您的 decToBase 被声明为 return 一个 int，但它实际上并没有 return 任何东西。您的编译器应该警告您这一点。由于您在这里没有 returning 任何内容，请将其 return 类型更改为 void。然后不要尝试打印它的 return 值，只需调用函数而不打印它：

std::cout << "Binary representation: ";
decToBase(inpNum, 2); // this already prints the number, no need to pass it to std::cout
std::cout << endl;

std::cout << "Hexadecimal representation: ";
decToBase(inpNum, 16);
std::cout << std::endl;

或者你当然可以将函数更改为 return 你想要打印的字符串，而不是在函数内部打印它。

还有一个问题：

int numArray[bitsNeeded];

当您尝试在此处访问它时，它超出了范围：

while (bitsNeeded > 0) {
        numArray[bitsNeeded] = (someNum % base);

以及稍后您尝试打印它时。要消除一个错误，您必须将其更改为

numArray[bitsNeeded-1] = (someNum % base);

并在输出中将其更改为

cout << numArray[(numArrayLength - k -1)];

当你在使用它时，我建议不要将其作为 VLA (which isn't part of C++ and only works if the compiler tolerates it)，而是推荐矢量：

std::vector<int> numArray(bitsNeeded+1); // include <vector> for this to work

此外，请注意整数除法已经被截断，因此除非您打算稍后支持负数，否则您可以通过更改以下内容来消除有关隐式 double 到 int 转换的警告：

someNum = floor(someNum / base);

为此：

someNum /= base;

C++：添加到输出中的不需要的数字

C++: Unwanted Number Added to Output

c++

ostream

output