-1

暗号は islower 部分では機能しますが、isupper 部分では機能しません。たとえば、キー 3 をI like pie!!指定して暗号化を入力すると、O olnh slh!!私も試しHELLOてみてNKRRU. isupper 部分は、文字だけでなく句読点も返します。また、元のメッセージが暗号メッセージと一致するように変更されている理由もわかりません。

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

int main (int argc, string argv[])
{
    /*
    Get key from user at command line
    Get plaintext from user
    Use key to encipher text: c[i] = (p[i] + k)%26
    Print ciphered message
    */        
    string message, cipher;
    int key;

    // command line, if user doesn't enter 2 arguments return 1 and request a valid 
    //encryption key and rerun.
    if (argc != 2)
        {
        printf("Please enter a valid encryption key and rerun program.\n");
        return 1;
        }
    else
        {
        key = atoi(argv[1]);
        }


    printf("Enter the message you wish to encrypt.\n");
    message = GetString();
    cipher = message;
    int length = strlen(message);

    for ( int i = 0; i < length; i++)
        {
        if (isalpha(message[i]))
            {
            if (isupper(message[i]))
                {
                cipher[i] = (message[i] - 'A' + key) % 26 + 'A';
                }
            else (islower(message[i]));
                {
                cipher[i] = (message[i] - 'a' + key) % 26 + 'a';
                }
            }
        else continue; //message[i] contains punctuation or a space
        } 
         printf("Your original message was..\n");
         printf("%s\n", message);
         printf("The encrypted message is...\n");
         printf("%s\n", cipher);         
         return 0;            
}
4

2 に答える 2

3

if@interjay によるタイプミスと欠落。

変化する

else (islower(message[i]));

に

//                           v
else if (islower(message[i]))
// or simply 
else  // Since `message[]` is an alpha, but not upper

エラーで、テキストが大文字の場合、cipher[i] = (message[i] - 'A' ...との両方cipher[i] = (message[i] - 'a' ...が発生しました。を考えるcipher = messageと、暗号は 2 回適用されました。

不足しているバッファに関する@keshlamのポイントは重大な問題です。でもどんなタイプか気になりstringます。これはある種の C++ ライト文字列ですか? の場合char *、コードはcipher = strdup(message);またはを使用できます

cipher = malloc(length + 1);
if (cipher === NULL) Handle_OutOfMemeory();
cipher[length] = '\0';
for ( int i = 0; i < length; i++) 
  ...
于 2014-02-09T19:24:27.443 に答える
2

cipher = message; と言ったので、メッセージを上書きしています。これは、両方が同じメモリ ブロックを指していることを意味します。新しい出力バッファを割り当てます。

そして、余分なセミコロンを見つけた Chux に 2 つのポイントがあります。

于 2014-02-09T19:24:52.553 に答える