Skip to main content
編集済みの本文
ソース リンク
kunif
  • 1.8万
  • 3
  • 18
  • 28

「短く実行できる形」にするには、forループを1つだけにすることです。

なお、質問の文言にある課題の表示内容に限って言えば、構造体を使う必要はありません。
ポインタも目に見える形ではほとんど使う必要はありません。

ちなみに質問の元のソースは、コメントにも指摘されている以下の理由で、きつい言い方になりますが、ほぼすべてが考慮するに値しない内容です。

  • 学生情報構造体の配列宣言が、ポインタの配列になっていて、実際のデータ用領域が確保されていない。
    かつ領域が確保されていないのに使われている。(下記理由でビルドも通らないが、ビルドできたとしてもメモリアクセス違反で異常終了する)
  • 内容の不明なinta,b,c,d という名前の変数が宣言されていて、かつプログラム上は構造体へのポインタだったり、構造体のメンバー扱い? だったりして整合性が取れていない。
    (そのため、まずビルドが正常に出来ない)
  • 3つのforループのどれも終了条件が指定されていない。
  • forループの中のif文の判定条件や、その真偽時両方の処理が意味不明。

課題なので実際のプログラムソースは提示しない方が良いでしょう(けれどほとんど提示しているも同然ですが)から、以下の内容を元にプログラムを組んでみてください。

  • 入力データ数の最大値(10)は定数マクロとして定義しておく。
  • 学生情報構造体の配列は、ポインタではなく、実際の領域を持った構造体の配列として宣言する。
  • 以下を合わせて13個の変数を用意する(課題内容に応じてintだけではなくdouble(身長?・体重?)も)
    • ループカウンタ
    • 有効な入力データ数(初期値は10)
    • 学籍番号・身長・体重の入力用で3個
    • 身長・体重の最高値/最低値およびそれぞれに対応する学籍番号で8個
      それぞれの値は、あり得ない数値で初期化しておく(学籍番号および最高値は-1,最低値は999999等)
  • forループで以下の処理を行う
    • for継続/終了条件(ループカウンタが10未満は継続,10以上は終了)は指定しておく
    • 入力ガイドのプロンプト表示printf
    • 学籍番号・身長・体重の入力用変数をパラメータにした入力値の読み取りscanf
    • 負の学生番号入力による終了判定を行い、途中終了の場合は有効な入力データ数を更新してループ脱出
    • 入力データのそれぞれを学生情報構造体の配列に格納
    • 身長・体重の最高値/最低値が更新されるか判定を行い、更新されるならば、その数値と学籍番号を入れ替える
  • forループを終了して、有効なデータが1個以上あれば、課題の身長・体重の最高値/最低値およびそれぞれに対応する学籍番号を表示するprintf

「短く実行できる形」にするには、forループを1つだけにすることです。

なお、質問の文言にある課題の表示内容に限って言えば、構造体を使う必要はありません。
ポインタも目に見える形ではほとんど使う必要はありません。

ちなみに質問の元のソースは、コメントにも指摘されている以下の理由で、きつい言い方になりますが、ほぼすべてが考慮するに値しない内容です。

  • 学生情報構造体の配列宣言が、ポインタの配列になっていて、実際のデータ用領域が確保されていない。
    かつ領域が確保されていないのに使われている。(下記理由でビルドも通らないが、ビルドできたとしてもメモリアクセス違反で異常終了する)
  • 内容の不明なinta,b,c,d という名前の変数が宣言されていて、かつプログラム上は構造体へのポインタだったり、構造体のメンバー扱い? だったりして整合性が取れていない。
    (そのため、まずビルドが正常に出来ない)
  • 3つのforループのどれも終了条件が指定されていない。
  • forループの中のif文の判定条件や、その真偽時両方の処理が意味不明。

課題なので実際のプログラムソースは提示しない方が良いでしょう(けれどほとんど提示しているも同然ですが)から、以下の内容を元にプログラムを組んでみてください。

  • 入力データ数の最大値(10)は定数マクロとして定義しておく。
  • 学生情報構造体の配列は、ポインタではなく、実際の領域を持った構造体の配列として宣言する。
  • 以下を合わせて13個の変数を用意する(課題内容に応じてintだけではなくdouble(身長?・体重?)も)
    • ループカウンタ
    • 有効な入力データ数(初期値は10)
    • 学籍番号・身長・体重の入力用で3個
    • 身長・体重の最高値/最低値およびそれぞれに対応する学籍番号で8個
      それぞれの値は、あり得ない数値で初期化しておく(学籍番号および最高値は-1,最低値は999999等)
  • forループで以下の処理を行う
    • forの終了条件(ループカウンタが10未満)は指定しておく
    • 入力ガイドのプロンプト表示printf
    • 学籍番号・身長・体重の入力用変数をパラメータにした入力値の読み取りscanf
    • 負の学生番号入力による終了判定を行い、途中終了の場合は有効な入力データ数を更新してループ脱出
    • 入力データのそれぞれを学生情報構造体の配列に格納
    • 身長・体重の最高値/最低値が更新されるか判定を行い、更新されるならば、その数値と学籍番号を入れ替える
  • forループを終了して、有効なデータが1個以上あれば、課題の身長・体重の最高値/最低値およびそれぞれに対応する学籍番号を表示するprintf

