我的javascript构造函数和相关语法有什么问题

Question

EDIT: Thank you to all who responded. I'm in the process of checking it with just console.log for the desired output, but at the moment I do know that it is not doing what I want with HTML. Anyone else know what could be wrong?

Thanks!

EDIT 2: The function "functions" when I define "television" directly within the console, but other than that, I can't figure out how to have television defined in the code. I took it out of the start(); function because I thought it might be out of scope. That didn't work either.

// Program Name: Television Store
// Television Store Website.

    /* Constructor Function: */
        function TelInput(ret, man, scr, conf, dis) {
        this.ret = parseInt(ret, 10);
        this.man = man;
        this.scr = parseInt(scr, 10);
        this.discount = this.ret * 0.9;
        this.dis = dis;
        this.conf = conf  
    };

var start = function(){    
/* Initial Variables */
var screen = prompt("Enter the screen size of TV in inches: ");
var manufacturer = prompt("Enter the manufacturer of TV: ");
var retail = prompt("Enter the retail value of TV: ");
var priceConf = confirm("Apply the 10% discount?");
var dispValues = confirm("Display Values when Finsihed?");

/* Display the Values */

console.log(television.ret);
console.log(television.man);
console.log(television.scr);
console.log(television.discount);

    if(television.conf){
        document.getElementById("l4").innerHtml = "Discounted Price:  " + television.dis;   
    }
};
console.log("Program Completed."); // Confirm Javascript functions.
start(); // Start function.
var television = new TelInput(retail, manufacturer, screen, priceConf, dispValues); // Create object

Answer 1

If telInput is supposed to be constructor function:

• Call it with new : var television = new telInput(retail, manufacturer, screen, priceConf, dispValues); . Right now the properties ret , man , etc will be assigned to window and television will be undefined .

• Pascal-case constructor functions: TelInput . This is a very strong convention. See this .

A few other issues:

• You are not assigning dis and conf to any property: add this.dis = dis; this.conf = conf this.dis = dis; this.conf = conf .

• this.discount = ret * 0.9; this should probably be this.discount = this.ret * 0.9; as this.ret contains the parsed value.