Piores praticas de programação

[munizart]

Olá pessoal!
Inspirado por artigos como "Como não ser um bom front-end" do @kvnol e afins, to pensando em escrever algo sobre más praticas de programação em javascript.

Então queria saber de vocês qual ou quais as piores praticas vocês já viram.
Vou listar algumas que eu considero ruim.

Usar a keyoword with

Como já avisa a MDN (vale a pena ler o artigo):

Essa é realmente rara, para nossa alegria.

É uma keyword que nem gosto de falar sobre, pra ninguém cair em tentação. O comportamento dela é bastante confuso (e cheio de inconsistências), mas basicamente permite você acessar propriedades de um objeto como se fossem varieis de escopo dentro de determinado bloco.
Exemplo:

with (Math) {
  console.log(round(PI)) //3
}

A noticia boa é que o código abaixo te dá um SintaxError, já que o modo estrito bloqueia o uso de with.

function strictFunction() {
  "use strict";
  with (Math) {
    console.log(round(PI)) //3
  }
}

Usar um loop para mapear um array para outro

Essa é extremamente comum, a maior parte das vezes por desconhecimento da função Array.prototype.map.
Eu especialmente não gosto porque todo loop acaba criando mais uma variável no escopo pra levar em consideração (normalmente com um péssimo nome, tipo i ou j).

var estrofes = [
  "brilha",
  "brilha",
  "estrelinha"
];
var refrao = [];
for (var i = 0; i < estrofe.length; i++) {
  refrao.push(estrofe[i].toUpperCase());
}

Com map ficaria:

var estrofes = [
  "brilha",
  "brilha",
  "estrelinha"
];
var refrao = estrofes.map(function gritando(estrofe) {
  return estrofe.toUpperCase();
});

Fallback com operador ternário

Isso:

function Pessoa (nome) {
  this.nome = nome ? nome : "Maria";
}

Invés disso:

function Pessoa (nome) {
  this.nome = nome || "Maria";
}

Código em português

Como vocês viram em todos os exemplos javascript é uma linguagem com keywords em inglês e toda essa troca de contexto entre inglês/português no meio código realmente dificulta a leitura de códigos maiores.
Não me levem a mal, português é minha língua favorita. Mas assim como não gosto de falar palavras como "blueberry" ou "hashtag" no meio das frases em português, também não gosto de usar nome de variáveis e funções em pt-BR.

E pra vocês, quais seriam as piores praticas? discordam de algo que disse?

[kivervinicius]

Eu já conhecia o with em outras linguagens, para ser sincero nem sabia que esse "empestiado" existia no JS.

[kvnol]

Que massa que meu artigo te inspirou, @munizart. Faz o seguinte, como eu fiz no meu artigo, coloca ele no ar e vai fazendo updates assim que pegar novas más práticas.

[Sergioamjr]

O Eslint me ajuda muito pra evitar esses erros, uma coisa que eu uso com frequencia é o destructuring,
Ao invês de

const id = obj.key

Usar

const { key: id } = obj

Referência http://exploringjs.com/es6/ch_destructuring.html

[dionarodrigues]

No Browser Diet tem bastante dicas interessantes: https://browserdiet.com/pt/#js

[AmandaFonseca]

A pior pratica pelo menos a que mais me prejudica, é quando eu vou pesquisar algum erro no google e acabo clicando em links aleatórios e no final do dia descubro que eu gastei a maior parte do meu dia lendo sobre a vida secreta dos golfinhos ....

[kivervinicius]

@AmandaFonseca tmj

[phsantiago]

Não escrever testes

[romulo2735]

Ainda sou muito noob em JS, mas vcs estão me ajudando! hehe

[Pedro-Souza]

Quando escrever poste aqui o link