質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.91%

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

受付中

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 201

gyoruo

score 59

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;
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • cateye

    2020/07/05 10:54

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

    キャンセル

  • gyoruo

    2020/07/05 10:58

    配列に何も指定しない場合は
    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と表示されます。

    キャンセル

回答 3

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/05 10:53

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

    キャンセル

  • 2020/07/05 11:01

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

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

    キャンセル

  • 2020/07/05 11:16

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

    キャンセル

0

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.

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/05 11:16

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

    キャンセル

  • 2020/07/05 11:32

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

    キャンセル

0

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

とりあえず、ポインター変数を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;
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/05 11:38

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

    キャンセル

  • 2020/07/05 12:15

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

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

    キャンセル

  • 2020/07/12 17:29

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.91%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る