Провести ревью кода.

go · basicsOzon / Озон (ревью кода)не решено

Условие

Провести ревью кода.

// ТЗ: мы хотим собирать информацию по курсам валют в разных банках.
// Требуется написать программу, которая каждую минуту будет отправлять запрос в банк и получать курс нескольких валют и сохранять результат в БД.
// Банков может быть несколько.

func main() {
  if len(os.Args) == 2 {
    cmdName := os.Args[1]
    if cmdName == "help " {
      fmt.Println("Usage is './currency update '")
    } else if cmdName == "update " {
      urlsBank := []struct {
        bankName string
        curFrom  string
        curTo    string
        url      string
      }{
        {
          bankName: "Bank 1 ",
          curFrom:  "RUB ",
          curTo:    "USD ",
          url:      "http://bank.example.com/rates/rub-usd ",
        },
        {
          bankName: "Bank 2 ",
          curFrom:  "RUB ",
          curTo:    "USD ",
          url:      "http://bank2.example.com/rates?currFrom=RUR &currTo=USD ",
        },
      }

      clientBank := &http.Client{}

      for _, url := range urlsBank {
        req, _ := http.NewRequest(http.MethodGet, url.url, nil)

        if url.bankName == "Bank 2 " {
          req.Header.Add("Authorization ", "auth_token=\"XXXXXXX\"")
        }

        resp, err := clientBank.Do(req)

        if err != nil {
          panic(err)
        }
        defer resp.Body.Close()
        body, _ := io.ReadAll(resp.Body)

        strBody := string(body)

        if url.bankName == "Bank 1 " {
          strBody = strings.ReplaceAll(strBody, ",", ".")
        }

        value, err := strconv.ParseFloat(strBody, 64)
        if err != nil {
          panic(err)
        }
        err = updateCurrency(url.bankName, url.curFrom, url.curFrom, value)
        if err != nil {
          panic(err)
        }
      }
    }
  } else {
    fmt.Println("Usage is './currency update '")
  }
}

const (
  host     = "localhost "
  port     = 5432
  user     = "postgres "
  password = "<password >"
  dbname   = "<dbname >"
)

func updateCurrency(bank, from, to string, value float64) error {
  psqlconn := fmt.Sprintf("host=%s port=%d user=%s password=%s dbname=%s sslmode=disable ", host, port, user, password, dbname)

  db, err := sql.Open("postgres ", psqlconn)
  CheckError(err)

  defer db.Close()

  err = db.Ping()
  CheckError(err)

  fmt.Println("Connected!")

  insertStmt := fmt.Sprintf(`insert into currency_rates ("bank ", "from ", "to ", "value ") values('%s ','%s ','%s ','%.2f ')`, bank, from, to, value) //
  _, err = db.Exec(insertStmt)
  return err
}

func CheckError(err error) {
  if err != nil {
    panic(err)
  }
}

Решение

Разбор

---

Источник: Ozon / Озон (ревью кода)

← (код без описания) · Все задачи · Go · Провести ревью код. Затем исправить про… →