Skip to content

var-scope.md

Latest commit

 

History

History
68 lines (53 loc) · 983 Bytes

var-scope.md

File metadata and controls

68 lines (53 loc) · 983 Bytes
Raw

Reduce Scope of Variables

Where possible, reduce scope of variables. Do not reduce the scope if it conflicts with Reduce Nesting.

BadGood
 
err := os.WriteFile(name, data, 0644)
if err != nil {
 return err
}
 
if err := os.WriteFile(name, data, 0644); err != nil {
 return err
}

If you need a result of a function call outside of the if, then you should not try to reduce the scope.

BadGood
 
if data, err := os.ReadFile(name); err == nil {
  err = cfg.Decode(data)
  if err != nil {
    return err
  }

  fmt.Println(cfg)
  return nil
} else {
  return err
}
 
data, err := os.ReadFile(name)
if err != nil {
   return err
}

if err := cfg.Decode(data); err != nil {
  return err
}

fmt.Println(cfg)
return nil