文字列複写の空白について

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
文字列を複写する作業なんですが、複写までは出来るんですが大量の空白が出てしまいます。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題（大きな空白）

変数にコピーする文字列を入力して下さい。
abcdefg
変数にコピーするバイト数を入力して下さい。
4
入力値『abcdefg』の4バイトをコピーします。
入力した値は『abcdefg』です。
コピーした値は『abcdefg』です。
moridaiakira-no-MacBook-Air:desktop Taisei$ gcc -o program P5.c
moridaiakira-no-MacBook-Air:desktop Taisei$ ./program
変数にコピーする文字列を入力して下さい。
abcdef
変数にコピーするバイト数を入力して下さい。
4     
入力値『abcdef』の4バイトをコピーします。
入力した値は『abcdef』です。
コピーした値は『abcd


























』です。

該当のソースコード

#include<stdio.h>
#include<string.h>
int main(void)
{

    char a[31];
    memset(a, '\n' ,sizeof a);
    
    char b[31];
    memset(b, '\n' ,sizeof b);

    int c=0;

    printf("変数にコピーする文字列を入力して下さい。\n");
    scanf("%s",a);
    printf("変数にコピーするバイト数を入力して下さい。\n");
    scanf("%d",&c);

    strncpy(b,a,c);
    printf("入力値『%s』の%dバイトをコピーします。\n",a,c);
    printf("入力した値は『%s』です。\n",a);
    printf("コピーした値は『%s』です。\n",b);
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

strncpy(b,a,c);

strncpyの解説を読みましょう。検索すれば出てきます
文字列をコピーしますが、cバイトを超える分はコピーされず、文字列終端の'\0'も書き込まれません

ということで、今回のような場合にはあとで終端の'\0'を書き込まなけりゃなりません。

Comments

[Hamada-hiro]

検索したら出てきました…。
本当に頭が悪くて申し訳ないですが

strncpy(b,a,c);
printf("入力値『%s』の%dバイトをコピーします。\n",a,c);
printf("入力した値は『%s』です。\n",a);
b[c]='\0';
printf("コピーした値は『%s』です。\n",b);
}

これだと言っている意味と違いますか？
『終端の'\0'を書き込む』の書き込む場所がわかりませんでした…
本当に恥ずかしいですが教えてもらってもよろしいですか？

[y_waiwai]

それでいいです
それで（今回のコードは）正常に動作するようになります

んで、今回のコード改正点を指摘しておきます
もしも、aに３１文字以上の文字列が入力された場合、ｃに３１以上の数字が入力された場合、アクセス違反となります

まあ、演習用でこのような制限のあるコードを提示されたり、このようなコードを要求されたりしますんでこれはこれでまあ、構わないんでしょうが、
実務では全く使えないコードだということをおぼえておきましょう

[Hamada-hiro]

あまりバイト数とか決めない方がいいですか？

[y_waiwai]

上に書いたように中途半端に小さい値だと入力しすぎると暴走します
ちょっとやそっとでは入力できないような大きなサイズが理想的ですが、そゆわけにもいきませんな

Answer 2

コピーした後、その直後に'\0'を書き込みます（文字列の終わりの印として）。

「Cにおける文字列とは何なのか」の理解が足りてないと思います。

Answer 3

#include<stdio.h>
#include<string.h>

int main(void)
{

    char a[31];
    a[0]='\0';
    char b[31];
    int c=0;

    printf("変数にコピーする文字列を入力して下さい。\n");
    scanf("%30s",a);
    printf("変数にコピーするバイト数を入力して下さい。\n");
    scanf("%d",&c);
    if(c >= sizeof(b)) c = sizeof(b) - 1;
    if(c >= strlen(a)) c = strlen(a);
    strncpy(b,a,c);
    b[c] = '\0';
    printf("入力値『%s』の%dバイトをコピーします。\n",a,c);
    printf("入力した値は『%s』です。\n",a);
    printf("コピーした値は『%s』です。\n",b);
}

他の人の回答と異なった視点から書くと、初期化は必要最小限に抑えたほうがよいと思います。aを先頭要素だけ設定しているのは、先頭要素がヌル文字なら空文字列を意味するルールになっているからです。あと、長い文字列が入力された場合へのケアが足りてなかったので、一部修正してあります。

Comments

[Hamada-hiro]

scanf("%30s",a);
30文字のみ表示される

if(c >= sizeof(b)) c = sizeof(b) - 1;
これは空欄が無くなるですか？
ごめんなさいここが少しわかりません

if(c >= strlen(a)) c = strlen(a);
バイト数が多くてもa[31]が増える

こんな解釈であってますか？

[majiponi]

scanf("%30s",a);は、空白文字以外を最大30文字読み取り、その後ヌル文字をaの終端に追加します。

例えば、stringという6文字の文字列を入力した後で、60文字コピー、と書いたとしましょう。そうした場合、strncpy関数はstringという6文字の後、54文字のヌル文字をbに書き込みます。バッファオーバーフローして死にます。c=sizeof(b)-1;はコピーの上限を設定して、バッファオーバーフローを防ぐためのものです。

最後の1つは、今回はなくても動きます。が、aとbのバッファサイズが違う場合、問題になりえます。例えば、bのバッファが100文字、aが10文字だとします。ここで、60文字コピーする、となった場合、aの59番目の文字(アクセスしてはいけない)に、strncpyはアクセスしようとしようとするでしょうか? きちんと答えられたら削っても大丈夫です。(ちなみに、同じような機能を持つmemcpy関数を使用する場合、削るとまずいことになります)

Answer 4

？
ご自身でmemsetを使って配列に突っ込んだ改行では。