独自定義関数から処理を中断するには？

Question

例えば、test()から
console.log("sample2")を実行しない為に
働きかけることはできるのでしょうか？
もし、可能であればこの動作の名称を教えていただけると助かります。
よろしくお願いいたします。

console.log("sample")
test();
//↑戻り値で以下の処理続行可否を決めるのではなく
//関数内で処理中断できないでしょうか？
console.log("sample2")


function test(){	
	return //ここで処理を中断したい
}

環境 windows 10 64bit

Answer 1

console.log("sample");
test();
console.log("sample2");//動作しない
console.log("sample3");//動作する


function test(){
    let temp = console.log;
    console.log =function(){console.log = temp};
    //console.log を捨てたので、次の1回は動作しない。その次からは戻されるので動作する。
}

Comments

[miyabi-sun]

こ、これは凄い解決方法だ……

[deep]

トリッキーな方法ありがとうございます。
驚きました。。

Answer 2

そんな強制終了のようなものはロジックがおかしいと思いますが
あえてやるならtry-catchで例外をthrowするとかでしょうか

try{
  console.log("sample");
  test();
  console.log("sample2");
}catch(e){
  
}
function test(){
  throw 1;
}

Comments

[deep]

戻り値で後続するか判定するのは
複数のtest()呼び出し元がある場合に
コードを追加するのが少し面倒だと感じたので

後続しない場合、関数内で処理を完結できたらと思い
質問させていただきました。
この考え方はロジック的に問題がありますでしょうか？

Answer 3

throw 文

それがエラーであるなら、エラーメッセージを出力して強制終了させることは可能です。

function test () {
  throw new Error('fooはbarでなければなりません');
}

例外キャッチか戻り値判定の２択なのですね。

x_x さんが仰るように、try-catch は使用しません。
throw はエラーをコンソールに出力してその場で即座に強制終了する為の機構です。
JavaScript には「不正な引数であった場合に強制終了する仕組み」があります。

(2).toString(-1); // RangeError: toString() radix argument must be between 2 and 36

return 文

エラーでないのなら、返り値から判断して return 文で終了させるか、

(function () {
  console.log("sample");

  var resutl = test();

  if (!result) {
    return;
  }

  console.log("sample2");
}());

返り値から後述コードを条件付き実行にするのが正しいアプローチだと思います。

(function () {
  console.log("sample");

  var resutl = test();

  if (result) {
    console.log("sample2");
  }
}());

Re: deep さん

Comments

[deep]

例外キャッチか戻り値判定の２択なのですね。
ありがとうございます。

[x_x]

think49さんの意図はわかりませんがキャッチはしていないのでは？
例外で終了です。

[think49]

親記事に追記しました。

Answer 4

不可能です。

---

どのくらい突飛な事をやろうとしているのか、別のモノで例えてみます。

最近妻(夫)が浮気してるような気がする。
探偵に頼もう(test関数)実行。
しばらくして探偵から連絡がありました。
探偵「いやあ、どうやら貴方の配偶者は浮気してたみたいなんで勝手に裁判してきました。これが和解金です。いやあ相場より若干少なかったですね。」

これくらい酷いことをやってます。

依頼を受けた探偵(test関数)が出来るのは、
依頼主に「貴方の配偶者は浮気してましたよ or 浮気してませんでしたよ」という結果を返すだけです。
それを元にどういう行動すべきかを自分で決めて下さい。

---

まぁ、JavaScriptならではの抜け道は色々とありますので考えていきます。
しかしtest関数の戻り値で今後の動作を決めるのが最も最適解ということは忘れないでください。

• 例外をスローして後続の処理を殺す
• 関数の外から変数を弄る
• オブジェクトの参照を渡して中から弄る
• 処理が成功した時に実行してほしいものを引数として渡す

1個目: 例外スローはyambejpさんの案です。
2個目: 関数の外から変数を弄る
この書き方は外の変数に依存するので使い勝手が非常に悪いです。

それとコードを見れば分かりますが、
いつの間にかflags.testの値がfalseになったのかコードからは中々読み取れません。
従ってこういう書き方を多くの場面でやっているプロジェクトは技術的負債で進捗が大幅に遅延することになります。

var main = function () {
  var flags = {
    test: true
  }
  var test = function () {
    // このように関数スコープの外にある変数には自由にアクセスできます
    flags.test = false;
  }
  console.log("sample");
  test();
  if (flags.test) return; // 質問文的にこの1行が抜けないのであまり意味なし？
  console.log("sample2");
}
main();

3個目: オブジェクトの参照を渡して中から弄る

ダサいですがありって感じですかね。
これもflags.testが勝手に書き換わるのが何処かは分かりかねますが、
testの引数がflagsを欲しがっている事が分かるので、まだ推測し易いと言えるでしょう。

var test = function (flags) {
  // 引数がオブジェクトや配列の場合参照渡し(※)になるので中から値を弄って変えられる
  flags.test = false;  
}
var main = function () {
  var flags = {
    test: true
  }
  console.log("sample");
  test(flags);
  if (flags.test) return; // 質問文的にこの1行が抜けないのであまり意味なし？
  console.log("sample2");
}
main();

4個目: 成功した時に今後やりたい動作を関数として定義してやる

ちょっとトリッキーですが、これは多くのオブジェクト指向言語のDIコンテナに似た解決方法です。

JavaScriptは言語仕様的に関数が第一級オブジェクトなので、
配列等と同様に変数に格納・引数に設定・戻り値として返す事が可能です。

上級者が質問文の条件で何とかやれと言われたら、
testに関数を引数としてよこすようにし、こういう形で解決を目指すことになるかと思います。

// 関数を引数にセットできる
var test = function (cb) {
  // 条件を満たしたときのみコールバック関数を実行
  if (true) {
    cb();
  }
}
var main = function () {
  console.log("sample");
  test(function () {
    console.log("sample2");
  });
}
main();

Comments

[deep]

メインフローで戻り値判定するのが一番のようですね。
分かりやすいご解説ありがとうございます。

Answer 5

console.log("sample");
test();
console.log("sample2");

function test() {
	console.log('test');
	fadoijafjoafjeoifdafafafaf!!! //ここがポイント
}

Comments

[miyabi-sun]

SyntaxErrorで最初のconsole.logも発火しませんでした
こんな感じに修正してみましたのでご査収の程よろしくお願いします。

console.log("sample");
test();
console.log("sample2");

function test() {
　console.log('test');
　eval('fadoijafjoafjeoifdafafafaf!!!'); //ここがポイント
}