how to declare an object in javascript es6 class constructor

Question

I declared a property in class constructor and accessing through methods, which is declared as 'static', with 'this' and it is not accessible. How to get access to constructor (class) variables inside static methods?

export class Reporter {
    constructor() {
        this.jsonReports = path.join(process.cwd(), "/reports/json")

        this.cucumberReporterOptions = {
            jsonFile: targetJson,
            output: htmlReports + "/cucumber_reporter.html",
            reportSuiteAsScenarios: true,
            theme: "bootstrap",
        }
    }

    static createHTMLReport() {
        try {
            reporter.generate(this.cucumberReporterOptions);
        } catch (err) {

        }
    }
}

Updated:

As per "@CodingIntrigue", I have done like this in 'reporter.js' file and called the method as Reporter.createHTMLReport() in my config file and its working as expected. But not sure if this is best practice.

const jsonReports = path.join(process.cwd(), "/reports/json")

const cucumberReporterOptions = {
    jsonFile: targetJson,
    output: htmlReports + "/cucumber_reporter.html",
    reportSuiteAsScenarios: true,
    theme: "bootstrap",
}

export class Reporter {
    static createHTMLReport() {
        try {
            reporter.generate(cucumberReporterOptions);
        } catch (err) {

        }
    }
}

You can't, that's why the method is static, there is no access to instance members. — Wiktor Zychla
– Wiktor Zychla, Commented May 25, 2018 at 12:15
Reporter doesn't sound like it should be a class at all. — CodingIntrigue
– CodingIntrigue, Commented May 25, 2018 at 12:17
Possible duplicate of: stackoverflow.com/questions/44091167/… — CodingIntrigue
– CodingIntrigue, Commented May 25, 2018 at 12:20
@mmar Can you link the docs of your framework about this? I really can't believe it requires a class. — Bergi
– Bergi, Commented May 25, 2018 at 13:21

CodingIntrigue · Accepted Answer · 2018-05-25 12:41:01Z

1

If you want to continue using class syntax, you can just make jsonReports and cucubmerReporterOptions static properties too:

export class Reporter {
    static createHTMLReport() {
        try {
            reporter.generate(Reporter.cucumberReporterOptions);
        } catch (err) {

        }
    }
}

Reporter.jsonReports = path.join(process.cwd(), "/reports/json")

Reporter.cucumberReporterOptions = {
    jsonFile: targetJson,
    output: htmlReports + "/cucumber_reporter.html",
    reportSuiteAsScenarios: true,
    theme: "bootstrap",
}

answered May 25, 2018 at 12:41

CodingIntrigue

79k32 gold badges176 silver badges178 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

mmar Over a year ago

I have updated the question with the solution I have done as per your suggestion and its working too. But not sure if this could be a good practice to follow.

CodingIntrigue Over a year ago

That practice is fine. You should post your code in an answer

Bergi Over a year ago

@CodingIntrigue Using a class without a constructor, without instance properties, without any instance methods, is not a "fine practice".

CodingIntrigue Over a year ago

@Bergi I did say in the comments above that it shouldn't be a class at all, but I was going on the possibly incorrect assumption that not everything was going to be static in the class

Collectives™ on Stack Overflow

how to declare an object in javascript es6 class constructor

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related