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

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

新規登録して質問してみよう
ただいま回答率
85.49%
C

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

Cygwin

Cygwinは、Unixのような環境を、Windows上で構築させるコマンドラインインターフェースです。

Q&A

解決済

1回答

1167閲覧

<c言語>malloc関数を用いて作成した文字列を管理するポインタ配列を実現したいです

iisan

総合スコア5

C

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

Cygwin

Cygwinは、Unixのような環境を、Windows上で構築させるコマンドラインインターフェースです。

0グッド

0クリップ

投稿2021/10/30 06:04

malloc関数を用いて作成した文字列を管理するポインタ配列を実現したい

fgets関数により、ファイルの中身(文字列)を受けとり、受け取った文字列を格納する配列をmalloc関数で作成する。
上記を繰り返しできた配列たちをポインタ配列に入れていき、最後に表示する。というのが実現したい内容です。

現在、ファイルの中身の最終行のみ受け取ることができている状態です。
よろしくお願いいたします。

発生している問題

入力ファイル 0 AAL - Aalborg : Denmark Aalborg 1 AAR - Aarhus : Denmark Tirstrup 2 ABI - Abilene : TX : USA Municipal 3 ABJ - Abidjan : Cote D'ivoire Port Bouet 4 ABR - Aberdeen : SD : USA Aberdeen Regional Airport 5 ABZ - Aberdeen : Scotland : United Kingdom Dyce 6 AES - Aalesund : Norway Vigra 7 AHB - Abha : Saudi Arabia - Abha Airport 答え 0 AAL - Aalborg : Denmark Aalborg 2 ABI - Abilene : TX : USA Municipal 4 ABR - Aberdeen : SD : USA Aberdeen Regional Airport 6 AES - Aalesund : Norway Vigra 1 AAR - Aarhus : Denmark Tirstrup 3 ABJ - Abidjan : Cote D'ivoire Port Bouet 5 ABZ - Aberdeen : Scotland : United Kingdom Dyce 7 AHB - Abha : Saudi Arabia - Abha Airport 実際の出力 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport 7 AHB - Abha : Saudi Arabia - Abha Airport

該当のソースコード

#include <stdio.h> #include <stdlib.h> #include <string.h> #define LINELEN 128 int main(void) { int i, count; char **Line[LINELEN]={0}; char s[LINELEN]; char *clone=NULL; i = count = 0; while (fgets(s, LINELEN, stdin) != NULL) { clone = (char *)malloc(sizeof(char)*strlen(s)+1); strcpy(clone, s); Line[i] = &clone; if(i>=LINELEN) { fprintf(stderr, "行数がサイズを超えました\n"); break; } i++; count = i; } fprintf(stderr, "%d", count); for(i=0;i<count;i = i+2) { printf("%s\n", *Line[i]); } for(i=1;i<count;i = i+2) { printf("%s\n", *Line[i]); } free(clone); clone = NULL; return 0; }

補足情報

fgets関数で入力しなければならない、という決まりがあります。
Cygwinによりコンパイルしています。

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

Line[i] = &clone;とすべてに同じ値を入れているからでしょう。

char *Line[LINELEN];Line[i] = clone;では?

あと、printfもおかしいのと、名前からして行の最大長を意図していると思われるLINELENを、保存する行数の上限と兼用しているのも変ですが、意図通りなのでしょうか?

投稿2021/10/30 06:32

otn

総合スコア84491

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

iisan

2021/10/30 06:42

回答ありがとうございます。参考にした結果解決しました。 Line配列にLINEKENを用いているのが不自然ということでしょうか?
otn

2021/10/30 06:47

LINEKEN というのが行の件数という意味ならそういう名前を定義して使えばいいかと思います。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.49%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問