Question

我这里有问题。这个功能：

BOOL WINAPI SFileGetFileName(HANDLE hFile, char * szFileName) {
    TMPQFile * hf = (TMPQFile *)hFile;  // MPQ File handle
    char *szExt = "xxx";                // Default extension
    DWORD dwFirstBytes[2];              // The first 4 bytes of the file
    DWORD dwFilePos;                    // Saved file position

    int nError = ERROR_SUCCESS;
    int i;

    // Pre-zero the output buffer
    if(szFileName != NULL)
        *szFileName = 0;

    // Check valid parameters
    if(nError == ERROR_SUCCESS)
    {
        if(hf == NULL || szFileName == NULL)

            nError = ERROR_INVALID_PARAMETER;

    }



    // If the file name is already filled, return it.

    if(nError == ERROR_SUCCESS && *hf->szFileName != 0)

    {

        if(szFileName != hf->szFileName)

            strcpy(szFileName, hf->szFileName);

        return TRUE;

    }



    if(nError == ERROR_SUCCESS)

    {

        if(hf->dwBlockIndex == (DWORD)-1)

            nError = ERROR_CAN_NOT_COMPLETE;

    }



    // Read the first 8 bytes from the file

    if(nError == ERROR_SUCCESS)

    {

        dwFirstBytes[0] = dwFirstBytes[1] = 0;

        dwFilePos = SFileSetFilePointer(hf, 0, NULL, FILE_CURRENT);   

        SFileReadFile(hFile, &dwFirstBytes, sizeof(dwFirstBytes), NULL);

        BSWAP_ARRAY32_UNSIGNED(dwFirstBytes, sizeof(dwFirstBytes) / sizeof(DWORD));

        SFileSetFilePointer(hf, dwFilePos, NULL, FILE_BEGIN);

    }



    if(nError == ERROR_SUCCESS)

    {

        if((dwFirstBytes[0] & 0x0000FFFF) == ID_EXE)

            szExt = "exe";

        else if(dwFirstBytes[0] == 0x00000006 && dwFirstBytes[1] == 0x00000001)

            szExt = "dc6";

        else

        {

            for(i = 0; id2ext[i].szExt != NULL; i++)

            {

                if(id2ext[i].dwID == dwFirstBytes[0])

                {

                    szExt = id2ext[i].szExt;

                    break;

                }

            }

        }



        // Create the file name

        sprintf(hf->szFileName, "File%08lu.%s", hf->dwBlockIndex, szExt);

        if(szFileName != hf->szFileName)

            strcpy(szFileName, hf->szFileName);

    }

    return (nError == ERROR_SUCCESS);

}

在＆＃39; make＆＃39;：

上给我这些错误

SFileReadFile.cpp: In function ‘bool SFileGetFileName(HANDLE, char*)’:
SFileReadFile.cpp:655:19: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
 char *szExt = "xxx";                // Default extension
               ^
SFileReadFile.cpp:700:19: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
         szExt = "exe";
               ^
SFileReadFile.cpp:702:19: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
         szExt = "dc6";
               ^
SFileReadFile.cpp:716:72: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘DWORD {aka unsigned int}’ [-Wformat=]
     sprintf(hf->szFileName, "File%08lu.%s", hf->dwBlockIndex, szExt);

请给我提示，如何解决这些问题？

我在C ++文档中做了很多尝试，但我找不到需要的东西。看，我不能在const char const*的声明中做szext因为我得到很多关于常量的错误。但我真的想摆脱这些错误。

请给我一些建议或更深入解释它为何会发生。

Answer 1

这些不是您收到警告消息的错误，您的程序将正常工作。

在c ++ 11中，使用您正在使用的方式初始化字符串。

请参阅其他答案，或者您可以使用std::string str="xxx";

但您需要包含标题

Answer 2

C ++中的字符串文字具有 const character 数组的类型。因此，如果要将字符串文字分配给字符指针，则必须使用指向const char的指针。

所以编写

会更正确

const char *szExt = "xxx";

或者，如果您不想使用限定符const

，则可以使用字符数组而不是指针

char szExt[] = "xxx";

考虑到任何修改字符串文字的尝试都会导致未定义的行为。

功能与C ++ 11规范不兼容

2 个答案: