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

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

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

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

ファイル

ファイルとは、文字列に基づいた名前又はパスからアクセスすることができる、任意の情報のブロック又は情報を格納するためのリソースです。

GCC

GCCはGNU Compiler Collectionの略です。LinuxのC言語コンパイラのデファクトスタンダードであり、数多くの他言語やプラットフォームサポートもします。

Ubuntu

Ubuntuは、Debian GNU/Linuxを基盤としたフリーのオペレーティングシステムです。

Q&A

解決済

2回答

727閲覧

構造体配列の整列を可変長にしたい

murakumo_

総合スコア2

C

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

ファイル

ファイルとは、文字列に基づいた名前又はパスからアクセスすることができる、任意の情報のブロック又は情報を格納するためのリソースです。

GCC

GCCはGNU Compiler Collectionの略です。LinuxのC言語コンパイラのデファクトスタンダードであり、数多くの他言語やプラットフォームサポートもします。

Ubuntu

Ubuntuは、Debian GNU/Linuxを基盤としたフリーのオペレーティングシステムです。

0グッド

0クリップ

投稿2020/11/25 10:31

編集2020/11/25 11:14

前提・実現したいこと

番号とテストの得点が格納されたデータファイルのファイル名(20文字以内)をキーボードから入力し、基本交換法で並べ替える。
データファイルの例
1001 96
1002 60
1003 87
.
.
.

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

プログラム自体は動くのですが、入力したレコードに対してのみ整列を行いたいがその方法が思いつかない。
(入力したレコードに対してのみ整列を行い,未入力の領域は整列対象にしないようにしたい)
このプログラムだと100個以上のデータに対応できないので対応できるようにしたい。

該当のソースコード

#include <stdio.h> #include <stdlib.h> int main(void) { int n=0,cnt=0,flg,j,i; char filename[21]; FILE *fp; struct seisekiA { int bango,tokuten; };struct seisekiA seiseki[100],work; printf("ファイル名を入力してください\n"); scanf("%s",filename); if((fp=fopen(filename,"r"))==NULL) { printf("FILE OPEN ERROR\n"); exit(1); } printf("番号 得点\n"); while(fscanf(fp,"%d%d",&seiseki[n].bango,&seiseki[n].tokuten)!=EOF) { printf("%4d %4d\n",seiseki[n].bango,seiseki[n].tokuten); n++; } printf("レコード件数=%d\n",n); printf("整列結果\n"); flg = 0; while(flg == 0) { flg = 1; for(i=0; i<n-1; i++) { cnt++; if(seiseki[i].tokuten<seiseki[i+1].tokuten) { work = seiseki[i]; seiseki[i] = seiseki[i+1]; seiseki[i+1] = work; flg = 0; } } } fclose(fp); for(j=0;j<n;j++) { printf("%4d %4d\n",seiseki[j].bango,seiseki[j].tokuten); } printf("交換回数=%d\n",cnt); }

試したこと

10行目のseiseki[100]の部分を[n]にしてみましたがコアダンプになってしまいました。

補足情報(FW/ツールのバージョンなど)

Ubuntu 18.04.4TLS gcc 7.5.0

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

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

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

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

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

guest

回答2

0

100個以上のデータに対応させたいのであれば、動的メモリ確保(malloc,realloc,freeあたり)について調べてみてください。

投稿2020/11/25 12:27

actorbug

総合スコア2429

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

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

0

ベストアンサー

未入力の領域は整列対象にしないようにしたい

元からそうなっているのではないでしょうか?整列処理ではseiseki[n-1]までしかアクセスしませんので、残りは変化しません。

投稿2020/11/25 10:43

maisumakun

総合スコア146018

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

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

maisumakun

2020/11/25 11:19 編集

もっとも、このコードでは正しく整列できません…と思っていましたが、それは勘違いでした。
murakumo_

2020/11/25 11:15

どこの部分が悪いのか教えていただきたいです。
maisumakun

2020/11/25 11:19 編集

すみません、「正しく整列できない」というのは勘違いでした。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問