C言語のstrncpy関数について

Question

前提

AOJのこの問題を解いています。

該当のソースコード・試したこと

問題を解こうと9b.cを書いたのですがstrncpyでcardcpy1に代入するときにうまく動作しない（指定した文字数以上を代入してしまう）ので、何が問題があるのかを知るために9btest.cを書いたのですが結局よくわかりませんでした。

9b.c

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main (){
    char card[201];
    char cardcpy1[201],cardcpy2[201];
    int m,h,i,j,substr,wordcount,lenght;
    while(1){
        scanf("%s",card);
        if(card[0] =='-')break;
        scanf("%d",&m);
        for(i = 0,lenght = strlen(card);i < m;i++){
            scanf("%d",&h);//hは対象文字数
            substr = h;//substrには対象文字以外の先頭の
            wordcount = lenght - h;

            /*cardcpy1に対象文字を代入する*/
            strncpy(cardcpy1,&card[0],h);
            printf("cardcpy1:%s\n");//ﾃﾞﾊﾞｯｸﾞ
            /*cardcpy2に対象文字以外を代入する*/
            strncpy(cardcpy2,&card[substr],wordcount);
            printf("cardcpy2:%s\n");//ﾃﾞﾊﾞｯｸﾞ

            /*cardcpy2にシャッフルの結果を代入*/
            strcat(cardcpy2,cardcpy1);
            printf("現在のシャッフル:%s\n",cardcpy2);//ﾃﾞﾊﾞｯｸﾞ
            strcpy(card,cardcpy2);
        }
    printf("%s\n",card);
    }
return 0;
}

aabc
3
1
2
1

aabc
3
1
cardcpy1:aabc
cardcpy2:abc
現在のシャッフル:abcaa
2
cardcpy1:abcaa
cardcpy2:caa
現在のシャッフル:ccaaa
1
cardcpy1:ccaaa
cardcpy2:caaa
現在のシャッフル:caaac
caaac

9btest.c

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main (){
    char card[201];
    char cardcpy1[201],cardcpy2[201];
    int m,h,i,j,substr,wordcount,lenght;

    scanf("%s",card);
    strncpy(cardcpy1,&card[0],1);
    printf("cardcpy1:%s\n");//ﾃﾞﾊﾞｯｸﾞ
    printf("%s\n",card);
return 0;
}

aabc

cardcpy1:aabc//本当はここでcardcpy1:aとなってほしい
aabc

Comments

[LouiS0616]

9btest.cは警告を山ほど吐きますが、確認されましたか。

[yokotatsu]

これは、どういう問題なのでしょうか。
入力のaabc及び3,1,2,1とはなんでしょうか。
どのようなoutputがでれば良いのでしょうか。
又、strncpy関数を使用することが必須なのですか。

[Courange]

改めてみるとかなりひどいコードというのは理解してはいるのですが、警告やエラーが一切出ていないのですよね・・・

[LouiS0616]

どのようにコンパイルしていますか？

Answer 1

全てのコードは見ていませんが、、、

まず、

strncpy(cardcpy1,&card[0],h);

h バイトのコピーとなります。 最後に、'\0' は付けません。C 言語での文字列は、最後が '\0' となる規則なので、コピーした文字列 + ゴミ となる事があります。

printf("cardcpy1:%s\n");//ﾃﾞﾊﾞｯｸﾞ

cardcpy1 の文字列を出力したいのでは無いかと推測しますが、期待する文字列は出力されません。 下記のようにすべきでしょう。 (cardcpy1 を出力する場合)
printf("cardcpy1:%s\n", cardcpy1);//ﾃﾞﾊﾞｯｸﾞ
%sは、次の引数を文字列と見なし、出力しますが、 次の引数が無いので、動作は不明となります。(ゴミが出力される?)

Comments

[dodox86]

> h バイトのコピーとなります。 最後に、'\0' は付けません。
少し語弊があるかもしれません。strncpyの場合、コピー元文字列の長さより"多く"第3引数がセットされているならば、'\0'は付けられます。

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

int main(void) {
char s[10];
memset(s,'*', sizeof(s));
s[9] = '\0';
strncpy(s, "12345", sizeof(s));
printf("%s\n", s);

memset(s, '*', sizeof(s));
s[9] = '\0';
strncpy(s, "12345", 5);
printf("%s\n", s);
return 0;
}

実行例です。
$ gcc -Wall t1.c -o t1.exe;./t1.exe
12345
12345****

