strcpy関数の自作

Question

C言語で以下のようなstrcpy関数を使った文字列のコピーを自作関数で実装したい。

#include <stdio.h>
#include <string.h>
 
int main(void) {
    char str1[16] = "Hello World!";
    char str2[16];
    
    // 文字列のコピー
    strcpy(str2, str1);
    printf("str2の文字列は: %s\n", str2);
    
    return 0;
}

途中までのコード

自分で以下のようなコードを考えたのですが、
コピー先に終端文字が入っていないなど若干問題点があるとおもいます
どう修正すればいいでしょうか。

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

void *strcpy(char *dst, char *src){
        int i=0;
        for(i=0;src[i]!='\0';++i){
                dst[i]=src[i];
        }
}

int main(void){
        char str[10]="Hello!";
        //char str2[10];
        strcpy(str+1,str);
        printf("%s\n",str);

        return 0;
}

Comments

[thkana]

本題ではありませんが、
標準ライブラリ関数のstrcpyではコピー元とコピー先の領域が重なっている場合の動作は未定義だったかと思います。
作った関数に怪しいふるまいがあるかどうかは別の話として、本家strcpyと同じ動作をするかどうかを議論するなら本家strcpyに許されていないテスト条件を持ち出しても仕方がないでしょう。

[kazuma-s]

//char str2[10];
strcpy(str+1,str);
なぜ、str2 を str+1 に変えたのか説明してください。

[退会済みユーザー]

夢も希望もないコードはこちらです。
https://sourceware.org/git/?p=glibc.git;a=blob;f=string/strcpy.c;h=d2a8531f1f416a56d6fab8ecb783722098c9c994;hb=HEAD
ご参考までに。

Answer 1

コメントにあったコードをもとに修正するなら、

void strcpy(char *dst, char *src)
{
  int i=0;
  do{
    dst[i]=src[i];
    if(src[i] == '\0'){
      break;
    }
    i++;
  }while(1);
}

こんな感じです。

ちなみに、私が(規格に沿って)書くと、

char* strcpy(char *dst, const char *src)
{
  int i = 0;
  while(dst[i] = src[i]) i++;
  return dst;
}

こんな感じですね。条件式の中で代入をしているので、行儀のよいコードではありませんが…。

Answer 2

終端文字に達した後、それをコピーせずに関数を終了していますが、終端文字をコピーしてから関数を終了すれば良いかと思います。

あと、値を返さない関数というつもりなら、void *strcpy(～)じゃなくてvoid strcpy(～)です。

Comments

[takahashi1031]

#include <stdio.h>

void strcpy(char *dst, char *src){
int i=0;
do{
dst[i]=src[i];
i++;
}while(src[i] != '\0');
}

int main(void){
char str[10]="Hello!";
char str2[10];
strcpy(str2,str);
printf("%s\n",str2);
return 0;
}

これでいいでしょうか？

[majiponi]

動かしてみれば分かるけど、まだ動かないですね。この後書く私の回答も参考にしてください。

[otn]

コピーの代入を2回書くのがいやと言うことですかね。
「今コピーしたのがNULだったか？」を判断しないといけないので、[] の中の値は同じでないと。
i++した後で判断するなら、src[i-1]!='\0' でしょうか。

私なら、
for(i=0;dst[i]=src[i];i++);
ですが、これだとコンパイラーが 「= は == の間違いではないですか？」という警告を出すかも知れないので、
for(i=0;(dst[i]=src[i])!='\0';i++);
でしょうか。

Answer 3

終端文字が出てきたらコピーする前にループを終了するロジックになっていますが、
コピーした文字が終端文字なら処理を終えるというロジックにすれば良いのではないかと思います。

あと、敢えて付け加えるなら、コピー先のバッファの容量を把握してバッファオーバーフロー対策をしないと、脆弱なコードになります。（もともとstrcpy自体そういうものですが）