我的程序创建一个char数组的缩写，不打印任何东西。我的错误在哪里？

Posted 2021-03-26

我应该创建一个程序，它创建一个带有常量char数组缩写的数组。虽然我的程序没有返回任何错误，但它也不会在我的某些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”，并且size_t大小应该限制我的char数组a，以防缩写太长。所以它应该只实现字母，直到我的数组已满，然后是'\ 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）是一个大写字符或者如果它是alphabetc（isalpha(*p)和前一个字符p[-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变量跟踪，后来增加（这就是为什么我们使用postscript ++而不是前缀）。

测试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）。

另一答案

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++;
    }
}

另一答案

你想做什么可以用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;
}