我的程序创建了char数组的缩写,但不打印任何内容。我的错误在哪里?

时间:2019-04-20 12:54:50

标签: c arrays pointers

我应该创建一个程序,该程序创建一个带有常量char Array缩写的数组。尽管我的程序没有返回任何错误,但它也没有在我的某些printf点上打印任何字符。因此,我假设程序无法正常运行,并且没有用任何字符填充数组。

void abbrev(const char s[], char a[], size_t size) {
    int i = 0;
    while (*s != '\0') {
        printf('%c', *s);
        if (*s != ' ' && *s - 1 == ' ') {
            a[i] = *s;
            i++;
            printf('%c', a[i]);
        }
        s++;
    }
}


void main() {
    char jordan1[60] = "  Electronic      Frontier  Foundation ";
    char a[5];
    size_t size = 5;
    abbrev(jordan1, a, size);
    system("PAUSE");
}

实际结果为空。至少我认为是这样,因为我的控制台没有显示任何内容。结果应该是“ EFF”,并且大小写应该限制我的char数组a,以防缩写太长。因此,它应该只执行字母,直到我的数组已满,然后再执行'\ 0',但由于我的程序显然根本没有填充数组,因此我尚未实现它。

3 个答案:

答案 0 :(得分:0)

#include <stdio.h>
#include <ctype.h>

/* in: the string to abbreviate
   out: output abbreviation. Function assumes there's enough room */
void abbrev(const char in[], char out[])
{
    const char *p; 
    int zbPosOut = 0;     /* current zero-based position within the `out` array */

    for (p = in; *p; ++p) { /* iterate through `in` until we see a zero terminator */

        /* if the letter is uppercase
              OR if (the letter is alphabetic AND we are not at the zero 
              position AND the previous char. is a space character) OR if the
              letter is lowercase and it is the first char. of the array... */
        if (isupper(*p) || (isalpha(*p) && (p - in) > 0 && isspace(p[-1])) 
                        || (islower(*p) && p == in)) {

            out[zbPosOut++] = *p; /* ... then the letter is the start letter
                          of a word, so add it to our `out` array, and 
                          increment the current `zbPosOut` */
        }
    }
    out[zbPosOut] = 0; /* null-terminate the out array */
}

此代码在几行中说了很多。让我们看一下:

isupper(*p) || (isalpha(*p) && (p - in) > 0 && isspace(p[-1])) 
           || (islower(*p) && p == in)

如果当前字符(*p)是大写字符,或者如果是字母(isalpha(*p)并且前一个字符p[-1]是空格,那么我们可以考虑{{1} }作为单词的第一个字符,应将其添加到我们的*p数组中,我们包括测试out,因为如果(p - in) > 0,则我们位于单词的零位置数组,因此p == in未定义。

此表达式中的顺序非常重要。如果我们将p[-1]放在(p - in) > 0测试之后,那么我们就不会利用isspace(p[-1])运算符的惰性:一旦遇到&&操作数为假,则不评估以下操作数。这很重要,因为如果使用p - in == 0,则我们不想评估isspace(p[-1])表达式。我们编写测试的顺序确保在确保我们不在零位之后评估isspace(p[-1])

最后一个表达式(islower(*p) && p == in)处理第一个字母为小写的情况。

out[zbPosOut++] = *p;

我们将字符*p附加到out数组中。 out的当前位置由zbPosOut变量跟踪,该变量随后递增(这就是为什么我们使用后记++而不是前缀的原因)。

测试abbrev操作的代码:

int main()
{
    char jordan1[] = "  electronic      frontier  foundation ";
    char out[16];

    abbrev(jordan1, out);
    puts(out);

    return 0;
}

它给出eff作为输出。为了使它看起来像首字母缩写词,我们可以更改代码以将字母*p附加到out到:

out[zbPosOut++] = toupper(*p);

大写添加到out数组中的每个字母(如果*p已经大写,toupper仅返回*p)。

答案 1 :(得分:0)

void print_without_duplicate_leading_trailing_spaces(const char *str)
{
    while(*str == ' ' && *str) str++;
    while(*str)
    {
        if(*str != ' ' || (*str == ' ' && *(str + 1) != ' '  && *str))
        {
            putchar(*str);
        }
        str++;
    }
}

答案 2 :(得分:-1)

您可以通过for()循环来简化您的操作。

#include <stdio.h>
#include <string.h>

void abbrev(const char s[], char a[], size_t size) {
  int pos = 0;
  // Loop for every character in 's'.
  for (int i = 0; i < strlen(s); i++)
    // If the character just before was a space, and this character is not a
    // space, and we are still in the size bounds (we subtract 1 for the
    // terminator), then copy and append.
    if (s[i] != ' ' && s[i - 1] == ' ' && pos < size - 1)
      a[pos++] = s[i];
  printf("%s\n", a); // Print.
}

void main() {
  char jordan1[] = " Electronic    Frontier  Foundation ";
  char a[5];
  size_t size = 5;
  abbrev(jordan1, a, size);
}

但是,我认为这不是实现您要尝试的工作的最佳方法。首先,由于检查了前一个字符,因此无法获得char s[0]。这使我想到第二个原因:在第一个索引上,您将检查s[-1]可能不是一个好主意。如果我要实现此功能,我会这样做:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void abbrev(const char s[], char a[], size_t size) {
  char *str = strdup(s); // Make local copy.
  size_t i = 0;
  // Break it up into words, then grab the first character of each word.
  for (char *w = strdup(strtok(str, " ")); w != NULL; w = strtok(NULL, " "))
    if (i < size - 1)
      a[i++] = w[0];
  free(str); // Release our local copy.
  printf("%s\n", a);
}

int main() {
  char jordan1[] = "Electronic    Frontier  Foundation ";
  char a[5];
  size_t size = 5;
  abbrev(jordan1, a, size);
  return 0;
}