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

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

ただいまの
回答率

87.34%

同じ結果を出力する下記のコードの違いとベストなコードを教えて下さい

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 792

score 16

前提・実現したいこと
下記の4つのコードの違いとベストな書き方が知りたいです。
たとえば1つ目のbornYear()はなぜthisでpersonを取得できるのか不思議です。

function person(name, age) {
    this.name= name;
    this.age = age;
    this.yearOfBirth = bornYear;
}
function bornYear() {
    return 2019 - this.age;
}

var p = new person("A", 22);

console.log(p.yearOfBirth());
function person(name, age) {
    this.name= name;
    this.age = age;
    this.yearOfBirth = bornYear(n);
}
function bornYear(n) {
    return 2019 - n;
}

var p = new person("A", 22);

console.log(p.yearOfBirth)
function person(name, age) {
    this.name= name;
    this.age = age;
    this.yearOfBirth = function(n) {
    return 2019 - n;
}
}

var p = new person("A", 22);

console.log(p.yearOfBirth(22));
function person(name, age) {
    this.name= name;
    this.age = age;
    this.yearOfBirth = return 2019 - age;
}

var p = new person("A", 22);

console.log(p.yearOfBirth);
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • yasutomi

    2019/07/30 08:12

    このコードはエラーになって実行されないため
    エラーを解消してから再度ご質問をお願いします。

    キャンセル

  • fiile

    2019/07/30 08:18

    失礼しました。修正いたしました。よろしくお願いします。

    キャンセル

回答 3

checkベストアンサー

+8

コンストラクタ関数

ごく普通のコンストラクタ関数なので、コンストラクタの基本を学んでから、比較を試みて下さい。
基本を押さえずに、比較しても理解できません。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Guide/Creating_New_Objects/Using_a_Constructor_Function

this

たとえば1つ目のbornYear()はなぜthisでpersonを取得できるのか不思議です。

this は person ではありません。
個人サイトには、下記の表現で説明しているところがありますが、

  • this は person である
  • this は自身である
  • thisはコンストラクタそのものである

比較すれば、間違っている事は明らかなので、信用しないで下さい。

console.log(this === person);

等価性を期待するなら、下記コードになります。

console.log(Object.getPrototypeOf(this) === person.prototype);

この原理を把握するには、プロトタイプチェーンを理解する必要があります。
https://teratail.com/search?q=prototype+chain

年齢から誕生年を逆算

年齢から誕生年を逆算する事は出来ません。
例えば、「誕生日を迎えた20歳」と「誕生日前の20歳」を区別する方法がありません。
例示の4つのコードは全て好ましくないと考えられます。

改善案として、「名前」「誕生日」をインスタンスプロパティに持つコンストラクタまたは、class構文があります。
が、ベストなコードを暗記する学習法は応用が利かなくなるので、お勧めしません。

Re: fiile さん

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/07/30 12:14

    回答内容に間違いがあれば、ご指摘下さい(低評価されているので)。

    キャンセル

  • 2019/07/31 09:12

    ありがとうございます。コンストラクタ関数についてもう少し理解してから比較してみます。
    また、thisの中身も勘違いしておりました。
    助かりました。

    キャンセル

+2

普通に今風にかけばclassですかね

class person {
  constructor(name, age){
    this.name= name;
    this.age = age;
    this.yearOfBirth = this.bornYear;
  }
  bornYear(){
    return 2019 - this.age;
  }
}
var p = new person("A", 22);
console.log(p.yearOfBirth());
//console.log(p.bornYear()); で十分?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/07/31 09:15

    ありがとうございます!
    クラス構文も合わせて学びたいと思います。

    キャンセル

+1

2番目と4番目はエラーを出します。特に4番目は文法がおかしいので論外です。
1番目と3番目の違いは、yearOfBirthが指す関数をインスタンスごとに持っているか、クラスで持っているか、の違いです。

function person(name, age) {
    this.name= name;
    this.age = age;
    this.yearOfBirth = bornYear;
}
function bornYear() {
    return 2019 - this.age;
}

var p1 = new person("A", 22);
var p2 = new person("A", 22);

p1.yearOfBirth === p2.yearOfBirth;
/*
true
*/
function person(name, age) {
    this.name= name;
    this.age = age;
    this.yearOfBirth = function(n) {
        return 2019 - n;
    }
}

var p1 = new person("A", 22);
var p2 = new person("A", 22);

p1.yearOfBirth === p2.yearOfBirth;
/*
false
*/

1つ目のbornYear()はなぜthisでpersonを取得できるのか不思議です

それは勘違いです。thispersonではなくてpです。

ベストな書き方が知りたいです。

いろいろな意見があると思いますが、個人的にはクラス構文で書いた方がいいと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/07/31 09:13 編集

    失礼しました。
    2番目は出力の際に引数が抜けていたのと、4番目はreturnが不要でした。

    'this'は'p'だったんですね、、
    前提から間違えて認識しておりした。
    ありがとうございます。

    キャンセル

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

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

関連した質問

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