From 6959d8ac5dc5b4316a86bb58b669da9cdbd07e2c Mon Sep 17 00:00:00 2001 From: Arseny Balobanov Date: Sun, 12 Mar 2023 19:50:35 +0300 Subject: [PATCH] [b/docs/gitfame_review_comments.md] Add note on style. --- docs/gitfame_review_comments.md | 24 ++++++++++++++++++++++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/docs/gitfame_review_comments.md b/docs/gitfame_review_comments.md index 399b1d1..659c4bf 100644 --- a/docs/gitfame_review_comments.md +++ b/docs/gitfame_review_comments.md @@ -189,13 +189,33 @@ var file []byte После работы утилиты пользователь ожидает, что он останется в той же директории, в которй запускал утилиту. Неправильно: -``` +```golang err := os.Chdir(repository) cmd := exec.Command("git", "blame", "--porcelain", revision, "--", file) ``` Правильно: -``` +```golang cmd := exec.Command("git", "blame", "--porcelain", revision, "--", file) cmd.Dir = repository ``` + +## Стиль + +### NIT Используйте общие var и const декларации + +Для однородных переменных и констант нет необходимости писать `var` перед каждой строкой + +Вместо +``` +var flagRepo = flag.String("repository", ".", "repo") +var flagRev = flag.String("revision", "HEAD", "revision") +``` + +Можно написать +```golang +var ( + flagRepo = flag.String("repository", ".", "repo") + flagRev = flag.String("revision", "HEAD", "revision") +) +```