配列の数値を条件を指定して文字に置き換えたいがエラーになります

Question

1の場合は$で2の場合は{ }に置き換えたいのですがうまくいきません。
コアダンプとよくわからないエラーが発生しています。数値を文字列に置き換えているのでnum_p[3][4]ではなくnum_p[][]としてもうまくいきません。
解決策などありましたらアドバイス等をお願い致します。

#include <stdio.h>
int main()
{
  char num[3][4] = {{1, 2, 3, 4},{1, 3, 2, 4},{4, 3, 1, 2}};
  char *num_p[3][4];
  int i, j;

  num_p[3][4] = &num[0][0];

  for (i = 0; i < 3; i++) {
    for (j = 0; j < 4; j++) {
      printf("%5d", *num_p[i][j]);
      if (*num_p[i][j] == 1) {
        *num_p[i][j] = "$";
      } else if (*num_p[i][j] == 2) {
        *num_p[i][j] = "{　}";
      }
    }
    printf("\n");
  }

  return 0;
}

Comments

[cateye]

エラーメッセージは、そのまま（加工などせずに）貼り付けて下さい、

[gyoruo]

配列に何も指定しない場合は
array has incomplete element type 'char *[]'
char *num_p[][];
^
test.c:17:9: error: expected expression
num_p[][] = &num[0][0];
^
test.c:17:11: error: expected expression
num_p[][] = &num[0][0];
が表示され、配列の大きさをこちらで指定した場合は
test.c:23:22: warning: incompatible pointer to integer conversion assigning to 'char' from 'char [2]' [-Wint-conversion]
*num_p[i][j] = "$";

^ ~~~
test.c:25:22: warning: incompatible pointer to integer conversion assigning to 'char' from 'char [6]' [-Wint-conversion]
*num_p[i][j] = "{ }";
^ ~~~~~~
test.c:17:3: warning: array index 4 is past the end of the array (which contains 4 elements) [-Warray-bounds]
num_p[3][4] = &num[0][0];
^ ~
test.c:14:3: note: array 'num_p' declared here
char *num_p[3][4];
^
test.c:17:3: warning: array index 3 is past the end of the array (which contains 3 elements) [-Warray-bounds]
num_p[3][4] = &num[0][0];
^ ~
test.c:14:3: note: array 'num_p' declared here
char *num_p[3][4];
のような注意喚起が表示され、実行するとzsh: segmentation fault ./a.outと表示されます。

Answer 1

文字の配列の要素は「ひとつの文字」なので、文字列は設定できません。

とりあえず、ポインター変数をchar *num_p;にして、num_pを使う場面では[]の類を全部取り、内側のループでnum_p++で位置をずらしつつ、"$"を'$'に、"{　}"は(1文字しか設定できないので)'{'にすれば、「配列の数値を文字に置き換える」というプログラムにはなりますね。

ついでに、元のコードだと、置き換えた後の状態を確認できないので、printfの位置を後ろに移動。

#include <stdio.h>

int main(void)
{
  char num[3][4] = {{1, 2, 3, 4}, {1, 3, 2, 4}, {4, 3, 1, 2}};
  char *num_p;

  num_p = &num[0][0];

  for (int i = 0; i < 3; i++) {
    for (int j = 0; j < 4; j++) {
      if (*num_p == 1) {
        *num_p = '$';
      } else if (*num_p == 2) {
        *num_p = '{';
      }
      printf("%5d", *num_p);
      num_p++;
    }
    printf("\n");
  }

  return 0;
}

Comments

[gyoruo]

ご回答ありがとうございます。
Daregadaさんのプログラムでできたのですが、文字も数値になってしまっていたので修正方法を考えてみます！

[Daregada]

いや、文字もコンピューターの内部では数値ですからね。
表示方法を変えればいいのですが、そうすると 3 や 4 が制御文字(普通に表示できない)になってしまうので、端末画面には文字が表示されないか、おかしな表示になるでしょう。

if (*num_p < ' ') {
printf("%5d", *num_p);
} else {
printf("%5c", *num_p);
}
とでもしておけばいいのですが。

[gyoruo]

ご回答ありがとうございます！
参考にさせていただきます！

Answer 2

usr ~/Project/test % cc t3.c
t3.c:13:30: error: subscripted value is not an array, pointer, or vector
      printf("%5d", *num_p[i][j]);
                     ~~~~~~~~^~
t3.c:14:20: error: subscripted value is not an array, pointer, or vector
      if (*num_p[i][j] == 1) {
           ~~~~~~~~^~
t3.c:15:18: error: subscripted value is not an array, pointer, or vector
        *num_p[i][j] = "$";
         ~~~~~~~~^~
t3.c:16:27: error: subscripted value is not an array, pointer, or vector
      } else if (*num_p[i][j] == 2) {
                  ~~~~~~~~^~
t3.c:17:18: error: subscripted value is not an array, pointer, or vector
        *num_p[i][j] = "{　}";
         ~~~~~~~~^~
5 errors generated.

num_p = &num[0][0];→*num_p[i][j] ←ポインタの配列ではありません。
文字列 "$"、"{　}"は、アドレスに変換されます。
で、char型の変数に複数の文字（"{　}"：UTF-8で5バイト）を入れることは出来ません。
・・・cの構文や演算子のを勉強しましょう。

追記：修正してみましたが、num[i][j] = "{　}";の意図が分かりません？
（やりたいことは分かりますが、出来ないのでどうしますか？）
・・・「char1個に半角文字1個しか入らない」は分かっているんですよね？

#include <stdio.h>

int main()
{
  char num[3][4] = {{1, 2, 3, 4},{1, 3, 2, 4},{4, 3, 1, 2}};
  char *num_p;
  int i, j;

  num_p = &num[0][0];

  for (i = 0; i < 3; i++) {
    for (j = 0; j < 4; j++) {
      printf("%5d", num[i][j]);
      if (num[i][j] == 1) {
        num[i][j] = '$';
      } else if (num[i][j] == 2) {
        num[i][j] = "{　}";
      }
    }
    printf("\n");
  }

  return 0;
}

t3.c:17:19: warning: incompatible pointer to integer conversion assigning to 'char' from 'char [6]' [-Wint-conversion]
        num[i][j] = "{　}";
                  ^ ~~~~~~
1 warning generated.

Comments

[gyoruo]

ご指摘ありがとうございます！
色々と見直してみます！

[gyoruo]

修正までしていただき、ありがとうございます。
配列1つに"{"しか入らないのは理解しています。
今はどうやって実現しようか考えています。

Answer 3

  printf("%5d", *num_p[i][j]);

num_p って変数の型みてみよう。
全然違いますがな

Comments

[gyoruo]

早急なアドバイスありがとうございます！
次はコアダンプで領域破壊を起こしているらしく、数値を文字列に置き換える時のnum_p[][]の[][]はなぜ空だとエラーになってしまうのでしょうか？

[y_waiwai]

num_p の型をみてください。
ポインタの理解ができていないようなので、まずはそこらへんから勉強し直しましょう。

コードを修正したようですが、これでもだめです。

[gyoruo]

ご指摘ありがとうございます！
どこがわからないのかもわからなかったのでありがたいです。
見直してみます！