LINUX.ORG.RU

История изменений

Исправление nikolnik, (текущая версия) :

В этой функции ужасно вообще все

1. Получишь панику в defer если с файлом что-то не так

2. Ошибка из file.Close() никак не обрабатывается

3. Путь к конфигу захардкожен

4. Условие проверки ошибки открытия файла некорректное

5. Корректно использовать config := &Config{}, без var

6. json.NewDecoder нужен для потоковой обработки жсона, здесь правильно скопировать данные с файла в буфер и использовать json.Unmarshal

7. Функция задизайнена так, что при каждом вызове, конфиг будет заново перечитываться

Исходная версия nikolnik, :

В этой функции ужасно вообще все

1. Получишь панику в defer если с файлом что-то не так 2. Ошибка из file.Close() никак не обрабатывается 3. Путь к конфигу захардкожен 4. Условие проверки ошибки открытия файла некорректное 5. Корректно использовать config := &Config{}, без var 6. json.NewDecoder нужен для потоковой обработки жсона, здесь правильно скопировать данные с файла в буфер и использовать json.Unmarshal 7. Функция задизайнена так, что при каждом вызове, конфиг будет заново перечитываться