C言語 別ファイルを配列に読み込んでコマンドで操作（reallocの使用方法）

Question

C言語初心者です。
別ファイルを配列に読み込んで、複数のコマンドで操作できるような下記のコードを書きました。
下記のコマンドで'a'（ カーソル行の後ろに１行挿入（カーソル移動させる））について、
元々256の1行あたりの最大文字数だったのを、
1024まで増やすことができるようreallocを使って改造したいです。
文字数MOJIを1024と定義しreallocして文字数を増やしたいのですが、
下記のコンパイルエラーがでます。
演算子の不適切な利用で起こるエラーのようですが、どうしたらコンパイルが通るのか理解できず
ご指摘いただけたら幸いです。

＜エラー内容＞コマンドaの箇所
invalid operands to binary(have 'int' and 'int*')
moji=realloc(moji,sizeof(int)(MOJI ++moji));

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

#define  GYO 200 //1ファイルの最大行数
#define  MOJI 1024 //1行の最大文字数
#define  LINE line+1

int kansu(char* str,FILE* fp);

int main(int argc,char* argv[])
{

FILE* fp;
int*  moji;
char str[GYO][MOJI];
moji=malloc(sizeof(int)*MOJI);
char retsu[128];
int i=0;
int line=0;//現在の行番号
int gyo;
int k;
line=malloc(sizeof(int)*LINE);
line=malloc(sizeof(int)*(LINE* ++line));


　　　if(argc==2){
　　　　if((fp=fopen(argv[1],"r"))==NULL){
　　　　printf("ファイルが開けませんでした\n");
　　　　exit(EXIT_FAILURE);
　　　　}
　　　}
else{
printf("引数の値が不正です\n");
exit(EXIT_FAILURE);
}
//strにファイルの内容読込み
while(fgets(str[i],MOJI,fp)!=NULL){
//ファイルの行数チェック
　　if(i>GYO){
　　　printf("ファイルの長さが[%d]を超えました",GYO);
　　　break;
　　}

　　if(strlen(str[i])==255){
　　　if(kansu(str[i],fp)){
　　　　exit(EXIT_FAILURE);
　　}
　　　　printf("strlen=[%ld]\n",strlen(str[i]));
　　}

　　　　printf("%d:%s",LINE,str[line]);
　　　　printf("%ld\n",strlen(str[line]));
　　　　line++;
　　　　i++;
　　}


　　　line--;
 　　 int line_max=line;//ファイル全体の行数
   　 while(1){
             printf("コマンドを入力してください:");
　　　　fgets(retsu,sizeof(retsu),stdin);
　　　//大文字も小文字も対応する処理
　　　　for(k=0;k<strlen(retsu);k++){
　　　　retsu[k]=tolower(retsu[k]);
　　　　}
　　　　//nコマンドのため
　　　　gyo=atoi(retsu);
　　　　gyo--;
            //各コマンドの処理
　　　　if(strchr(retsu,'o')){
　　　　　　for(int i=0;i<line_max;i++){
　　　　if(i==line){
　　　　printf("*%d:%s\n",i+1,str[i]);
　　　　}else{
　　　　printf("%d:%s\n",i+1,str[i]);
　　　　　　}
　　　　}
　　　　printf("line:%d\n",LINE);
　　　　printf("line_max:%d\n",line_max+1);

　　　　}else if(strchr(retsu,'d')){
　　　　　　　　if(line==0){
　　　　　　　　strcpy(str[line],"");
　　　　　　　　printf("配列削除しました\n");
　　　　}else if (line_max==line){
　　　　　　　　strcpy(str[line],"");
　　　　　　　　line--;
　　　　　　　　line_max--;

　　　　}else{
　　　　　　　for(i=line;0<=(line_max-i);i++){
　　　　　　　strcpy(str[i],str[i+1]);
　　　　　　　}
　　　　　　line_max--;
　　　　}

　　　　　　　printf("*%d:%s\n",LINE,str[line]);

　　　}else if(strchr(retsu,'r')){
　　　　　　　if(line<0){
　　　　　　　printf("文字列が入っていません\n");
　　　　　　　continue;
　　　}
　　　　　　　printf("Enter sentence:");
　　　　　　　fgets(str[line],MOJI,stdin);
　　　　　　　printf("%d:%s\n",LINE,str[line]);
　　　　　　　continue;

　　　}else if(strchr(retsu,'a')){
　　　if(line<0){
　　　　printf("文字列が入っていません\n");
　　　　continue;
　　　}
　　　for(int i=line_max; i>=line; i--){
　　　　　strcpy(str[i+1],str[i]);
　　　}
　　　　　line_max++;
　　　　　line++;
　　　　　fgets(str[line],sizeof(str[line]),stdin);
         moji=realloc(moji,sizeof(int)*(MOJI* ++moji));
　　　　　printf("*%d:%s\n",LINE,str[line]);
　　　　　continue;

　　　}else if(strchr(retsu,'i')){
　　　　　for(int i=line_max;i>=line;i--){
　　　　　strcpy(str[i+1],str[i]);
　　　}
　　　　　　line_max++;
　　　　fgets(str[line],sizeof(str[line]),stdin);
　　　　printf("*%d:%s\n",LINE,str[line]);

　　　}else if(strchr(retsu,'p')){
　　　　　　　if(line<1){
　　　　　　　printf("ファイルの先頭です\n");
　　　　　　　continue;
　　　}
　　　　　　　line--;
　　　　　　　printf("*%d:%s\n",LINE,str[line]);

　　　}else if(strchr(retsu,'n')){
　　　if(line>line_max){
　　　　　　　printf("最端行数を超えました\n");
　　　　　　　line=line_max;
　　　}
　　　　　　　line++;
　　　　　　　printf("*%d:%s\n",LINE,str[line]);
　　　　　　　continue;

　　　}else if(strchr(retsu,'s')){
　　　　int p=line-10;
　　　　　　if(p<1){
　　　　　　 p=1;
　　　}
　　　　int n=line+10;
　　　　if(n>line_max){
　　　　n=line_max;
　　　}
　　　for(i=p;i<=line;i++){
　　　　　　printf("%d:%s\n",i,str[i-1]);
　　　}
　　　　　　printf("*%d:%s\n",LINE,str[line]);
　　　for(i=line+1;i<n+1;i++){
　　　　　　printf("%d:%s\n",i+1,str[i]);
　　　}
　　　continue;

　　　}else if(strchr(retsu,'f')){
char z[MOJI];
if(line-1<0)
{
printf("入替えできません\n");
continue;
}
strcpy(z,str[line]);
strcpy(str[line],str[line-1]);
strcpy(str[line-1],z);
printf("%d:%s\n",LINE,str[line]);
continue;



　　}else if(strchr(retsu,'b')){
　　　　char y[MOJI];
　　　　if(line-1<0)
　　{
　　　　printf("入替えできません\n");
　　　　continue;
　　}
　　　　strcpy(y,str[line]);
　　　　strcpy(str[line],str[line+1]);
　　　　strcpy(str[line+1],y);
　　　　printf("%d:%s\n",LINE,str[line]);
　　　　continue;

 　 }else if (strchr(retsu, 'v')) {
　　char fname[128];
　　char c2[5];
　　FILE* wfp;
　　printf("file name is [%s] ?:[y or n]: ", argv[1]);
　　　
　　fgets(c2, sizeof(c2), stdin);

　　if (strchr(c2, 'y')) {
　　　　strcpy(fname, argv[1]);
　　} else if (strchr(c2, 'n')) {
　　　　printf("Enter file name:");
　　fgets(fname,sizeof(fname),stdin);
　　} else if(c2[0] == '\n') {
　　　　continue;
　　} else {
　　　printf("Command Error.[%s]\n",c2);
　　　continue;
　　}

　　if ((wfp = fopen(fname, "w")) == NULL) {
　　　　printf("File open Error:w\n");
　　}
　　　　for (i=0; i <= line_max; i++) {
　　　　fprintf(wfp, "%s", str[i]);
　　}
　　　　fclose(wfp);
　　　　printf("Write Finish...\n");
　　　　continue;


　　}else if(strchr(retsu,'t')){
　　　line=1;
　　　printf("%d:%s\n",LINE,str[line]);
　　}else if(strchr(retsu,'l')){
　　line=line_max;
　　printf("%d:%s\n",LINE,str[line]);
　　printf("linemax=%d\n",line_max+1);


　　}else if(strchr(retsu,'x')){
　　　printf("終了\n");
　　　break;
　　}else if(gyo<=line_max && gyo>=0){
　　　line=gyo;
　　　printf("*%d:%s\n",LINE,str[line]);
　　　continue;
　　}else{
　　　printf("retsu:%s\n",retsu);
　　　printf("line:%d\n",line);
　　　printf("コマンドが不正です\n");
　　　continue;
　　}

　　}
　　　fclose(fp);
　　　exit(EXIT_SUCCESS);
　　}


