Não sou expert, só estou "brincando" um pouco com a sintaxe do ES6 e queria saber porque não usar o Date.parse ao invés de dar replace na string? Outra dúvida é se estaria errado fazer as transformações dos valores diretamente no construtor da classe:
class NegociacaoController {
constructor() {
let select = x => document.querySelector(x);
this._inputData = Date.parse(select('#data').value);
this._inputQuantidade = parseInt(select('#quantidade').value);
this._inputValor = parseInt(select('#valor').value);
}
adiciona(event) {
event.preventDefault();
let negociacao = new Negociacao(
this._inputData,
this._inputQuantidade,
this._inputValor
);
}
}