正しい出力が得られない

Question

前提・実現したいこと

現在、モールス信号をアルファベットに直すプログラムを作成中です。
モールス信号のアルファベットを以下のように数字に対応させ、「・」を読み取ると変数numを2倍、「-」を読み取るとnumを2倍して1を足すことで、ファイルから1文字ずつ読み取り復号させることを考えました。
しかし、何も出力されずにプログラムが終了してしまいます。
理由が見当たらないため、分かる方にご教授頂きたいです。

該当のソースコード

#include <stdio.h>

int main(){
    int num = 1;
    char *s;
    char *alphabet = "**etianmsurwdkgohvf*l*pjbxcyzq**";

    while(scanf("%[^\n]%*c", s) != EOF){
        while(*s){
            if(*s == '.') num = num * 2;
            else if(*s == '-') num = num * 2 + 1;
            else if(*s == ' '){
                printf("%c", alphabet[num]);
                num = 1;
            }
            else if(*s == '/') printf(" ");
            s++;
        }
        printf("%c\n", alphabet[num]);
        num = 1;
    }
    return 0;
}

###input.txtの内容

-- --- .-. ... . / -.-. --- -.. .

###出力
期待される出力

morse code

実際の出力

（何も表示されずプログラム終了）

試したこと

ファイル入力を用いず、標準入力のときも出力されませんでした。

補足情報

Windows, VSCodeでプログラムを記述し、gccでコンパイルしました。

Answer 1

char *s;
これでは、sの値が不定なため、文字列を格納するバッファが保障されません。
char buff[4096];
char *s = buff;
のようにして、文字列を格納できる十分な領域を確保してください。
kazuma-sさんの提示されている実行結果がうまくいっているのは、たまたまです。

Comments

[progokina8]

回答有り難うございます。
領域が保証されないのが原因だったのですね。
もう一度基礎的な部分を学習しようと思います。

Answer 2

    char *s;
    //略
    while(scanf("%[^\n]%*c", s) != EOF){

C/C++では、ローカル変数の初期値は「不定」です。不定というのは、ソースコードからは値が決まらず、実行してみるまでどうなるかわからない、ということです。sはポインタですから、全メモリ空間のどこかを指しているのですが、それがどこかわかりません。もしかしたらプログラムの動作に必要なデータを指しているのかも。
そういうsが指している先に、データを保管しようとしています。正常な動作に必要なデータを勝手に上書きされてしまったら...さてなにがおこるかわかりません。
もしかしたら期待する動作をしちゃうかも知れないし、パソコンが爆発するかも知れません(しないけど)。「未定義」の動作というやつです。

データを格納する領域を確保してやりましょう。
文字のデータだけなら、数字までいれて長点短点あわせて5コまで。普通に(普通の人はモールス符号使わないか?)使う範囲では訂正符号の/HH(短点8コ)が最長でしょうか。
例えば、

    char s0[8+1];//長点短点+終了記号のデータを格納する領域を確保
    char* s=s0;//ポインタが確保した領域を指すように初期化
    //略
            else if(*s == ' '){
                printf("%c", alphabet[num]);
                num = 1;
                s=s0; //格納場所の頭に再設定
            }

としてやればとりあえず動くのでは。

あるいは、あえてそこでポインタを使わずに

    char s0[8+1];//長点短点+終了記号のデータを格納する領域を確保
    // char *s; 廃止
    int index=0;
    //略
    while(scanf("%[^\n]%*c", s0[0]) != EOF){
        while(s0[index]){
            if(s0[index] == '.') num = num * 2;
            else if(s0[index] == '-') num = num * 2 + 1;
            else if(s0[index] == ' '){
                printf("%c", alphabet[num]);
                num = 1;
                index=0;
            }
            else if(s0[index] == '/') printf(" ");
            index++;
        }
        printf("%c\n", alphabet[num]);
        num = 1;
    }

としてもよいのでは。

もっとも...ここでちゃぶ台返し。
scanf("%[^\n]%*c", s)でデータを拾おうとすると、100文字入力されたとしたらsの位置から100バイトデータを書き込んでしまいます(バッファオーバーラン)。これでは、確保していない領域に書くつまり「未定義動作」となってしまうでしょう。100文字と決まっているなら先のs0[100]とでもしてやればいいですが、入力されるのが100文字以下という保証があるわけでもありません。入力はn文字以下にしてね、とお願いしたとしても聞いてもらえるとは限りません。

ならば、最低の文字数1で勝負するほうが確実ではないでしょうか。

    char c;
    //char *s; 廃止
    //略
    while(scanf("%c",&c) != EOF){
      if(c == '.') num = num * 2;
      else if(c == '-') num = num * 2 + 1;
      else if(c == ' '){
          printf("%c", alphabet[num]);
          num = 1;
      }
      if( c== '\n'){
        if(num!=1){
          printf("%c", alphabet[num]);
          num = 1;
        }
        printf("\n");
      }
      else if(c == '/') printf(" ");
    }

(scanfでも"%8s"などとして入力を制限してやるとか、fgetsを使って文字列として扱う手もありますが)

Comments

[progokina8]

回答有り難うございます。
バッファオーバーランという言葉を聞いたことはあったのですが、理解しきれないままでいました。
様々な解決策を提案して下さり感謝です。
ベストアンサーにさせて頂きます。

Answer 3

$ cat -n a.c
     1	#include <stdio.h>
     2	
     3	int main(){
     4	    int num = 1;
     5	    char *s;
     6	    char *alphabet = "**etianmsurwdkgohvf*l*pjbxcyzq**";
     7	
     8	    while(scanf("%[^\n]%*c", s) != EOF){
     9	        while(*s){
    10	            if(*s == '.') num = num * 2;
    11	            else if(*s == '-') num = num * 2 + 1;
    12	            else if(*s == ' '){
    13	                printf("%c", alphabet[num]);
    14	                num = 1;
    15	            }
    16	            else if(*s == '/') printf(" ");
    17	            s++;
    18	        }
    19	        printf("%c\n", alphabet[num]);
    20	        num = 1;
    21	    }
    22	    return 0;
    23	}
$ gcc a.c
$ cat input.txt
-- --- .-. ... . / -.-. --- -.. .
$ ./a.out <input.txt
morse *code
$ 

標準入力で表示されますよ。
ただし、スペースが 2個連続すると * が出ます。
/ の両端のスペースを連続したものと解釈し、
num=1 を表示しています。

追記
thkanaさんの指摘のように s が未初期化なので、動いていたのは偶然です。
* が表示されてしまうという指摘だけを参考にしてください。

Comments

[progokina8]

追記までして頂き、有り難うございます。
*が表示されてしまうのは気づいていなかったので助かりました。

Answer 4

デバッグしましょう
ブレークポイントを設定すれば、コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

Comments

[progokina8]

まだ初心者のため、ブレークポイントは使ったことが無かったです。
ご指摘有り難うございます。デバッグしてみたいと思います。