C言語 ポインタを使って文字列を別の文字列に追加したいです。

Question

前提・実現したいこと

プログラミング初学者です。課題で文字列に文字列を追加するという課題があります。ポインタを利用して文字列に文字列を追加しようとしたのですが現状何も出力されない状態が続いています。問題点を指摘いただけたらありがたいです。

ちなみに調べたところstrcatなどの関数があるようですが、今回は関数名は指定されているのでそちらは使えないです。お手数ですがどうぞよろしくお願いします。それと#使うと文字が大きくなってしまったのでincludeの前の#は省略してあります。

発生している問題・エラーメッセージ

エラー自体は特になしですが何も出力されません。関数の呼び出し方が違うのでしょうか…？

該当のソースコード

include <stdio.h>
void addChar(char *str,char *str2){

int i;
unsigned long num = sizeof(str) / sizeof(str[0]);
for(i = 0;str2[i] != '\0';i++){
        str[i + num] = str2[i];
}
printf("%c",*str);

}

int main(int argc, const char * argv[]){
char *str = "HELLO";
char *str2 = "WORLD";
addChar(str,str2);

return 0;

}

試したこと

ネットで調べましたがstrcatなどの関数を使用した方法は出てきましたが使わない方法はわかりませんでした。

Comments

[hoshi-takanori]

まず課題をしっかり確認して欲しいのですが、str が指す先の文字列に str2 の内容を追加して破壊的変更を行う必要があるのでしょうか？ また、addChar の中で出力も行うのでしょうか？ (addChar で出力を行うなら破壊的変更は不要だと思うので…。)

[misa_san]

わかりづらくて申し訳ありません、addchar関数の中身は「strの末尾にstr2の文字列を追加する」です。出力までは求められていません。

Answer 1

解決済みですが

こういう処理でオススメなのはsprintf関数（とこの派生系）です。strcat禁止なので、これも禁止かもしれませんが。

大抵の場合、printf関数を最初に学んでいるはずでしょうから、printfの画面出力を変数格納にするだけなので使い方が簡単です。

Answer 2

#include <stdio.h>

void addChar(char *str,char *str2){
  int i;
  int num;

  // str の末尾位置をnumに求める
  for ( num = 0; str[num] != '\0'; num++) {}

  // strにstr2のナカミを追加する
  for( i = 0; str2[i] != '\0'; i++) {
    str[i + num] = str2[i];
  }

}

int main(int argc, const char * argv[]){
  char str[16] = "HELLO"; // 十分な領域を用意すべし
  char str2[]  = "WORLD";
  addChar(str,str2);
  printf("[%s]\n", str);
  return 0;
}

Comments

[misa_san]

コード及び丁寧にコメントまでつけてのご回答ありがとうございます。良くないと思いつつも丸写ししたらできました…。自分がまず何を理解できていなかったのかを改めて確認しつつ、学習に励みたいと思います。

Answer 3

char *str = "HELLO";
char *str2 = "WORLD";

上記のような“文字列リテラル”は、書き換え不可です。
結合用には、別の配列を用意しましょう。
参考：ポインタと文字列→「文字列のポインタのデメリット」

Comments

[misa_san]

参考URLまで貼り付けていただきありがとうございます。理解できました…。

Answer 4

sizeof(str) / sizeof(str[0])

これは、strが配列の時に配列要素数を求める式ですが、strは配列じゃなくてポインターだし、そもそも配列だったとしても配列サイズを求めても何の意味も無いです。

numには、「strから何番目に'\0'があるか」という番号を入れれば良いでしょう。
関数を使うとstrlen(str)ですが、自分でやるなら、その下の継ぎ足しのforと似たようなことをやれば良いです。

Comments

[misa_san]

ご回答いただきありがとうございます。確かに配列ではないですね…。
自分のコードだと他にも多々問題点ありましたが、otn様のおっしゃる通りfor文で出来ました！