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, May 25 '18 at 12:15
`Reporter` doesn't sound like it should be a `class` at all. — CodingIntrigue, May 25 '18 at 12:17
Possible duplicate of: https://stackoverflow.com/questions/44091167/es6-classes-updating-static-properties — CodingIntrigue, May 25 '18 at 12:20
@CodingIntrigue, I get it. If so, whats your suggestion. because I am using a protractor cucumber framework for my testing project and I need to inform this instruction in the config file which will be invoked at the start of execution. — mmar, May 25 '18 at 12:26
@mmar Can you link the docs of your framework about this? I really can't believe it requires a `class`. — Bergi, May 25 '18 at 13:21

score 1 · Accepted Answer · answered May 25 '18 at 12:41

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 '18 at 12:41

CodingIntrigue

75,930
30
170
176

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. – mmar May 25 '18 at 13:05
That practice is fine. You should post your code in an answer – CodingIntrigue May 25 '18 at 13:06
@CodingIntrigue Using a `class` without a constructor, without instance properties, without any instance methods, is *not* a "fine practice". – Bergi May 25 '18 at 13:19
1

@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 – CodingIntrigue May 25 '18 at 13:24

how to declare an object in javascript es6 class constructor

1 Answers1