Como o encadeamento opcional é uma adição relativamente recente no JavaScript (estandardizado com o ECMAScript 2020), o TypeScript (dependendo da opção target
) pode emitir o código de modo a funcionar em "ambientes" antigos.
É por isso que este código:
const foo = {} as any;
console.log(foo?.bar?.baz);
Compila para isto:
"use strict";
var _a;
const foo = {};
console.log((_a = foo === null || foo === void 0 ? void 0 : foo.bar) === null || _a === void 0 ? void 0 : _a.baz);
Mas note que a expressão passada para o primeiro argumento de console.log
continuará avaliando para a mesma coisa, concorda?
Desse modo, em runtime (fora o código adicional), não há nenhuma diferença entre usar o encadeamento opcional ou essa geringonça aí. A expressão sempre avaliará para o mesmo resultado.
No entanto, a mensagem do ESLint é clara:
Extract it to a separate variable so it can be statically checked.
Quando se trata de ambiente estático, pode-se afirmar que trata-se do próprio código-fonte. Nesse caso, é bem diferente você usar uma expressão pequena como foo?.bar?.baz
do que usar o resultado compilado do TypeScript (que, embora também seja uma expressão, é bem maior e, portanto, parece exigir mais da análise estática feita pelo eslint-plugin-react-hooks
).
Note que, embora a expressão sempre avalie para a mesma coisa, a análise estática (que é sob a qual ESLint e TypeScript trabalham) fica mais difícil no segundo caso. É mais código para analisar.
O plugin eslint-plugin-react-hooks
foi criado para "cuidar" do array de dependências do useEffect
(e similares). E provavelmente, para "facilitar" o trabalho, ele pede que se utilizem expressões "menos complexas".
Com isso, creio que não haja problema em utilizar o encadeamento opcional no array de dependências se a única preocupação for a diferença entre código-fonte (TypeScript) e o código "compilado". A expressão sempre avaliará para a mesma coisa.
Mesmo se você tivesse utilizando o "código compilado" diretamente no seu código-fonte, o ESLint só ia pedir para que você extraísse a expressão para uma variável fora do array para ficar menos "complexo" — como a própria mensagem de aviso diz. Não sei porque, mas ele diz que isso facilitaria a "análise estática". Como é estático, provavelmente não terá nenhuma influência no runtime do seu código, então pode usar sem medo.