[pepperleaf]

> コピー元文字列の長さより"多く"第3引数がセットされているならば、'\0'は付けられます。
失礼、ちょっと手抜きでした。
コピー元の文字列より短い文字数(バイト数)が指定された場合、自動で、'\0'が付けられないが正しいですね。

[Courange]

正直理解にかなり時間がかかりました・・・
ですが、以下のコードでなんとか正解することができました！

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main (){
char card[201];
char cardcpy1[201],cardcpy2[201];
int m,h,i,j,substr,wordcount,lenght;
while(1){
scanf("%s",card);
if(card[0] =='-')break;
scanf("%d",&m);
for(i = 0,lenght = strlen(card);i < m;i++){
scanf("%d",&h);//hは対象文字数
substr = h;//substrには対象文字以外の先頭の
wordcount = lenght - h;

/*cardcpy1に対象文字を代入する*/
strncpy(cardcpy1,card,h);
cardcpy1[h]='\0';
//printf("cardcpy1:%s\n",cardcpy1);//ﾃﾞﾊﾞｯｸﾞ
/*cardcpy2に対象文字以外を代入する*/
strncpy(cardcpy2,&card[h],wordcount);
cardcpy2[wordcount]='\0';
//printf("cardcpy2:%s\n",cardcpy2);//ﾃﾞﾊﾞｯｸﾞ

/*cardcpy2にシャッフルの結果を代入*/
strcat(cardcpy2,cardcpy1);
memset(card,0,sizeof(card));
card[200]='\0';
strcpy(card,cardcpy2);
//printf("現在のシャッフル:%s\n",card);//ﾃﾞﾊﾞｯｸﾞ

}
printf("%s\n",card);
}
return 0;
}

[Courange]

(インデント全くないのはご了承ください・・・)

[rubato6809]

期待どおりの結果が得られて結構でした。でもムダが多い。改良の余地があります。これで完成と思わないほうが良いです。今の時点では３点アドバイスできます。
(1) strncpy() でなく strcpy() で実現可能
(2) 配列は２本あれば十分
(3) memset() と card[200] = '\0'; および cardcpy2[wordcount]='\0'; は不要

[Courange]

アドバイスありがとうございます。なるべく無駄を減らせるよう頑張りたいと思います。

Answer 2

9b.cを修正しました。いかのようにしてください。
strncpy関数は実にやっかいな関数です。コピー先の終わりに終端文字('\0')を付加されるときと、付加されないときがあるのです。
今回のケースでは終端文字が付加されないので、意図的に付加してあげる必要があります。（//追加のコメントがある文です。）
それと、デバッグのために、cardcpy1,2を印字していますが、誤っているために、正しく修正しておきました。（//修正のコメントがある文です。）
strncpyで終端文字('\0')が付加されるかされないかは、説明を精読し、十分理解した後、使ってください。
（strncmp,strncatも同様にやっかいな関数ですので、それらを使用するときも気を付けてください）

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
    char card[201];
    char cardcpy1[201], cardcpy2[201];
    int m, h, i, j, substr, wordcount, lenght;
    while (1) {
        scanf("%s", card);
        if (card[0] == '-')
            break;
        scanf("%d", &m);
        for (i = 0, lenght = strlen(card); i < m; i++) {
            scanf("%d", &h);                        //hは対象文字数
            substr = h;                             //substrには対象文字以外の先頭の
            wordcount = lenght - h;
            /*cardcpy1に対象文字を代入する */
            strncpy(cardcpy1, &card[0], h);
            cardcpy1[h]='\0';                       //追加　終端文字設定
            printf("cardcpy1:%s\n",cardcpy1);       //修正　ﾃﾞﾊﾞｯｸﾞ正しく印字
            /*cardcpy2に対象文字以外を代入する */
            strncpy(cardcpy2, &card[substr], wordcount);
            cardcpy2[wordcount]='\0';               //追加　終端文字設定
            printf("cardcpy2:%s\n",cardcpy2);       //修正　ﾃﾞﾊﾞｯｸﾞ正しく印字

            /*cardcpy2にシャッフルの結果を代入 */
            strcat(cardcpy2, cardcpy1);
            printf("現在のシャッフル:%s\n", cardcpy2);  //ﾃﾞﾊﾞｯｸﾞ
            strcpy(card, cardcpy2);
        }
        printf("%s\n", card);
    }
    return 0;
}