「短く実行できる形」にするには、forループを1つだけにすることです。

なお、質問の文言にある課題の表示内容に限って言えば、構造体を使う必要はありません。
ポインタも目に見える形ではほとんど使う必要はありません。

ちなみに質問の元のソースは、コメントにも指摘されている以下の理由で、きつい言い方になりますが、ほぼすべてが考慮するに値しない内容です。

  • 学生情報構造体の配列宣言が、ポインタの配列になっていて、実際のデータ用領域が確保されていない。
    かつ領域が確保されていないのに使われている。(下記理由でビルドも通らないが、ビルドできたとしてもメモリアクセス違反で異常終了する)
  • 内容の不明なinta,b,c,d という名前の変数が宣言されていて、かつプログラム上は構造体へのポインタだったり、構造体のメンバー扱い? だったりして整合性が取れていない。
    (そのため、まずビルドが正常に出来ない)
  • 3つのforループのどれも終了条件が指定されていない。
  • forループの中のif文の判定条件や、その真偽時両方の処理が意味不明。

課題なので実際のプログラムソースは提示しない方が良いでしょう(けれどほとんど提示しているも同然ですが)から、以下の内容を元にプログラムを組んでみてください。

  • 入力データ数の最大値(10)は定数マクロとして定義しておく。
  • 学生情報構造体の配列は、ポインタではなく、実際の領域を持った構造体の配列として宣言する。
  • 以下を合わせて13個の変数を用意する(課題内容に応じてintだけではなくdouble(身長?・体重?)も)
    • ループカウンタ
    • 有効な入力データ数(初期値は10)
    • 学籍番号・身長・体重の入力用で3個
    • 身長・体重の最高値/最低値およびそれぞれに対応する学籍番号で8個
      それぞれの値は、あり得ない数値で初期化しておく(学籍番号および最高値は-1,最低値は999999等)
  • forループで以下の処理を行う
    • for継続/終了条件(ループカウンタが10未満は継続,10以上は終了)は指定しておく
    • 入力ガイドのプロンプト表示printf
    • 学籍番号・身長・体重の入力用変数をパラメータにした入力値の読み取りscanf
    • 負の学生番号入力による終了判定を行い、途中終了の場合は有効な入力データ数を更新してループ脱出
    • 入力データのそれぞれを学生情報構造体の配列に格納
    • 身長・体重の最高値/最低値が更新されるか判定を行い、更新されるならば、その数値と学籍番号を入れ替える
  • forループを終了して、有効なデータが1個以上あれば、課題の身長・体重の最高値/最低値およびそれぞれに対応する学籍番号を表示するprintf
ソース リンク
kunif
  • 1.8万
  • 3
  • 18
  • 28

「短く実行できる形」にするには、forループを1つだけにすることです。

なお、質問の文言にある課題の表示内容に限って言えば、構造体を使う必要はありません。
ポインタも目に見える形ではほとんど使う必要はありません。

ちなみに質問の元のソースは、コメントにも指摘されている以下の理由で、きつい言い方になりますが、ほぼすべてが考慮するに値しない内容です。

  • 学生情報構造体の配列宣言が、ポインタの配列になっていて、実際のデータ用領域が確保されていない。
    かつ領域が確保されていないのに使われている。(下記理由でビルドも通らないが、ビルドできたとしてもメモリアクセス違反で異常終了する)
  • 内容の不明なinta,b,c,d という名前の変数が宣言されていて、かつプログラム上は構造体へのポインタだったり、構造体のメンバー扱い? だったりして整合性が取れていない。
    (そのため、まずビルドが正常に出来ない)
  • 3つのforループのどれも終了条件が指定されていない。
  • forループの中のif文の判定条件や、その真偽時両方の処理が意味不明。

課題なので実際のプログラムソースは提示しない方が良いでしょう(けれどほとんど提示しているも同然ですが)から、以下の内容を元にプログラムを組んでみてください。

  • 入力データ数の最大値(10)は定数マクロとして定義しておく。
  • 学生情報構造体の配列は、ポインタではなく、実際の領域を持った構造体の配列として宣言する。
  • 以下を合わせて13個の変数を用意する(課題内容に応じてintだけではなくdouble(身長?・体重?)も)
    • ループカウンタ
    • 有効な入力データ数(初期値は10)
    • 学籍番号・身長・体重の入力用で3個
    • 身長・体重の最高値/最低値およびそれぞれに対応する学籍番号で8個
      それぞれの値は、あり得ない数値で初期化しておく(学籍番号および最高値は-1,最低値は999999等)
  • forループで以下の処理を行う
    • forの終了条件(ループカウンタが10未満)は指定しておく
    • 入力ガイドのプロンプト表示printf
    • 学籍番号・身長・体重の入力用変数をパラメータにした入力値の読み取りscanf
    • 負の学生番号入力による終了判定を行い、途中終了の場合は有効な入力データ数を更新してループ脱出
    • 入力データのそれぞれを学生情報構造体の配列に格納
    • 身長・体重の最高値/最低値が更新されるか判定を行い、更新されるならば、その数値と学籍番号を入れ替える
  • forループを終了して、有効なデータが1個以上あれば、課題の身長・体重の最高値/最低値およびそれぞれに対応する学籍番号を表示するprintf