0

所以这里的练习是设计一个程序，它接受一个字符串并删除该字符串中出现在第二个字符串中的所有字符。因此，对于我在下面选择的第一个字符串是 abc 而第二个字符串是 cde 的字符串，我希望得到一个输出ab而不是abc.

我已经看到了一种非常巧妙的方法来执行此挤压功能，它只需要两个简单的 for 循环，但我想知道为什么我的冗长方法不起作用。

#include<stdio.h>

void squeeze(char s1[], char s2[]);
void copy(char to[], char from[]);

int k=0;

main()
{
    char array1[4]="abc";
    char array2[4]="cde";
    squeeze(array1, array2);
    printf("%s", array1);
}

void squeeze(char s1[], char s2[])
{
    int j,i,m;
    m=j=i=0;
    char s3[1000];
    while(s1[i]!='\0')   //What I'm doing here is taking a character in the string
    {                      //and comparing it with all characters in the second string.
        copy(s3,s1);       //If it exists in the second string I'm 'deleting' that letter
        while(s2[j]!='\0') //from the first string. Then I start all over again with the
        {                 // next letter in line. Or at least that's the plan.
            if (s1[i]==s2[j])
            {
                k=1;
            }
            ++j;
        }

        if (k==1)
        {
            m=i;
            while(s3[m+1]!='\0')
            {
                s1[m]=s3[m+1];  //I'm 'deleting' the letter by pushing each character
                ++m;            //that is to the right of the deleted character to the  
            }                   //left of the array.
        }

        if(k!=1)
        {
            ++i;
        }
    }
    s1[i]='\0';
 }

void copy(char to[], char from[])
{
    int i;
    i=0;

    while(from[i]!='\0')
    {
        to[i]= from[i];
        ++i;
    }
    to[i]=='\0';
}
4

2 回答 2

2

在您的外部内部，而您应该将“j”重置为零。

如果“k”变为 1，则不再增加“i”。如果你第二次运行squeeze()，你不会再次初始化“k”。

永远不要使用像“k”这样的全局变量（或模块局部变量）。这使您的代码线程不安全。

于 2013-07-21T17:41:17.267 回答
1

高五——我也在读那一章。我想我已经读了大约4遍了，反正我真的需要边做边学，几个小时内我就忘记了所有内容。这就是为什么我几乎完成了该章中的所有练习——接下来是第 3 章！

这是我的解决方案 - 挤压功能与 getline 功能不兼容（/0'强制printf不打印到标准输出。

编译gcc 4.7.2_

gcc -Wall -std=c99 -pedantic squeeze.c

#include <stdio.h>
#define LIM 100

void squeeze(char[], char[]);
int ggetline(char[], int);

int main()
{
    //the getline function is buggy; it would produce null strings.
    char line1[LIM] = "hello";
    char line2[LIM] = "hello2";
    squeeze(line1, line2);
    printf("%s %s\n", line1, line2);

return 0;

}


/* getline: reads line into s, returns length */
int ggetline(char s[], int lim)
{
    int c, i;

    for (i = 0; i < lim-1 && (c = getchar()) != EOF && c!='\n'; ++i)
        s[i] = c;
    if (c == '\n') {
        s[i] = c;
        ++i;
    }
    s[i] = '\0';
    return i;
}

void squeeze(char s1[], char s2[])
{
    int j, l, i, k;

    for (i = 0; s2[i] != '\0'; i++) {
        for (k = l = 0; s2[j] != '\0'; j++) {
            if (s1[j] != s2[i])
                s1[k++] = s1[j];
        }
        s2[k] = '\0';

    }
}

祝你好运！

于 2013-07-21T20:43:31.607 回答