1

Использую golang 1.5.
Использую библиотеку gopkg.in/rana/ora.v3 для подключения к Oracle.

В пример Working With The Oracle Package Directly есть реализация использования ресурса Oracle environment:

env, err := ora.OpenEnv(nil)
defer env.Close()
if err != nil {
    panic(err)
}

Мы видим, что в defer закрытие ресурса идёт напрямую через ссылку env.

Но что будет если произойдёт сбой. В реализации OpenEnv мы видим, что при ошибке для env возвращается значение nil. С эмитируем данную ситуацию:

package main

import (
    "gopkg.in/rana/ora.v3"
    "errors"
)

func main() {   
    defer fmt.Println("Hello")

    // env, err := ora.OpenEnv(nil)

    var env *ora.Env = nil
    var err error = errors.New("Test error")
    defer env.Close()
    if err != nil {
        panic(err)
    }
}

Получаем следующее сообщение об ошибки:

Hello
panic: Test error
panic: runtime error: invalid memory address or nil pointer dereference [signal 0xc0000005 code=0x1 addr=0x0 pc=0x48753a]

goroutine 1 [running]:
gopkg.in/rana/ora%2ev3.(*Env).Close(0x0, 0x0, 0x0)

И хоть мы видим, что возникшая ошибка не прерывает выполнение остальных defer методов, но ошибка всё таки фиксируется.

Корректно ли оставлять так?
Или всё же стоит гарантировать, что при evn == nil defer env.Close() никогда не вызовется?
Или в теле метода, переданного в defer, следует проверять env на nil?

defer func () { if env != nil { env.Close() }}()

1 ответ 1

3

defer обычно ставится после проверки на ошибку, как раз из-за того, что большинство функций, возвращающих указатель и error, в случае ошибки возвращают nil. Например:

f, err := os.Open(os.Args[1])
if err != nil {
        panic(err)
}
defer f.Close()

Пример в той библиотеке с этой точки зрения действительно странные. ЕМНИП, все примеры в стандартной библиотеке вызывают defer после проверки. Например, в database/sql.

8
  • А вам не кажется, что использование log.Fatal(err) является тоже не корректным. Ведь реализация log.Fatal содержит os.Exit(1), т.е. выполнение программы буде прервано и "Hello" не будет выведено, т.е. не будут корректно закрыты другие ресурсы.
    – mals
    23 окт 2015 в 9:35
  • Это всего-лишь пример. Заменил на panic.
    – Ainar-G
    23 окт 2015 в 9:36
  • Но, если данный шаблон рассматривать как стандарт, то что гарантирует, что f == nil? А если он не nil, то для надёжности всё таки стоит дернуть Close? Т.е. я хочу сказать что данный шаблон не универсален, в нём есть условие.
    – mals
    23 окт 2015 в 9:45
  • Гарантий нет. Даже если будет ситуация, при которой f != nil, нет гарантий, что f.Close() что-нибудь не поломает, не запаникует или не вернёт другую ошибку. Смотрите по ситуации, по конкретным ресурсам, по документации модуля и по требованиям к устойчивости.
    – Ainar-G
    23 окт 2015 в 9:56
  • 1
    @MaLS, ну, в C# тоже вам никто не гарантирует, что в finally ссылка на ресурс не null
    – hardsky
    26 окт 2015 в 9:05

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.