凯撒密码中指向 integer 转换的不兼容指针

Question

我正在尝试创建一个“凯撒密码”。 我希望我的代码终于完成了，但是在尝试运行代码时遇到了一些错误。 我是 C 的完整初学者，所以我怀疑我只是没有正确调用 function但我的代码可能会有更多问题。 这里是：

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

bool only_digits(string s);
char rotate(char c, int n);

int main(int argc, string argv[])
{
    if ((argc == 2) && only_digits(argv[1]) == true)
    {
        //convert string from argv[1] into int
        int n = (atoi(argv[1]));
        string plaintext = get_string("plaintext: ");
        rotate(plaintext,n);
        printf("ciphertext: %s \n", plaintext)
    }
    else
    {
        printf("usage: ./caesar key\n");
    }
}

bool only_digits(string s)
{
    size_t n = strlen(s);
    for (size_t i = 0; i < n; i++)
    {
        if (!isdigit(s[i]))
        {
            return false;
        }
    }

    return true;
}

char rotate(char c, int n)
{
    if (c >= 65 && c <= 90)
    //establish capital letter
    {
        while (n>0)
        {
            (c++ && n--);
            if (c > 90)
            {
                (c ==65)
            }
        }
        return c;
    }
    if (c >= 97 && c <= 122)
    //establish lowercase
    {
        while (n>0)
        {
            (c++ && n--);
            if (c > 122)
            {
                (c == 97)
            }
        }
        return c;
    }
    if (c >= 48 && c <= 57)
    //establish number
    {
        while (n>0)
        {
            (c++ && n--);
            if (c > 57)
            {
                (c == 48)
            }
        }
        return c;
    }
    else
    {
        if (n=0)
        {
            continue
        }
    }
}

我遇到了多个错误，其中一些我已经解决了。 这是最新的错误：

caesar.c:17:16: error: incompatible pointer to integer conversion passing 'string' (aka 'char *') to parameter of type 'char'; dereference with * [-Werror,-Wint-conversion]
        rotate(plaintext,n);
               ^~~~~~~~~
               *
caesar.c:8:18: note: passing argument to parameter 'c' here
char rotate(char c, int n);
                 ^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
2 errors generated.
make: *** [<builtin>: caesar] Error 1

Answer 1

这个错误是不言自明的。 rotate function 需要一个char但您提供的string只是char *的 typedef。

鉴于它的使用方式，我怀疑您想使用循环rotate输入字符串中的每个字符。

其他一些注意事项：

char是数字类型。 例如，您可以使用字符文字'a' ，而不是在代码中使用97 。

通常的约定是预先设置任何会导致程序终止并返回非零退出代码的条件。

int main(int argc, string argv[])
{
    if (argc != 2 && !only_digits(argv[1]))
    {
        printf("usage: ./caesar key\n");
        return 1;
    }

    // convert string from argv[1] into int
    int n = (atoi(argv[1]));
    string plaintext = get_string("plaintext: ");
    rotate(plaintext,n);
    printf("ciphertext: %s \n", plaintext);

    return 0;
}

Answer 2

function rotate将单个字符作为参数并返回相应的旋转字符。

然而，在行

rotate(plaintext,n);

您试图传递整个字符串（即几个字符）而不是单个字符。 你不能这样做。

如果要旋转字符串中的所有字符，则必须使用循环为每个字符调用 function rotate一次：

printf( "ciphertext: " );

int len = strlen( plaintext );

for ( int i = 0; i < len; i++ )
{
    char c;

    c = rotate( plaintext[i], n );

    printf( "%c", c );
}

printf( "\n" );