Constructor literal

Esta categoría de advertencia se escribe [literal-constructors] por qmllint.

No usar función como constructor

¿Qué ha ocurrido?

Se utilizó una función de construcción literal como constructor.

¿Por qué es malo?

Llamar a una función de construcción literal como Number como una función regular coacciona el valor pasado a un número primitivo. Sin embargo, llamar a Number como constructor devuelve un objeto derivado de Number que contiene ese valor. Esto es un desperdicio y probablemente no es el resultado esperado. Además, puede llevar a un comportamiento inesperado o confuso debido a que el valor devuelto no es primitivo.

Ejemplo

import QtQuick

Item {
    function numberify(x) {
        return new Number(x)
    }
    Component.onCompleted: {
        let n = numberify("1")
        console.log(typeof n)   // object
        console.log(n === 1)    // false

        if (new Boolean(false)) // All objects are truthy!
            console.log("aaa")  // aa
    }
}

Para solucionar este problema, no llame a estas funciones como constructores, sino como funciones normales:

import QtQuick

Item {
    function numberify(x) {
        return Number(x)
    }
    Component.onCompleted: {
        let n = numberify("1")
        console.log(typeof n)   // number
        console.log(n === 1)    // true

        if (Boolean(false))
            console.log("aaa")  // <not executed>
    }
}