Question

我正在编写一个程序，它使用指向字符串的指针改变C_String。我有一个正常的实现。我遇到的唯一问题是，当我到达程序结束时，如果我尝试删除指针，我会收到错误。

我的代码：

void CStringSwitcher()
{

    string input = "";
    char* cStringArray = new char[ASIZE];
    char* reversed = new char[ASIZE];
    const char* originalReversed = reversed;
    char* pointers[POINTER_SIZE];
    memset(reversed, '\0', ASIZE);
    memset(cStringArray, '\0', ASIZE);
    memset(pointers, '\0', POINTER_SIZE);
    char* current = cStringArray;
    cout << "Enter a sentence on each line. Input a 0 to stop." << endl;

    // Receives input from the user and stores it into cStringArray
    int i = 0;
    do
    {
        cout << ">";
        cin.clear();
        fflush(stdin);
        input = "";
        getline(cin, input);
        if (input == "0")
            break;
        else
        {
            input.append("\n");
            pointers[i] = current;
            _CRT_SECURE_STRCPY(pointers[i], ASIZE - 1, input.c_str());
            current += input.length();
            i++;
        }
    } while(i < POINTER_SIZE);
    char* end = current;

    --i;
    do
    {
        /// Check if done
        if(i < 0)
            break;
        /// Copy from current to end
        current = pointers[i];
        do
        {

            *reversed++ = *current++;
        }while(current < end);
        /// Update end
        end = pointers[i];
        /// Update i
        --i;
    }while(true);
    *reversed = '\0';
    cout << endl << originalReversed << endl;
    system("PAUSE");
    //delete[] originalReversed;
    //delete[] cStringArray;
    return;
}

如上所述，代码工作正常，但是如果我在返回之前取消注释两个删除行，则会出现错误：

Project_06.exe已启动断点

程序崩溃了。奇怪的是我只是再次运行该程序以获得错误消息的确切措辞，它运行没有错误？关于为什么会这样的想法？

Answer 1

我猜这个代码是一个教育/练习片段，试图巩固你对指针的认识，但坦率地告诉你：这是一个绝对的恐怖阅读。

这个答案符合“教人钓鱼”的精神。

首先删除所有分配，然后使用固定大小的数组。

char cStringArray[ASIZE] = "";
char reversed[ASIZE] = "";

这暂时不需要memset，这个赋值实际上将整个数组设置为0（参见http://ideone.com/WmLtQp）。

这样做可以在通过调试器运行它时更容易发现损坏。

然后将数组切换到动态分配。

最后，不要混合使用stdin和cin，这样做可以调用未定义的行为。

----编辑----

这是一个C ++ - 重构你的代码。这篇特别的部分展示了如何手动完成（手动复制字节）和使用C ++功能来减少我们自己必须完成的工作量。

ideone现场演示：http://ideone.com/0KuGiB

#include <iostream>
#include <string>
#include <vector>

void CStringSwitcher()
{
    std::vector<std::string> inputs;
    size_t totalLength = 0;

    std::cout << "Enter a sentence on each line. Input a 0 to stop." << std::endl;
    inputs.reserve(16);

    for ( ; /* until break */ ; ) {
        std::cout << ">";
        std::string input;
        getline(std::cin, input);
        if (input == "0")
            break;
        inputs.push_back(input);
        totalLength += input.length() + 1; // for the '\n'
    }

    std::string reversed = "";
    reversed.reserve(totalLength); // eliminate allocations

    // walk backwards thru the list of strings.
    for (auto inputsIt = inputs.rbegin(); inputsIt != inputs.rend(); ++inputsIt) {
        const std::string& input = *(inputsIt);

#ifndef REAL_CODE
        // educational, Do-It-Yourself way
        const size_t length = input.length();

        // walk backwards thru the characters
        for (size_t i = 0; i < length; ++i) {
            reversed += input[length - 1 - i];
        }
#else
        // call append with reversed iterators to do it for us.
        reversed.append(input.rbegin(), input.rend());
#endif

        // add the trailing '\n'
        reversed += '\n';
    }

    std::cout << std::endl << reversed << std::endl;

    // don't pause, set a break point at the end of the function
    // or run without debugging.

    return;
}

int main(int argc, const char* argv[])
{
    CStringSwitcher();

    return 0;
}

Answer 2

_CRT_SECURE_STRCPY(pointers[i], ASIZE - 1, input.c_str());
current += input.length();

ASIZE-=input.length();

<强> ASIZE- = input.length（）;

不知道为什么它有助于摆脱错误。如果新字符串的大小＆gt;这应该只防止溢出。剩余字节的大小。看起来像是一些特定于Microsoft的魔法。

你的代码中也有很多错误，但这是另一个故事。只需考虑使用vector，unique_ptr。

--- --- EDIT

我带来了一些奇怪的东西。

_CRT_SECURE_STRCPY定义为strcpy_s

#include <iostream>
#include <numeric>

using namespace std;

int main()
{
    char arr[10];
    iota(arr, end(arr), 'A');
    for (auto i : arr) cout << i << '\t'; cout << '\n';

    strcpy_s(arr, "");

    for (auto i : arr) cout << i << '\t'; cout << '\n';
}

我的输出是：

A B C D E F G H I J

（nonpritable characters）

这意味着strcpy_s会重写整个目标缓冲区。即使你传了一个字符。

删除已声明为新的指针时出错？

2 个答案: