去掉这句代码

......
S.prototype=new P();
......

js面向对象这么写累死人，推荐[aralejs]http://www.aralejs.org/框架。
贴一个arale面向对象的hello world!
1、定义对象：

/* pig.js */
define(function(require, exports, module) {
    var Class = require('arale/class/1.1.0/class');

    var Pig = Class.create({
        initialize: function(name) {
            this.name = name;
        },

        talk: function() {
            alert('我是' + this.name);
        }
    });

    module.exports = Pig;
});

2、继承对象

/* red-pig.js */
define(function(require, exports, module) {
    var Pig = require('./pig');

    var RedPig = Pig.extend({
        initialize: function(name) {
            RedPig.superclass.initialize.call(this, name);
        },

        fly: function() {
            alert(this.color + "猪飞起来了");
        },

        color: '红色'
    });

    module.exports = RedPig;
});

3、使用对象

var RedPig = require('./red-pig');
var redPig = new RedPig("猪");
redPig.talk();
redPig.fly();

符合common.js规范。

因为你在 S 的构造函数内已经 call 了 P，于是也就已经有了 this.name 和 this.age，因此已经没有必要 new P() 赋给 S.prototype 了，所以把这一句去掉即可：

S.prototype = new P();

尝试面向对象编程时也遇到题主的问题，翻看《JavaScript 高级程序设计》 第六章的组合继承，和题主是一样的，书中给出的参数是变量，如果想要以对象作为参数，在子类的原型 new父类的构造函数时，就报错了。问题就出在这里，经过尝试，可以将 父类的原型直接赋值给子类的原型，

将S.prototype=new P(); 换成 S.prototype = P.prototype 

即可。但有一个严重的问题，就是引用类型的值在所有实例中共享。