时间戳

首页 > 解决方案 > 将 sprintf 更改为字符数组 C++

c++ - 将 sprintf 更改为字符数组 C++

问题描述

我有一个二维单词数组。

void main() {
    const int rowsCount = 2;
    const int colsCount = 5;

    char*** szData = new char** [rowsCount];

    //Allocate memory for each row
    for (int i = 0; i < rowsCount; i++)
    {
        szData[i] = new char* [colsCount]; //how many words in every row
        for (int j = 0; j < colsCount; j++)
        {
            szData[i][j] = new char[15]; //maximum symbols in a word
        }
    }

    //Assign some data
    for (int i = 0; i < rowsCount; i++)
    {
        char s[] = "Williams";
        szData[i][0] = s;

        sprintf(szData[i][0], "Williams%d", 0);
        sprintf(szData[i][1], "J.%d", 0);
        sprintf(szData[i][2], "#3%d", 0);
        sprintf(szData[i][3], "100%d", 0);
        sprintf(szData[i][4], "01.13%d", 0);
    }
...
}

在分配数据时，我试图改变这个

sprintf(szData[i][0], "Williams%d");

对此

char s[] = "Williams";
szData[i][0] = s;

为什么我不断收到我的项目“已触发断点”的消息？

有没有sprintf使用字符数组的替代方法？不能使用string（此任务的条件之一）

另外，不是那么必要，但是如果我在这里最后删除 0

sprintf(szData[i][0], "Williams%d", 0);
sprintf(szData[i][1], "J.%d", 0);
sprintf(szData[i][2], "#3%d", 0);
sprintf(szData[i][3], "100%d", 0);
sprintf(szData[i][4], "01.13%d", 0);

每个单词后都会出现一些数字。例如：Williams3937516。这是为什么？

完整代码：

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include <iomanip>
#include <conio.h> 

using namespace std;

void main() {
    const int rowsCount = 2;
    const int colsCount = 5;

    char*** szData = new char** [rowsCount];

    //Allocate memory for each row
    for (int i = 0; i < rowsCount; i++)
    {
        szData[i] = new char* [colsCount]; //how many words in every row
        for (int j = 0; j < colsCount; j++)
        {
            szData[i][j] = new char[15]; //maximum symbols in a word
        }
    }

    //Assign some data
    for (int i = 0; i < rowsCount; i++)
    {
        char s[] = "Williams";
        szData[i][0] = s;

        sprintf(szData[i][0], "Williams%d");
        sprintf(szData[i][1], "J.%d", 0);
        sprintf(szData[i][2], "#3%d", 0);
        sprintf(szData[i][3], "100%d", 0);
        sprintf(szData[i][4], "01.13%d", 0);
    }

    //print all the elements
    for (int i = 0; i < rowsCount; i++)
    {

        for (int j = 0; j < colsCount; j++)
        {
            cout << szData[i][j] << " ";
        }
        cout << endl;

    }

    //free memory here
    for (int i = 0; i < rowsCount; i++)
    {
        for (int j = 0; j < colsCount; j++)
        {
            delete[] szData[i][j];
        }
    }

    for (int i = 0; i < rowsCount; i++)
    {
        delete[] szData[i];
    }

    delete[] szData;
}

标签： c++arrayschar

解决方案

如果追溯此指针的分配：

char s[] = "Williams";

你会发现你正试图在这里释放它的内存：

delete[] szData[i][j];

但是，您不能这样做——该字符串"Williams"从未使用 分配new，因此无法使用 释放它delete。事实上，它是静态分配的，在你的二进制文件中。所以堆抱怨，这就是断点的原因。

如果您delete打算szData使用new.

第二个问题 - 当你打电话时：

sprintf(szData[i][0], "Williams%d", 0);

您正在覆盖您提供的内存的边界。请记住，您的缓冲区只有"Williams"字节数（加一）！您没有空间在结尾处添加整数的字符串表示形式。使用sprintf_s并确保您的缓冲区有足够的空间！

第三个问题 - 写入静态分配的字符串数据非常粗略，可能并非在所有情况下都有效。

底线：为所有内容动态分配正确大小的缓冲区，您的代码将正常工作。

这是一个修复该部分代码的简单示例。它演示了修复代码的概念，但它并不真正适合您的代码库，您需要正确使用它。

const size_t MAX_NAME_LENGTH = 15; // Fix this magic number to be whatever max name length you want
char *s = new char[MAX_NAME_LENGTH];  // allocate on heap
strcpy_s(s, MAX_NAME_LENGTH, "Williams");  // use strcpy_s for safety
     
szData[i][0] = s;
sprintf_s(szData[i][0], MAX_NAME_LENGTH, "Williams%d", 0);  // use sprintf_s for safety

推荐阅读