包括

Question

#include <iostream>
#include <stdlib.h>
#include <string>
#include<string.h>
#include <stdio.h>

using namespace std;

void OpCode()
{
    string mnemonic;
    int hex;
    char *op;

    cout << "Entre mnemonic : ";
    cin >> mnemonic;

    char *str1 = strdup(mnemonic.c_str());

    if(strcmp(str1, "ADD") == 0)
    {
        hex = 24;
        itoa(hex,op,16);
        cout << op;
        cout << "\nEqual";
    }
    else
    cout << "\nFalse";
}

int main()
{
    OpCode();
    return 0;
}

它运行到我使用op变量的部分，我尝试在主函数中复制和粘贴它完美地工作，为什么它不能在OpCode函数中工作？！提前致谢

Answer 1

itoa写入第二个参数指向的内存。它不会分配内存本身。这意味着由你来传递一个有效的内存指针。你不是;你从不分配任何记忆。它主要靠运气而不是设计。

一种简单的方法是替换您将op定义为char op[9];的行，但请记住这是本地分配的内存，因此您无法从函数中返回它。

Answer 2

以下是带注释的修复

包括

#include <stdlib.h>
#include <string>
#include<string.h>
#include <stdio.h>

using namespace std;

void OpCode()
{
    string mnemonic;
    int hex;
    char op[10];  // allocate a pointer op that points to 10 empty spaces of type char.

    cout << "Entre mnemonic : ";
    cin >> mnemonic;

    char *str1 = strdup(mnemonic.c_str());

    if(strcmp(str1, "ADD") == 0)
    {
        hex = 24;
        itoa(hex,op,16);   // convert hex to an ASCII representation and write the ASCII to the 10 empty spaces we allocated earlier.
        cout << op;
        cout << "\nEqual";
    }
    else
    cout << "\nFalse";
    free (str1); // free the memory that was allocated using strdup so you do not leak memory!
}

int main()
{
    OpCode();
    return 0;
}

有人可以告诉我这段代码有什么问题吗？

2 个答案:

包括