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

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

ただいまの
回答率

90.52%

  • C

    3686questions

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

  • 関数

    218questions

    関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。

C言語:関数定義の場所について

解決済

回答 5

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 440

cresc

score 8

C言語で関数定義をする際に、たいていの参考書ではmain関数の外で定義されてますが、C言語で書くプログラムのコードが膨大になるにつれて、関数定義も増えてくるので、main関数の外で定義すると、main関数と関数定義間を行ったり来たりする際にまたぐ行数が多くなるため、コードを読むのに時間がかかると思うので、main関数内の初めて自作の関数を使う直前に関数定義をした方がいいと思うのですが、この書き方は良くないですか?

#include <stdio.h>

int main(){
    int na, nb;

    puts("二つの整数を入力してください。");
    printf("整数1 : ");
    scanf("%d", &na);
    printf("整数2 : ");
    scanf("%d", &nb);

    int maxof(int x, int y){
        if (x > y)
            return (x);
        else
            return (y);
    }    
    printf("大きい方の値は%dです。\n", maxof(na, nb));

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • ttyp03

    2018/01/17 10:00

    main関数内に定義するの意味がわからないのですが、具体例を書いていただけますか。

    キャンセル

  • cresc

    2018/01/17 21:01

    具体例のコードを追加しました。

    キャンセル

回答 5

+6

残念ながら、標準的なC言語では、関数の外でしか関数を定義できません。関数内関数は、GCCなど一部のコンパイラの拡張です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/17 10:02

    あ、僕はてっきり「main()の定義されたファイルの外(=別ファイル)にそいつが呼んでる関数を...」って言いたいのかと思ってた。

    キャンセル

  • 2018/01/17 19:55

    そうだったのですか。main関数内に定義してもちゃんと動いたのでC言語は関数内関数の機能があると思ってたのですが、使っているコンパイラがたまたま関数内関数の機能が拡張されてたということなのですか?

    キャンセル

  • 2018/01/17 21:28

    多分そうだと思います(たとえば、clangやVisual C++では動かないようです)。

    キャンセル

  • 2018/01/18 00:25

    なるほど、それではコンパイラによって実行できなくなる危険があるからmain関数に関数は定義しない方がいいですね。回答ありがとうございました。^^

    キャンセル

checkベストアンサー

+2

そもそも論ですが...
『その場で定義した方が読みやすい』≒『一度しか呼ばない』関数を書くのはほとんど無駄です。
適切にコメントを付ければべた書きしてもいいんじゃないでしょうか。

コールバック関数として定義する場合、今後再利用が望まれる場合は除きます。


また、コードが膨大になってあちこち飛び回る必要があるのは、構造化が弱いせいかもしれません。
責任に着目して適切にモジュールを分割、さらに適切に命名/コメントをしてください。
関数の中身を読み返す機会はそこまで増えないかと。

追記

誤解を招いてしまったようなので...

手続き的な処理をとにかく関数に細分化しようとする人がいます。
特にひどいときは、void hoge(void)みたいな関数が乱立されているのです。
『いったりきたり』という質問内容でしたので、このような状況を想定してしまいました。

適切に機能を切り分けること、つまり強度/結合度が担保されていることが重要です。
これが保たれ、同時に適切に命名出来るのなら、むしろ関数に切り分けるべきでしょう。

詭弁に逃げるようで恐縮ですが、機能の切り分けは『多過ぎはダメ』『少なすぎもダメ』です。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/17 20:50

    回答ありがとうございます。
    おっしゃる通り「一度しか呼ばない関数」を定義するのだったら適切なコメントを付けた方が綺麗なコード書けそうですね。
    関数を作りすぎるのは構造化がしっかりできてなかったという考えは今までなかったので、とても勉強になりました。これからは構造化がしっかりできてるかということも気を付けて書いてみます。

    キャンセル

  • 2018/01/17 21:11 編集

    質問のコードを見ましたが、この程度なら条件演算子を使えば良さそうです。
    printf("大きい方の値は%dです。\n", (na > nb) ? na : nb);

    濫用するととんでもなく読みづらくなるので注意は必要ですが。

    キャンセル

  • 2018/01/18 00:28

    質問のコードはmain関数内に関数を定義することのいみがわからないとのことだったので例としてあげたものです。私の想定している状況は大きなプログラムですので、LouiS0616さんの回答でしっかりと私の疑問の回答になってました。

    キャンセル

+1

私見では「好きにすれば?」です。
いまどきの開発環境なら呼んでる関数にジャンプする機能がアタリマエに用意されてますから。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

C++になってしまいますが、短い関数なのにいちいち外出ししないといった制約をどうにかするためにラムダ式があります。
C++も許容されているならばあまり書き直しがないのでC++にしてラムダ式で書くといった手もあります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/17 20:16

    回答ありがとうございます。
    今回私が想定しているのは短い関数ではなく、関数定義全般に共通しての話です。C++はまだ学習したことがないため、C++の学習をしましたら、ラムダ式も使ってみようと思います。

    キャンセル

0

標準のC言語では、関数外でしか定義できません。
関数定義が増えてきたなら、機能や処理別に
別ファイルに定義して、コンパイル/リンクするだけです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

  • 受付中

    C言語を用いて、カレンダーを作成したい。

    前提・実現したいこと ある月の日数と、一日の曜日を入力させて、その月のカレンダーを例のように出力するプログラムを作る。 <例> 一か月の日数: 31 一日目の曜日

  • 解決済

    打ち込んだ数値をすぐに表示したい。

    今一度お願いいたします。 void pos_training(void)の /* ブロックを選択させる */ do { printf("番号(練習中止は99)

  • 解決済

    3つの文字列の長さを比較したい

    こんにちは C言語についての質問ですが キーボードから入力された3つの文字列の長さを比較して一番長い文字列をprintf()で表示させたいのですがこの場合strcmp関数で比較する

  • 解決済

    AOJの問題。2行目に、指定された数のデータを一度にinputする方法がわからない。

    前提・実現したいこと 只今、PCKに向けてAOJを頑張っている高専生です。 言語はCを使っています。 そこで質問なのですが、 1行目に入力するデータ(整数)の数を提示し、 2行目に

  • 解決済

    C言語でわからないこと

    include <stdio.h> float calc(char op, float a, float b); main(){ float number; number 

  • 解決済

    C言語学習中の高校生です。

    前提・実現したいこと 複数の入力での計算がしたい 発生している問題・エラーメッセージ 「9行目」で記述エラーを発見しました。 「identifier」を付け忘れています。

  • 解決済

    c言語 入門レベル問題

    a,b 入力したら、b、a 逆の順番に出てたいです、 下記プログラミング7,8入力したら、a = 0, b = 7 出てしまいました。 不備なところ教えていただけますか。 #i

  • 解決済

    あと少しなんですがうまくいきません、、、

    C C言語で 各学生の学籍番号・物理の点数・化学の点数の組がスペース区切りで一行に記述されたファイルを読み込み,全学生の物理の平均点・最高点・最低点,および化学の平均点・最高点・最

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

  • C

    3686questions

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

  • 関数

    218questions

    関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。