我应该创建一个程序,它创建一个带有常量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;
}