This is in C language. When I call rotate() in main, the function returns false for isalpha() even though the string entered for plaintext uses alphabetic characters. Perhaps it’s identifying an alphabetic character by its ASCII value (‘A’ = 65)? I tried to test that out and used (char) with the letter variable in rotate() but it didn’t change anything.

PORTION OF MAIN

string plaintext = get_string("plaintext:  ");

    int length = strlen(plaintext);
    char ciphertext[length];

    for (int i = 0; i < length; i++)
    {
        ciphertext[i] = rotate(plaintext[i], key);
    }

ROTATE FUNCTION

char rotate(char letter, int key)
{
    if (isalpha(letter) == true)
    { ...
  • jjagaimo@lemmy.caEnglish
    6·
    1 year ago

    isalpha documentation:

    Return value

Non-zero value if the character is an alphabetic character, zero otherwise.

    You should be either checking for not equal to 0 instead of true, as its not necessarily guaranteed to be 1 ~= true, or removing the comparison entirely

    Also make sure that your loop condition is < and not “& lt” without the space unless that’s a weird formatting issue

    For more information, make sure to check the documentation for the standard library functions

    • lolcatnip@reddthat.comEnglish
      3·
      1 year ago

      Never, ever write “== true” or “== false” unless it’s absolutely necessary. In C that’s pretty close to never.