このjavascriptの書き方は古い書き方ですか？

Question

[知りたいこと]
・この書き方は古いですか？
・もっと良い書き方はありますか？

function bookCreate(title, summary, isbn, author, genre, cb) {
  bookdetail = {
    title: title,
    summary: summary,
    author: author,
    isbn: isbn,
  };
  if (genre != false) bookdetail.genre = genre;

  var book = new Book(bookdetail);
  book.save(function(err) {
    if (err) {
      cb(err, null);
      return;
    }
    console.log('New Book: ' + book);
    books.push(book);
    cb(null, book);
  });
}

Comments

[m.ts10806]

基準が分かりません（古い、良い）
自身が当該コードをきちんと理解できていれば出ない疑問かとは思うのですが、
そのあたりいかがでしょうか。

[miyabi_pudding]

開発対象となる環境にもよるので、一概に言えません。
ブラウザの場合、モダンな書き方でも、古いブラウザだと動作しなかったりしますし、
たとえNodeでも、Nodeのバージョンが古ければ、モダンな記述に対応してなかったりしますし。
なので、対象とする環境も記載お願いします。
また、やりたいこと、にもよりますので、何をしたいか、何を開発したいか、も記載された方が、より回答を得られるかと。

Answer 1

・この書き方は古いですか？

何をもって古いとするかによります。新しい書き方でないことは確かですが、動くのであれば「サポート範囲内」とも言えます。

・もっと良い書き方はありますか？

どんな書き方が「良い」かは状況次第、人次第です。もっとモダンな書き方があることは確かです。

Answer 2

このjavascriptの書き方は古い書き方ですか？

ご質問のコードは、Bookオブジェクトのsaveメソッドが「コールバック」方式で書かれています。一方で「フロー制御」方式での書き方もあります。

• コールバック方式: jQuery登場以前からある方式。可読性は悪いが、処理は早い。
• フロー制御方式: jQuery登場後の方式。可読性が高いが、処理速度を犠牲にしいる。

Book.save() の実装である「コールバック方式」を、歴史的視点で語るなら確かに古いですが、実際のところは トレードオフ です。

両方共、使えるようにするのが大事（ちなみに、コールバック方式を理解できていないと、フロー制御は理解できません）。