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

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

質問する

ただいまの
回答率

90.75%

  • JavaScript

    15275questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Python 3.x

    5308questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

  • Google Apps Script

    716questions

    Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

自作関数の中で別の自作関数を使う場合のリスクについて

解決済

回答 3

投稿 ・編集

  • 評価
  • クリップ 0
  • VIEW 178
pypanman

pypanman

score 6

ご覧いただきありがとうございます。

質問ですが、自作関数(関数Aとする)の中で、別の自作関数(関数Bとする)を使う場合、
関数Bを変更・削除した場合、関数Aが動かなくなるリスクがあります。

下記のような場合だと、関数Aを削除すると、関数Bが動作しなくなります。

このようなリスクを防止するためには、どのような対策をするのが一般的なのでしょうか？ 

function A(num1, num2){
 return num1 + num2;
}

function B(num){
 var tmp = A(num, num);
 print(tmp);
}

function main(){
 var magicNumber = 10;
 B(magicNumber);
}

質問の意図ですが、ソースコードのマネジメント方法的なものを知りたい、といったところです。
例えば、「自作関数は言語に標準的に用意された関数やライブラリのみ使用するのが通例」といったルールがあれば教えて欲しいです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

more_vert3 件の質問への追記・修正依頼

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

  • hayataka2049

    hayataka2049

    2018/05/25 11:00

    具体的にどのような状況を想定しているのか、簡単なコード例を示して頂けるとわかりやすいと思います

    キャンセル

  • fuzzball

    fuzzball

    2018/05/25 11:31

    質問内容と、最後の「自作関数は言語に標準的に用意された関数やライブラリのみ使用するのが通例」の繋がりが分からないのですが。

    キャンセル

  • HayatoKamono

    HayatoKamono

    2018/05/25 11:36

    「関数Aを削除すると」について、もう少し詳しく教えて下さい。

    キャンセル

回答 3 件

link

checkベストアンサー

+2

具体的なコードがないので、抽象的な回答になりますが、

そうなる、というのは、関数Bの動作に、関数Aの動作が関わっているからそうなるということです。
そうならないようにするには、関数Bを独立させ、関数Aの影響を受けないようにすればいいということになりますねー

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/25 11:19

    ご回答ありがとうございます。

    自作関数は、言語に標準的に用意された関数・ライブラリのみで構成するのが一般的なのでしょうか？
    まぁ、開発の規模によるのかもしれませんけど。

    キャンセル

  • 2018/05/25 11:30

    その開発中のソースで標準的に用意した変数、関数で構成すればいいのです。
    そういう関数は、当然ながら、他が使用していることを考慮し、修正には考慮が必要となります。
    気軽に削除したり修正するもんでもありません。
    他の回答にもありますが、そういう標準的に使用する関数、変数は、気軽に修正できない／しないようにモジュールやクラスでまとめておく、ということをして管理しますねー

    そして、そのようなクラス／モジュールに手を入れる場合は、他の開発メンバーに周知し、承認をうけるような仕組みが必要となります

    キャンセル

  • 2018/05/25 11:47

    まー、自分ひとりでやってる場合は、関数を削除するリスクよりも、バグを仕込むリスクの方を気にするべきかと。
    自分のプログラムは、自分が神様、すべては自分が作り出すもんですから。

    キャンセル

link

+1

互いに依存関係にあるようなもの、外部の変数を共有するようなものはモジュールやクラスとしてまとめておき、その単位で責任を持って保守するとか、テストを書くとかするようにします。
あとはオブジェクト指向や関数型プログラミングの技法を使って、できるだけ結合が疎になるようにコードを書き、そういう事態に陥る可能性を減らしておくことで予防的に対策できます。

関数Aを変更・削除した場合、関数Bが動かなくなるリスクがあります。

AとBが逆なのでは。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/25 11:17

    ご回答ありがとうございます。

    >AとBが逆なのでは。
    おっしゃる通りでございます。

    キャンセル

  • 2018/05/25 11:32

    ある程度は技法（オブジェクト指向・関数型・テスト駆動）でなんとかなりますが、使わなといけないライブラリの仕様で・・・とか他の人の書いたコードで・・・とかそもそも設計が・・・とか止むに止まれぬ場合もあるでしょうから、最後は注意と精神力、つまり気力です。
    気力は大切なリソースなので、できるだけ無駄に消耗しないように工夫します。たとえば、言葉の説明ではBを削除する話をしているのに、コード例はAを削除する・・・というのは悪い例です。注意書きを書いても、それを見落とされたら困るので。
    あとは、どうせ把握しきれないのでIDE等の支援に頼るという考え方もあります。削除したり変更したりした瞬間に「この関数が未定義で/型エラーでコンパイル/実行できなくなるよ」と言ってくれるものがあれば良い訳で。ただ、IDE頼り切りでけっきょく使いこなせない、というのも悲喜劇を産みかねないので、考えものですけど

    キャンセル

link

0

自作関数(関数Aとする)の中で、別の自作関数(関数Bとする)を使う場合、
関数Aを変更・削除した場合、関数Bが動かなくなるリスクがあります。

ありません。関数Bは、一般的には関数Aがなくても動作します。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/25 11:17

    ご回答ありがとうございます。
    AとBが逆でした。

    キャンセル

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

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

関連した質問

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

  • JavaScript

    15275questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Python 3.x

    5308questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

  • Google Apps Script

    716questions

    Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

SNSアカウントでログイン

  • Facebookでログイン
  • Twitterでログイン
  • Googleでログイン
  • Githubでログイン
  • Hatenaでログイン

teratailアカウントでログイン

思考するエンジニアのためのQ&Aサイト「teratail」について詳しく知る