///関数


int kansu (char* str,FILE* fp)
{
int i=254;
int r=0;

printf("str[i]==[%d]\n",(unsigned char)str[i]);
printf("str[i]==[%c]\n",(unsigned char)str[i]);
printf("str[i]==[0x%x]\n",(unsigned char)str[i]);

　　if((unsigned char)str[i]=='\n' || str[i]=='\0'){
　　　　 printf("改行、NULLを検知する\n");
　　　　　return 0;
　　}

　　else if((unsigned char)str[i]>=0 && (unsigned char)str[i]<=127){
　　　printf("00〜7Fに入りました\n");
　　　str[i+1]=10;
　　　str[i+2]=0;
　　　return 0;
　　}

　　　　
　　else if((unsigned char)str[i]>=224 && (unsigned char)str[i]<=239){printf("E0〜EFに入りました\n");
　　　str[i]=10;
　　　str[i+1]=0;
　　　r=fseek(fp,sizeof(char)*-1,SEEK_CUR);
　　　　if(r){
　　　　　　printf("seekに失敗しました\n");
　　　　　　return -1;
　　　}
　　　　　　return 0;
　　　}
　　　else if((unsigned char)str[i]>=128 &&(unsigned char)str[i]<=191){
　　　　printf("80~8Fに入りました\n");
　　　　　　if((unsigned char)str[i-1]>=128 &&(unsigned char)str[i-1]<=191){
　　　　　　str[i+1]=10;
　　　　　　str[i+2]=0;
　　　　　　return 0;
　　}
　　　else{
　　　　　　str[i-1]=10;
　　　　　　str[i]=0;
　　　　　　return 0;
　　　　　　r=fseek(fp,sizeof(char)*-2,SEEK_CUR);
　　　　　　if(r){
　　　　　　　　　　　printf("seekに失敗しました\n");
　　　　　　　　　　　return -1;
　　　　　　}
　　　　　　　　　　　return 0;
　　　　}
　}
             else if((unsigned char)str[i]>=224 && (unsigned char)str[i]<=239){
                 printf("E0〜EFに入りました\n");
                 str[i+1]=10;
                 str[i+2]=0;
                 return 0;

　　　　　　　}
}

