Не идёт запись в файл. В чём проблема?

Question

[Роман Харченко]

Добрый день.

Пытаюсь подружиться с журналированием, не особо получается:

import (
"log"
)
/*******************/
type Client struct {
	Logger        *log.Logger
}

ex, _ := os.Executable()
	path := filepath.Dir(ex)
	pathlog := path + string(filepath.Separator) + "log" + string(filepath.Separator)
	if _, err := os.Stat(pathlog); os.IsNotExist(err) {
		_ = os.MkdirAll(pathlog, os.ModePerm)
	}
	f, err := os.OpenFile(pathlog + time.Now().Format("02-01-2006_15-04-05") +".log", os.O_CREATE,0777)
	if err != nil {
		panic(err)
	}
	defer f.Close()

	//f.Sync()

	client.Logger = log.New(bufio.NewWriter(f), version, log.Lshortfile)
	client.Logger.Print("Test text")

Директория создаётся, если её нет. Файл создаётся. НО! в файл ни чего не записывается.
В чём причина?

Answer 1

[Сергей Тихонов]

bufio.NewWriter(f)
Вот эта штука буферизует запись на диск, ее надо флашить отдельно от самого файла. Без этой обертки (New(f, ...)) в лог запись идет.

Comments

[Роман Харченко]

Называет Copy-Past при изучении зло. Скопировал из примера bufio и не понял его отличия от io.
На деле чуть-чуть иначе, чем Вы предложили переделал:

client.Logger = log.New(io.Writer(f), version, log.Lshortfile)

Спасибо.

[Сергей Тихонов]

Вот теперь совсем странно стало.
io.Writer - это интерфейс, реализуемый типом os.File. Зачем приводить к интерфейсу?

[Роман Харченко]

Дополню сам себя.

client.Logger = log.New(f, version, log.Lshortfile)

В моём коде можно и так, Вы правы! Первый раз когда писал GoLand подсвечивал, что тип передаваемых данных log.New(f...) не соответствует. Потом ещё выше раз 10 менял код создания папки и файла, и видимо завтыкал когда поменял тип f. устал к вечеру уже.

[Роман Харченко]

func New(out io.Writer, prefix string, flag int) *Logger {
	return &Logger{out: out, prefix: prefix, flag: flag}
}

Описание функции, тут увидел io.Writer вот и упёрся рогом в этот тип .

Answer 2

[Папа Стифлера]

Могу предположить, что вот в этом: bufio.NewWriter(f).
Либо делайте Flush(), либо используйте небуферизованный io.Writer.

w := bufio.NewWriter(f)
defer w.Flush()
client.Logger = log.New(w, version, log.Lshortfile)

Comments

[Роман Харченко]

Спасибо. Понял свою ошибку уже. На будущее буду знать про Flush.