C++ 暗号化 謎のエラー

Question

前提・実現したいこと

大学の課題で作ったプログラムで、提出する際にエラーメッセージが出るのですが原因が分かりません。どこをどのように直すべきか教えてください。

問題

There is a trick to encrypt the data and use a word as its key. Here's 
how it works: First, choose a word as the key, such as TRAILBLAZERS. If 
the word contains repeated letters, only the first one is kept, and the 
rest are discarded. Now, the modified word is listed below the alphabet, 
as shown below:
A B C D E F G H I J K L M N O P Q R S T U V W X Y Z
T R A I L B Z E S
Then arrange the letters in the alphabet that do not appear in the 
modified word after the modified word to get the coding table:
A B C D E F G H I J K L M N O P Q R S T U V W X Y Z
T R A I L B Z E S C D F G H J K M N O P Q U V W X Y
Therefore, based on the key of TRAILBLAZERS, if our original message is 
ATTACK, it will be encrypted as TPPTAD.
Input requirements: input key and original information, output encrypted 
information
Example 1: Input TRAILBLAZERS ATTACK and output TPPTAD.

Input : Enter a line, including the input key and the original 
information (both capital letters), separated by spaces.

Output : Output a line, representing the encrypted information

エラーメッセージ

1. The data is opened too small, resulting in access to a memory area 
that should not be accessed
2. A division by zero error occurs
3. A large array is defined in the function, causing the program stack 
area to be exhausted
4. The pointer is used incorrectly, resulting in access to a memory area 
that should not be accessed
5. It is also possible that the program throws an unreceived exception

該当のソースコード

#include<iostream>
using namespace std;
int main()
{
    char list1[26] = {0}, list2[26] = {0};//暗号化する文字列, 新たなアルファベット表
    int size = 0; char gomi;//入力した文字の文字数
    bool check = false;// 前に同じ文字が出たかどうか
    for(int i = 0; ; ++i){
        gomi = getchar();
        for(int j = 0; j <= i; ++j){
            if(gomi == list2[j]) check = true;
        }
        if(gomi == ' ') break;//空欄で終了
        else if(check == true) {i -= 1; check = false; continue;}//前に同じ文字があれば飛ばす
        else list2[i] = gomi;
        size += 1;
    }
    for(int i = 0; ; ++i){//暗号化する文字列の格納
        gomi = getchar();
        if(gomi == '\n') break;//改行で終了
        else list1[i] = gomi;
    }
    int num = 65;//ASCⅡコード
    for(int i = size; i < 26; ++i){
        for(int j = 0; j < size; j++){
            if(num == list2[j]){num += 1; j = -1;}//前に同じコードがあれば確認を最初からやり直し
        }
        list2[i] = num;
        num += 1;
    }
    for(int i = 0; i < 26; ++i){
        if(list1[i] == 0) break;//暗号化する文字列の出力が終わったら終了
        cout << list2[list1[i] - 65];
    }
    cout << endl;
}

Comments

[gentaro]

https://teratail.com/help/avoid-asking
「何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。」

学校の課題であれば、それまでの講義や教材の中で解決策は示されているはずです。
そうでない場合は講義や教材の質の問題が悪い可能性もあるため、教員に質問・相談しましょう。

[raccy]

マルチポスト https://stackoverflow.com/questions/61348928/i-cant-understand-the-error-messages-encripting-data
なお、学校の課題であれ、問題文には著作権があります。引用の要件を満たすために、大学名、講義名、出題者名(教員の名前)を明記してください。

[JJJJJJJJJamm]

返信ありがとうございます、著作権のことは初めて知りました。ご指摘ありがとうございます、次からは気をつけます。Tsinghua university, C++Programme Design and Training, Hong Zhao　です。助教にも質問しましたが、自信の中国語が拙いこともありあまり理解出来なかったため、こちらでも質問させていただきました。

[majiponi]

How about your English? He will explain what he thinks to you in English if you will, as the whole of the question is written in English, not in 汉语. Maybe asking for "a written answer", such as an e-mail, helps you understand his Chinese, since we can read and share what he thinks. (您能阅懂得汉语吧?)

[JJJJJJJJJamm]

Sorry for being late to reply, I've seen your reply already but I was busy for my classes that I haven't tried your advices yet.....Let me try that in minutes. The original question was written in Chinese, but I translated to English. （我看的懂中文，如果对您来说更方便的话我们就用中文来沟通吧。）

Answer 1

エラーメッセージは大学か教員が用意したようなものですか?
一般的にエラーメッセージはそのプログラムについて起こったエラーについて報告します。
ところが2番のメッセージは0で割り算してると言ってるのに、そもそも割り算が使われてるようには見えません。
単なるありがちな原因を並べただけにも思えます。どういう場合にどのようなメッセージが出されるのか確認してみてください。

そのうえでぱっと見のアルゴリズム上の問題としては

1．暗号化するメッセージの長さが26字以下と決めつけてる
2．keyとなる文字列にアルファベットがすべて含まれていた場合、list2の27番目の要素へのアクセスが発生する

このあたりでしょうか。

Comments

[JJJJJJJJJamm]

エラーメッセージとして書いたものはRuｎtime Errorというエラーが表示された際の主な原因として教員の方から教えていただいたものです。説明不足で申し訳ありません。yudedako67さんのご指摘とても助かりました、どれもうっかりしていて気が付きませんでした。ご回答ありがとうございました。

Answer 2

I'll give you some tips, which don't answer your questions directly.

1. Check what getchar() returns; It returns negative values when an error, including EOF, happens. Otherwise this encrypter crashes.

2. Discard non-capital letters. Same as above.

3. Write "j < i;" instead of "j <= i" to avoid accessing list2[26].

4. You don't have to preload the message. You can getchar() after creating the substitution table. Needless to care the length of the message.

5. Avoid using magic numbers like 65. You can use 'A' to represent that.

Comments

[JJJJJJJJJamm]

我刚才按照您的回复改变了自己的代码，结果通过了！ 好像是因为getchar()哪里有了问题。谢谢您的回复 ！