0

最初の scanf() が使用され、Y と答えると、2 番目の scanf() は「オプションが選択されていません。終了しています...」に直接スキップします。このメッセージは、キーファイルがソースファイルよりも大きく、最後の scanf が適切に機能した場合にも表示されます。だから私はここで途方に暮れています、何が悪いのですか?(コードはうまくコンパイルされるので、気軽に試してみてください)

編集: 反対票を投じた人が少なくとも理由を投稿すると役に立ちます。私はあまり上手なプログラマーではなく、ここで学ぼうとしています。

#include <stdio.h>
#include <stdlib.h>
#include <sys/stat.h>

int main(int argc, char **argv)
{
struct stat statbuf;
struct stat keybuf;

int key;
int data;
int output;
int count;
char ans;
FILE * keyfile;
FILE * sourcefile;
FILE * destfile;

if(argc<4)
{
printf("OTP-Bunny 1.0\n");
printf("USAGE: OTP <source file> <output file> <keyfile>\n");
return (0);
}

/* Check number of arguments. */
if(argc>4)
{
printf("Too many arguments.\n");
printf("USAGE: OTP <source file> <output file> <keyfile>\n");
return(1);
}

/* Check if sourcefile can be opened. */
if((sourcefile = fopen(argv[1], "rb"))== NULL)
{
printf("Can't open source file.\n");
printf("Please enter a valid filename.\n");
printf("USAGE: OTP <source file> <output file> <keyfile>\n");
perror("Error");
return (1);
}

/* Get size of sourcefile */
fstat(fileno(sourcefile), &statbuf); 

/* Check if keyfile can be opened. */
if((keyfile = fopen(argv[3], "rb"))== NULL)
{
printf("Can't open keyfile.\n");
printf("Please enter a valid filename.\n"); 
printf("USAGE: OTP <source file> <output file> <keyfile>\n");
perror("Error");
return(1);
}                               

/* Get size of keyfile */
fstat(fileno(keyfile), &keybuf);

/* Check if keyfile is the same size as, or bigger than the sourcefile */
if((keybuf.st_size) < (statbuf.st_size))
{
printf("Source file is larger than keyfile.\n");
printf("This significantly reduces cryptographic strength.\n");
printf("Do you wish to continue? (Y/N)\n");
scanf("%c", &ans);
if(ans == 'n' || ans == 'N')
{
return (1);
}
if(ans == 'y' || ans == 'Y')
{
    printf("Proceeding with Encryption/Decryption.\n");
    }
else
{
printf("No option selected. Exiting...\n");
return (1);
}
}   

/* Check if destfile can be opened. */
if((keyfile = fopen(argv[2], "wb"))== NULL)
{
printf("Can't open output file.\n");
perror("Error");
return(1);                  
}    

/* Open destfile. */
destfile=fopen(argv[2], "wb");

/* Encrypt/Decrypt and write to output file. */
while(count < (statbuf.st_size))
{
key=fgetc(keyfile);
data=fgetc(sourcefile);

output=(key^data);

fputc(output,destfile);
count++;
}

/* Close files. */
fclose(keyfile);
fclose(sourcefile);
fclose(destfile);

printf("Encryption/Decryption Complete.\n");

/* Delete keyfile option. */
printf("Do you wish to delete the keyfile? (Y/N)\n");
scanf("%c", &ans);
if(ans == 'y' || ans == 'Y')
{
if ( remove(argv[3]) == 0)
    {
    printf("File deleted successfully.\n");
    }
else
    {
    printf("Unable to delete the file.\n");
    perror("Error");
    return(1);
    }
}

if(ans == 'n' || ans == 'N')
{
return(0);
}
else
{
printf("No option selected. Exiting...\n");
}
return(0);
}
4

2 に答える 2

6

実際にscanf()は正しく動作していますが、正しく使用するのは非常にトリッキーな関数であり、文書化された動作とは異なる期待を抱くのは簡単です.

最初のscanf()呼び出しでは、改行を残して 1 文字を読み取ります。

2 番目scanf()の呼び出しでは改行が読み取られます。これは次の...問題につながるためです。

" %c";を使用してコードを修正できます。先頭の空白は (オプションの) 空白をスキップします。

一般に、より適切な修正は、fgets()または同等のもの (readline()おそらく POSIX 2008) との組み合わせを使用することsscanf()です。fgets();を使用して、データ行をメモリに読み込みます。で分析しsscanf()ます。そうすれば、エラーをより適切に報告することもできます。ユーザーが入力したすべての情報をエラー報告に使用できます。

于 2012-10-15T17:37:50.230 に答える
-1

これは scanf の標準的な動作です。つまり、スペースまたは改行が検出された場合にのみ入力を取得し、次のステートメントの入力として取得される入力バッファーに \n を残します。

C++ ではなく厳密な C を使用している場合を除き、各呼び出しの後に cin.ignore() を使用して回避できます。その場合、 may be fgets を使用する方がはるかに優れています

于 2012-10-15T17:37:37.200 に答える