在检查和压缩句子中的最后三个单词时,对任务感到困惑

时间:2017-11-12 19:22:31

标签: c

我有一个问题,我无法修复我需要用第四句的最后三个单词检查第三个单词的句子

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


int main() {

char firstRow[256];
char secondRow[256];
char thirdRow[256];
char fourthRow[256];

printf("Enter four row of lyrcis:\n");
gets(firstRow);
gets(secondRow);
gets(thirdRow);
gets(fourthRow);

if ( strcmp(a1+strlen(a1)-1, a4+strlen(a4)-1) &&
     strcmp(a1+strlen(a1)-2, a4+strlen(a4)-2) &&
     strcmp(a1+strlen(a1)-3, a4+strlen(a4)-3)  == 0 ){

    printf("Good job last three words of first and fourth sentence are same");

}
else {

    printf("nothing");

}

return 0;
}

这是我尝试的东西,但显然问题是我不能使用如果只有一个strcmp它的工作原理。也许我需要strcpy命令?救命啊!

1 个答案:

答案 0 :(得分:0)

首先 - 请勿使用&#39; 。这是非常不安全的。它将读取的字符数没有限制,或者您提供的缓冲区大小是否有足够的存储空间。这允许缓冲区溢出漏洞,并且是从C库中删除的主要原因。如果你的教授坚持使用它 - 找一位新教授。

您遇到的另一个问题是无法验证流程中的每个步骤。在将指针传递给getsstrcmp之前,您无法检查strlen是否实际读取了任何内容。

此外,您的索引是无稽之谈。 strlen(x) - n不会对缓冲区中的end - n字进行索引。为此,您必须对字符串进行标记(将其拆分为单词)。有很多方法可以做到这一点。

一种方法无论是简单地找到字符串的结尾(例如strlen(line) - 1)并使用指针从字符串的末尾向开始迭代直到找到第一个空格(或者你到达开头)。

C库(在string.h中)提供strrchr,为您自动执行该过程。它将从字符串的末尾开始并向后迭代,直到它找到第一次出现的字符,告诉它找到返回指向该字符的指针(或者返回NULL该字符未找到)。这里唯一的缺点是你只能搜索一个角色。

C库(在string.h中)提供strtok,它不提供反向搜索,但提供了根据一组分隔符分割字符串的功能space,tab,&#39;。&#39;等中的任何一个。)。在这里,您只需存储指向每个单词(或单词的副本)的指针,并将最后3个索引进行比较。

以下提供了一个示例,该示例使用strrchr假定您的单词或由一个(或多个)spaces分隔。以下使用的方法和strtok 修改原始字符串,因此如果字符串最初存储在只读内存中(例如字符串),请在解析之前复制字符串字面)。

程序期望读取的文件名作为第一个参数提供(如果没有提供参数,它将从stdin读取)。代码中提供了附加注释,内嵌在下面:

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

#ifndef BUF_SIZ         /* if you need constants... define them */
#define BUF_SIZ 8192    /* don't put 'magic' numbers in code   */
#endif

#define NLAST 3

int main (int argc, char **argv) {

    size_t len = 0;
    char line1[BUF_SIZ] = "",
         line4[BUF_SIZ] = "",
         *last[NLAST] = { NULL };
    FILE *fp = argc > 1 ? fopen (argv[1], "r") : stdin;

    if (!fp) {  /* validate file open for reading */
        fprintf (stderr, "error: file open failed '%s'.\n", argv[1]);
        return 1;
    }

    if (fgets (line1, BUF_SIZ, fp) == NULL) {   /* read 1st line */
        fprintf (stderr, "error: failed to read line1.\n");
        return 1;
    }

    for (int i = 0; i < NLAST; i++) /* read/discard lines 2,3 read line 4 */
        if (fgets (line4, BUF_SIZ, fp) == NULL) {
            fprintf (stderr, "error: failed to read line4.\n");
            return 1;
        }

    if (fp != stdin) fclose (fp);       /* close file if not stdin */

    len = strlen (line1);               /* get length of line1 */
    if (len && line1[len-1] == '\n')    /* validate last is '\n' */
        line1[--len] = 0;               /* overwrite with nul-character */
    else {              /* error: handle line too long or no POSIX EOL */
        fprintf (stderr, "error: line1 too long or no POSIX EOL.\n");
        return 1;
    }

    len = strlen (line4);               /* same thing for line4 */
    if (len && line4[len-1] == '\n')
        line4[--len] = 0;
    else {
        fprintf (stderr, "error: line4 too long or no POSIX EOL.\n");
        return 1;
    }

    if (!*line1 || !*line4) {   /* test if either is empty-string */
        fprintf (stderr, "error: one or both line(s) empty.\n");
        return 1;
    }

    for (int i = 0; i < NLAST; i++) {       /* loop NLAST times */
        char *p1 = strrchr (line1, ' '),    /* get pointer to last ' ' */
             *p4 = strrchr (line4, ' ');

        if (!p1) {                  /* validate result of strrchr */
            if (i < NLAST - 1) {    /* if not last iteration - handle error */
                fprintf (stderr, "error: only '%d' words in line1.\n", i+1);
                return 1;
            }
            else    /* if last iteration, assign line to pointer */
                p1 = line1;
        }
        if (!p4) {                  /* same for line4 */
            if (i < NLAST - 1) {
                fprintf (stderr, "error: only '%d' words in line4.\n", i+1);
                return 1;
            }
            else
                p4 = line1;
        }
        /* copy to last array in order - checking if p1 is beginning of line */
        last[NLAST - 1 - i] = p1 == line1 ? p1 : p1 + 1;   

        while (p1 > line1 && *p1 == ' ')    /* nul-terminate at space */
            *p1-- = 0;
        while (p4 > line4 && *p4 == ' ')
            *p4-- = 0;
    }

    printf ("\nthe last %d words in lines 1 & 4 are the same:\n", NLAST);
    for (int i = 0; i < NLAST; i++)
        printf ("  %s\n", last[i]);

    return 0;
}

示例输入文件

$ cat dat/last14-3.txt
My dog has fleas
My snake has none
The cats have none
Cats are lucky because the dog has fleas

示例使用/输出

$ ./bin/lines1_4_last3 < dat/last14-3.txt

the last 3 words in lines 1 & 4 are the same:
  dog
  has
  fleas

无论您选择哪种方法来标记线条,都必须验证整个过程中的每个步骤。仔细研究一下,确保你理解为什么每次验证都是必要的,如果没有,只要问一下,我很乐意进一步提供帮助。