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

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

ただいまの
回答率

90.50%

  • C

    3700questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

テキストファイルを逆順に出力

解決済

回答 2

投稿

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

hatena777

score 1

実現したいこと

mallocとreallocを使ってテキストファイルの記述を一行ずつ、逆順でもう一つのテキストファイルに出力させようとしているのですが、エラーがでて一向にできず困っています。

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

0xC0000005: 場所 0xCDCDCDCD の読み取り中にアクセス違反が発生しました。

該当のソースコード

int main()
{
    FILE *fp_R;
    FILE *fp_W;
    char buff[256];
    char **ptr = (char**)malloc(0);
    fopen_s(&fp_R, "略", "r");
    fopen_s(&fp_W, "略", "w");

    if (fp_R == NULL) {
        puts("ファイルのオープンに失敗しました。");
        exit(-1);
    }

    if (fp_W == NULL) {
        puts("ファイルのオープンに失敗しました。");
        exit(-1);
    }

    int i = 0;
    while (fgets(buff, 100, fp_R) != NULL) {
        char *p = (char *)malloc(strlen(buff) + 1);
        strcpy_s(p, strlen(buff) + 1,buff);
        ptr[i] = p;
        ++i;
        ptr = (char **)realloc(ptr, sizeof(char *)* i);
    }
    --i;
    while (i >= 0) {
        fprintf(fp_W,"%s\n",ptr[i]);
        --i;
    }

    free(ptr);

    fclose(fp_R);
    fclose(fp_W);
    return 0;
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+2

すでに解決済みになっていますが、おそらく解決していないと思われるので、詳しく説明しておきます。

ファイル読み込みループの中の

ptr[i] = p;
++i;
ptr = (char **)realloc(ptr, sizeof(char *)* i);


これだと領域の確保と利用の順番が逆になっています。確保してからでないと利用できません。
また、領域のサイズをsizeof(char *)* iとしていると、1件目、つまりi == 0のときにサイズが0となってしまうので、件数でサイズを計算するためにsizeof(char *) * (i + 1)としないといけません。

それらを踏まえて、このようにすれば良いでしょう。

ptr = (char **)realloc(ptr, sizeof(char *) * (i + 1));
ptr[i] = p;
++i;


それと、最初のptrの初期化に(char**)malloc(0)は無意味(にもかかわらず意味ありげに思えてしまい誤解を与える危険性がある)なので、単純にNULLで初期化すれば良いです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/22 13:22

    詳しく説明して頂きありがとうございました。とてもわかりやすくて、すぐに解決することができました。

    キャンセル

checkベストアンサー

+1

領域を確保していないからです。

char **ptr = (char**)malloc(0);

reallocを使っているようですが、使う前に範囲外で落ちています。
また毎周reallocを使うのはちょっとコストが高すぎなんじゃないか、とも思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/21 23:01

    mallocとrealloc覚えたてでよくわかっていませんでした。

    キャンセル

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

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

関連した質問

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

  • C

    3700questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。