16

TypeError の問題があります:

function artist(name) {
    this.name = name;
    this.albums = new Array();

    this.addAlbum = function(albumName) {
        for (var i = 0; i < this.albums.length; i++) {
            if (this.albums[i].name == albumName) {
                return this.albums[i];
            }
        }

        var album = new album(albumName);
        this.albums.push(album);

        return album;
    }
}

function album(name) {
    this.name = name;
    this.songs = new Array();
    this.picture = null;

    this.addSong = function(songName, track) {
        var newSong = new songName(songName, track);
        this.songs.push(newSong);

        return newSong;
    }
}

次のエラーが発生します。

TypeError: album is not a constructor

問題が見つかりません。他の投稿をたくさん読みましたが、同様の問題は見つかりませんでした。別のオブジェクトでオブジェクトを作成することは許可されていないのでしょうか? どうすればこの問題を解決できますか?

4

1 に答える 1

42

この行

var album = new album(albumName);

外部album関数をシャドウします。そうalbumです、関数内のコンストラクターではありません。正確に言えばundefined、この時点です。

この種の問題を避けるために、大文字で始まる「クラス」に名前を付けることをお勧めします。

function Album(name) {

より一般的には、疑問がある場合は Google スタイル ガイドに従うことをお勧めします。

于 2013-02-21T17:40:29.497 に答える