Skip to content

Instantly share code, notes, and snippets.

@dremendes
Last active May 2, 2016 02:56
Show Gist options
  • Save dremendes/8299245 to your computer and use it in GitHub Desktop.
Save dremendes/8299245 to your computer and use it in GitHub Desktop.
try & catch example
/*
Não sabemos o que tem dentro de "e" pode ser uma adição,
remoção ou ambos. O snippet abaixo funciona em todos os casos.
*/
Estado.atualizaEstado = function(e){
//Try to add a state;
try{
if(typeof(e.added.id) !== undefined){
Estado.selected.push(e.added.id);
console.log('Estado added to list');
}
}catch(e){
console.log('This wasn\'t an addition');
}
//Try to remove a state;
try{
if(typeof(e.removed.id) !== undefined){
var index = Estado.selected.indexOf(e.removed.id);
Estado.selected.splice(index, 1);
console.log('Estado removed from list');
}
}catch(e){
console.log('This wasn\'t a remotion');
}
}
@millermedeiros
Copy link

se o try/catch no seu caso é apenas para evitar o erro na hora de acessar uma propriedade inexistente eu usaria alguma abstração tipo o mout/object/get ou refatoraria o código para algo do tipo:

Estado.atualizaEstado = function(e) {
  if ((e.added && e.added.id) != null) {
    Estado._add(e);
  } else {
    Estado._remove(e);
  }
};

Estado._remove = function(e) {
  var index = Estado.selected.indexOf(e.removed.id);
  Estado.selected.splice(index, 1);
  console.log('Estado removed from list');
};

Estado._add = function(e) {
  Estado.selected.push(e.added.id);
  console.log('Estado added to list');
};

if/else e try/catch normalmente são considerados como red flags (indicadores de que algo está errado).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment