javascriptの書き方

Question

下記の関数内の2行を1行で書けるでしょうか。

function test(){

  var id = getNewId();

  if(id == null)return;

  //処理
}

イメージとしては下記の様なもの？

function test(){

  var id = getNewId() == null ? return : getNewId();

  //処理
}

追記
ご回答有難うございます。
勉強になりました。

Answer 1

{}を使わずに全体が1文で書くことはできましたが、わかりやすいいいコードであるとはとても思えません。

var list = [1, 2, 3];
function getNewId() {
    return list.shift();
}

function test(){
  for(var id = getNewId();id == null;)return;
  console.log(id);
}

test();
test();
test();
test();

Answer 2

「セミコロンで区切って1行に詰め込む」というトンチしか、手段はないと思います。

「変数宣言」も「return」も、それぞれ1文が必要な操作なので、両者を1文で同時に行うことはできません。

Answer 3

JSはC言語譲りのALGOL系の言語です。
JSコードゴルフでもやってるなら話は別ですが、
可読性が落ちるので普通の開発で1行で書く選択肢はありえません。

変数の宣言で1行、○○であれば条件分岐という流れで1行はどうしても欲しいです。
それが嫌なら、下記のような感じで工夫しましょう。

• そもそもJSを諦める
• 関数の内部で値を宣言せずに、引数として持ち込む作りを徹底する

var test = function (id) {
  if (id == null) return;
  // 処理
}

JSを捨てる手法はメタプログラミングって言うらしいですよ。
私も業務でAltJSを使っていた時期があり、真面目な選択肢としてありだと思ってます。

基本的な候補はCoffee系の記述量が減る言語ですかね。
TypeScript系はスーパーセットであり、型記述の為にコード量を増やす選択肢をしているので質問者さんとは相性が悪そうです。
あまり知りませんが、ScalaやNimなんかもJavaScriptが吐けるらしいですよ。

例えばCofeeScript系のAltJSの一つであるLiveScriptはひたすら簡素な記述なので
質問文の事を全て満たしたキモくなりがちなコードも、許せる長さに収まります。

test = ->
  return if not (id = get-new-id!)?
  # 処理

Answer 4

できなくはないですね。

"use strict";
function getNewId() {
	return null;
	// return 555;
}
function test() {
  console.log( 'test' );
	switch ( getNewId() ) { case null: return; default: var id = getNewId(); }
  console.log( 'done', id );
}
test();
```**動くサンプル：**[https://jsfiddle.net/am2crzgL/](https://jsfiddle.net/am2crzgL/)

Comments

[Lhankor_Mhy]

細かいことを言うのも無粋ですが、これは getNewId() が参照透過性を持っていることが前提なりますね。

[kei344]

そうですね。
switch ( id = getNewId() ) { case null: return; default: var id; }
「巻き上げ悪用」案をぱくれば問題なくなります。

Answer 5

「変数宣言巻き上げ」を悪用してこんな感じで。

var id = 'hoge';

function test(){
  if( ( id = getNewId() ) == null) return; else var id; //else節で宣言しても巻き上げられる
  console.log(id);
}

function getNewId(){
  return null
}

test();
id; // 'hoge'

if...else...で1文とみなしていいのかが、ちょっと疑問ですが。

もう一つ思いついたので追記

var id = 'hoge';

function test(){
  if( ( eval('var id'), id = getNewId() ) == null) return; //eval関数で宣言してもスコープ内で定義される
  console.log(id);
}

function getNewId(){
  return null
}

test();
id; // 'hoge'

邪悪。

Comments

[maisumakun]

ネタにマジレスしてもそう意味はないですが、strictモードの場合は、evalでスコープに変数を発生させることはできなくなっている、とのことです（ES6 Moduleやclassなどは暗黙でstrictになるので要注意かも）。

https://developer.mozilla.org/ja/docs/Web/JavaScript/Strict_mode#%E5%A4%89%E6%95%B0%E3%81%AE%E4%BD%BF%E7%94%A8%E3%81%AE%E5%8D%98%E7%B4%94%E5%8C%96

[Lhankor_Mhy]

おお、それは知らなかったです。ありがとうございます。

returnとかは外に漏れないのに変数・関数定義が外に漏れてくるの、おかしな挙動だなあ、と思ってたので、妥当な仕様ですね。