1

我现在正在编写一个程序来改变一个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 已启动断点

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

4

2 回答 2

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;
}
于 2013-11-10T18:32:22.327 回答
0 
_CRT_SECURE_STRCPY(pointers[i], ASIZE - 1, input.c_str());
current += input.length();

ASIZE-=input.length();

ASIZE-=input.length();

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

您的代码中也有很多错误，但这是另一回事。只需考虑使用向量，unique_ptr。

- -编辑 - -

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

_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';
}

我的输出是：

ABCDEFGHIJ

（非专有字符）

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

于 2013-11-10T19:18:35.483 回答