Question

我是一名初学程序员，正在尝试学习C.目前，我正在上课并分配了一个项目，我设法很快完成，至少是它的主要部分。我在main（）if函数编码时遇到了一些麻烦，因为我开始使用一些新函数（即fgets和strncmp）。现在，我的代码在我的编译器中工作，但在任何在线编译器中都没有。所以我想知道我是否做错了，或者我有什么方法可以改进它。

感谢任何帮助或贡献，谢谢！

下面是代码，加密和解密函数是main之前的前两个函数，我相信大多数凌乱的快捷代码可能都是。

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

char * Encrypt(char sentence[])
{
int primes[12] = {1,2,3,5,7,11,13,17,19,23,29,31};
int x = 0;
int counter = 0;
int ispositive = 1;

 while(sentence[x] != 0)
    {
    if (counter == 0)
        {
        ispositive = 1;
        }
    else if(counter == 11)
        {
        ispositive = 0;
        }

    if (ispositive == 1)
        {
        sentence[x] = sentence[x] + primes[counter];
        counter++;
        }
    else if (ispositive == 0)
        {
        sentence[x] = sentence[x] + primes[counter];
        counter--;
        }
    x++;
    }

    return sentence;
}

char * Decrypt(char sentence[])
{
int primes[12] = {1,2,3,5,7,11,13,17,19,23,29,31};
int x = 0;
int counter = 0;
int ispositive = 1;

 while(sentence[x] != 0)
    {
    if (counter == 0)
        {
        ispositive = 1;
        }
    else if(counter == 11)
        {
        ispositive = 0;
        }

    if (ispositive == 1)
        {
        sentence[x] = sentence[x] - primes[counter];
        counter++;
        }
    else if (ispositive == 0)
        {
        sentence[x] = sentence[x] - primes[counter];
        counter--;
        }
    x++;
    }

    return sentence;
}

int main()
    {
    char message[100];
    char input[7];
    char *p;
    int c;
    int condition = 1;

    while(condition == 1)
    {

    printf("Would you like to Encrypt or Decrypt a message? (Type TurnOff to end the program) \n \n");

    fgets(input,7, stdin);

    fflush(stdin);

        if (!strncmp(input,"Encrypt",strlen(input)))
            {

            printf("\n \n Enter the message you want to Encrypt below: \n \n");

            fgets(message, 100, stdin);

            Encrypt(message);

            printf("\n Your encrypted message is: ");

            printf("%s", message);

            fflush(stdin);

            printf("\n \n");

            }
         else if (!strncmp(input,"Decrypt",strlen(input)))
         {
            printf("\n \n Enter the message you want to Decrypt below: \n \n");

            fgets(message, 100, stdin);

            Decrypt(message);

            printf("\n Your Decrypted message is: ");

            printf("%s", message);

            fflush(stdin);

            printf("\n \n");
         }
         else if (!strncmp(input,"TurnOff",strlen(input)))
         {
            printf("\n \n Thank you for using the program! \n \n");
            condition = 0;
         }
         else
         {
             printf("That's not a valid input \n \n");
         }
    }


    }

Answer 1

在printf之后你做fflush（stdin）而不是你必须做fflush（stdout）。因为您正在打印输出。输出以stdout打印。因此，您必须刷新stdout缓冲区而不是stdin缓冲区。

您可以使用strcmp而不是strncmp。因为在这里你要比较输入数组中的孔字符。所以，strcmp就足够了。

strcmp（输入，“加密”）。

strcmp或strncmp函数将数组中的输入获取为null或您声明的字符串的大小。

输入数组的大小太少。

让我们输入如下所示。

加密\ n sureshkumar \ n

在这里，你首先在主函数fgets中读取“加密”，它不会跳过'\ n'。

'\ n'表示另一个fgets。因此，它没有获得加密消息“sureshkumar”。

因此，您必须修改代码。您将增加输入数组的大小。

并检查下面的情况。

if(strcmp(input, "Encrypt\n") == 0)
{
/*
 You will do what you want
*/
}

您可以使用上述方法，或者您可以读取输入并在输入数组中覆盖'\ n'到'\ 0'，并在完成之前进行比较。但你必须使用strcmp。因为数组大小增加了。

这是使用fgets的正确方法。使用fgets是读取新行。

您必须为字符数组使用null字符。因为这对于字符数组是必要的。

Answer 2

你主动使用strcmp（）和fgets（）是好的，但需要遵循以下理解：
1. fgets（）将大小为1的字符写入缓冲区，然后以'\ 0'结尾。在你的情况下，

fgets(input,7, stdin);

您输入了“加密”/“解密”/“TurnOff”
但
'input'缓冲区将数据设为“加密”/“解密”/“转动” 因为size = 7（只读取（7-1）= 6个字符，fgets（）为'\ 0'字符保留的最后位置。）

你的strncmp（）调用可以正常使用你当前的代码，因为对于strncmp（），要比较的长度

n = strlen（输入）= 6;

在“加密”/“解密”/“TurnOff”的所有三种情况下，6个字符匹配正常。

总结是您当前的代码可以正常工作，但您的实际意图是违反的。您实际上想要读取并比较选项字符串的全长。

编辑完成：修改建议：

#define SIZE 9   <-- EDIT : Change done here, instead of 7, size = 9 is used
                            to allow reading '\n' so that it does not affect
                            fgets() read in successive iteration
char input[SIZE];

fgets(input, SIZE, stdin); // read str is e.g. "Encrypt\n"
input[SIZE-2] = '\0'; // To replace '\n' with '\0'

同样，在使用fgets（）读取'message'数组时需要小心。

我是否以正确的方式使用strncmp和fgets？

2 个答案: