ポインタについて

Question

いつもお世話になっています。c言語初心者で勉強中です。課題で、文字列のn文字目の後ろに、標準入力から入力した単語を挿入時に文字列の長さが1023文字を超えても動作しつづけるプログラムを作成したいんです。
以下がコードです。```C言語
コード

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

#define BUFSIZE 1024

int main(){
  char init[]="[Wakamatsu]"; /* 初期データ用文字列 */
  char *str,*buf,*mou;
  int n;
  char input[BUFSIZE];       /* 入力用文字列バッファ */
  /* その他の変数を宣言 */
  int kazu,mozi;
  char *p;
  
  /* strの初期化を記述する*/
     str=init;
/* strlen,malloc,strcpyを使いstrにinitの内容のコピーを保持させる
 */
     kazu=strlen(init);/*文字列をコピーするからinit[i]みたいに文字として表記しない。*/printf("kazu=%d\n",kazu);
 str=(char*)malloc(kazu*sizeof(char));
 strcpy(str,init);/*文字列をコピーするからinit[i]みたいに文字として表記しない。*/
 
  while (1) {
      printf( "%s\n", str );
    if( scanf("%d %s", &n, input) != 2 ) break;
    /*
     * 処理の内容を記述する
     */ mozi=strlen(input);
     buf=(char*)malloc(kazu+2+mozi*sizeof(char));
     mou=(char*)malloc(mozi+2*sizeof(char));
     // mou[0]='[';
     // mou[mozi+2]=']';
      strncpy(mou,input,mozi+2);
      mou[0]='[';
      mou[mozi+2]=']';
    printf( "%s\n", mou);
     
    
     
   
     
       
  }   
       
       
     
     
    
    
    
        
       
  free(str);
  return 0;

Answer 1

質問者さんが期待する回答ではないと思いますが・・・。

何がおかしいか教えて下さい。

と書いていらっしゃいますが、コンパイラーが発するエラーメッセージが
「何がおかしいか」を教えています。エラーメッセージを「真剣に」読んで、理解しましょう。

提示していただいているソースは、他の回答で指摘されているとおり、「変なところ」が
満載です。
一つだけ、指摘すると、

define BUFSIZE 1024

は 元々'#'がないのでしょうか?それとも、コピペ時の単なるミスでしょうか?
いずれにしろ、BUFSIZE は stdio.h で定義済みのはずです。
改めて、定義しなおす理由は何でしょうか?

ソースコードを最初から、1行ずつ、「なぜこう書くのか」、「論理は正しいか」を
確認して下さい。

Comments

[ikuo-biyori]

すみません。よく確認するように気をつけます。

Answer 2

実行がうまくいきませんとは、どういう意味でしょうか？
MasahikoHirataさんの指摘にあるコンパイルエラーのことでしょうか？

実行がうまくいかないという箇所を具体的に明示いただけませんか？
また、コンパイラ等の環境も明示いただけませんか？

Comments

[ikuo-biyori]

コンパイルエラーでした。解決しました。ありがとうございました。

Answer 3

答えを書くのは簡単なのですが、まずコンパイラからのエラーが出ているはず。
このエラーを読み解かないと勉強にならないのじゃないかな？

Comments

[MasahikoHirata]

まず先頭から徐々に解説も。`strlen(init[])`は`strlen(init)`です。次に’malloc(strlen(init[10])*sizeof(char); ’これも’malloc(strlen(init[10])*sizeof(char))'で右の’）’が足りないし先の’strlen('と同じくポインターの使い方。（既にここでコンパイラーからエラーが出てないとオカシイ。

[ikuo-biyori]

文字列を文字として表記していました。お答えいただき、ありがとうございます。