1

我在这里看到一篇较旧的帖子，询问如何做相对相同的事情，但他们的方法不同，我很想知道我的程序中的漏洞。

我正在尝试编写一个程序，将字符接收到 10 个字符长度的数组中。我希望程序评估第一个数组位置并删除它稍后在数组中找到的任何重复项，方法是识别重复项并将其右侧的所有值向左移动一个。然后数组的“大小”减一。

我相信我用于删除函数的逻辑是正确的，但程序只为数组中的第一个值和第四个值打印一个“a”。

任何帮助将不胜感激，这是我的代码：

#include <iostream>
using namespace std;

int letter_entry_print(int size, char array[10]);
int delete_repeats(int& size, char array[10]);
int final_array_print(int size, char array[10]);

int main()
{
    char array[10];
    int size = 10;

    letter_entry_print(size,array);
    delete_repeats(size,array);
    final_array_print(size,array);

    cout<<"\n";
    system("pause");
}

int letter_entry_print(int size, char array[10])
{
    int i;

    for (i=0;i<size;i++)
    {
        cout << "Enter letter #" << i+1 << endl;
        cin >> array[i];
        cout << "\n";
    }

    cout << "\nYour array index is as follows:\n\n";

    for (i=0;i<size;i++)
    {
        cout << array[i];
        cout << " ";
    }

    cout <<"\n\n";
    return 0;
}

int delete_repeats(int& size, char array[10])
{
    int ans;
    int loc;
    int search;
    int replace;
    char target='a';

    cout << "Enter 1 to delete repeats.\n\n";
    cin >> ans;
    if(ans==1)
    {
        for(loc=0;loc<size;loc++)
        {
            array[loc]=target;
            for(search=1;search<(size-loc);search++)
            {
                if(target=array[loc+search])
                {
                    for(replace=0;replace<(size-(loc+search));replace++)
                    {
                        array[loc+search+replace]=array[loc+search+replace+1];
                        array[size-1]=0;
                        size=(size-1);
                    }
                }
            }
        }
    }else(cout<<"\nWhy didn't you press 1?\n\n");
    return 0;
}

int final_array_print(int size, char array[10])
{
    cout<<"\nYour new index is as follows:\n\n";
    int i;
    for(i=0;i<size;i++)
    {
        cout<<array[i];
        cout<<" ";
    }

    cout<<"\n";
    return 0;
}
4

1 回答 1

1

好的，您的代码有些地方看起来很奇怪。

1）你在整个地方重复 10 到你无法合理改变它的地步，但你也会传递大小。与其让所有函数都采用 10 个字符的数组，不如考虑只传递一个指向 char 的指针，例如：

int final_array_print(int size, char *array)

那么您可以更轻松地更改数组的大小。如果您要将自己永远限制为 10 个项目，那么到处传递大小是没有意义的，如果您提供一个大小，就没有充分的理由传递 10 个项目的数组！

2) 好的，所以现在您要查找重复项。为什么用'a'覆盖数组中的第一个元素？

 char target='a';
...
array[loc]=target;

你不想反过来做吗？

3）接下来，正如@Mahesh 指出的那样，您可能希望在查找重复项时使用比较运算符'=='而不是赋值运算符=：

if(target=array[loc+search])

应该是

if(target == array[loc+search])

4) 接下来，不要害怕在你的单词和标点符号之间使用很小的空格。这样更容易识别打字错误和拼写错误。

5）您实际执行替换的循环具有非常复杂的索引。如果您不从 replace = 0 开始，而是从 replace = search + 1 开始，尝试一下，也许您会发现所有其余索引都变得简单得多。

于 2012-09-08T05:54:19.803 回答