環境はLinuxのubuntu上です。

Comments

[e-watt]

エラーの括弧内に「intとint*じゃん」と書いてあります。
確かにそういうコードになっているので、どこがそうなのかを見定めて、どう書けば目標が実現できるか考えましょう。

[dodox86]

main関数の冒頭 int *moji; と宣言してmoji=malloc(sizeof(int)*MOJI);としていますが、mojiがなぜint*なのか分かりませんし、そもそもint *mojiがそのあと使われていませんが。どういう意図があったのでしょうか。

[thkana]

実際に試したソースコードを提示して下さい。
質問のコードにはいわゆる全角空白が多数入っていて、コンパイルを通るはずがありません。

それはともかく...
そもそもの話としてデータを
char str[GYO][MOJI];
に格納しているのなら、malloc/reallocの出る幕ではありませんし、

#define LINE line + 1
line = malloc(sizeof(int) * LINE);
line = malloc(sizeof(int) * (LINE * ++line));
この3行にどれだけ「おかしなこと」が含まれているか...
これを見ただけでもあなたが自分の現在のスキルを大きく超えたことに挑戦しているとしか思えなくなります。挑戦はよいのですが、「無謀」にならないように適宜理解を積み重ねていって下さい。

Answer 1

malloc/reallocの目的は、行を追加したときに200行をはみ出るから、だったと思います。
「元々256の1行あたりの最大文字数だったのを、1024まで増やす」については、MOJIの定義を1024にしたことで片付いています。

まず、strの定義ですが、charの256バイトの文字列の配列へのポインタで定義します。
strはポインタで、mallocで獲得した領域のアドレスを設定します。
初期サイズは従来どおり「1バイト×MOJI×GYO」を確保します。

  int *moji;
  char str[GYO][MOJI];
  moji = malloc (sizeof (int) * MOJI);
  ：
  line = malloc (sizeof (int) * LINE);
  line = malloc (sizeof (int) * (LINE * ++line));

↓

  //int*  moji;
  char (*str)[MOJI];
  str = malloc (sizeof (char) * MOJI * GYO);
  ：
  //line = malloc (sizeof (int) * LINE);
  //line = malloc (sizeof (int) * (LINE * ++line));

line_maxは行番号ではなく行数で扱うようにしたほうが扱いやすいです。
条件を書くときに「< line_max」の未満か「>= line_max」の以上のいずれかで表現が統一しやすくなります。

  line--;
  int line_max = line;                //ファイル全体の行数

↓

  int line_max = line;                //ファイル全体の行数
  line--;

'a'の処理はfgetsで読み込んでからサイズ拡張では遅いので、サイズ拡張してから読み込むようにします。
reallocのサイズは単純に「1バイト×MOJI×line_max」にしました。

      else if (strchr (retsu, 'a'))
          ：
          line_max++;
          line++;
          fgets (str[line], sizeof (str[line]), stdin);
          moji = realloc (moji, sizeof (int) * (MOJI * ++moji));

↓

      else if (strchr (retsu, 'a'))
          ：
          line_max++;
          line++;
          str = realloc (str, sizeof (char) * MOJI * line_max);
          fgets (str[line], sizeof (str[line]), stdin);

Comments

[nekoatsume]

無事、実装できました。教えていただきありがとうございました！

Answer 2

Cのコードを書くなら、デバッグできる環境を整えましょう。
VisualStudioやEclipseなど、任意の行で実行を止め、変数の中身を確認できます。
また、１行づつ実行するようなこともできます。
そうすれば理解が早いのではないでしょうか

Comments

[episteme]

いやまだコンパイル通ってすらいないしwww

Answer 3

moji=realloc(moji,sizeof(int)(MOJI ++moji));

MOJIは 1024すなわちint
moji は int*

... int と int* を足して掛けてなにがしたいのですか?

Comments

[thkana]

intとint*の足し算ならともかく、intとint*を掛け算しようとしたからコンパイラに怒られたのでは?

[episteme]

ゴメン、"足して"じゃないや"掛けて"だ。(修正しました)