Почему при открытии файла срабатывает ошибка «The system cannot find the file specified»?

Question

[Vadim Rublev]

Почему-то в блоке с функцией ReadDir() не может открыть файл. Срабатывает ошибка ERR_1.

filesDir, err := ioutil.ReadDir(DirFiles)
    if err != nil { log.Fatal(err) }
                    
    for _, fileDir := range filesDir {
        if filepath.Ext(fileDir.Name()) == ".html" {
            // Чтение содержимого файла построчно.
            my_File, err := os.Open(fileDir.Name())
                if err != nil {
                    fmt.Println("ERR_1. Ошибка открытия файла:", err)
                    return
                }
                defer my_File.Close()

                var scanner = bufio.NewScanner(my_File)
                ...
        }
    }

Хотя такое же открытие файла в рядом стоящем блоке функции Walk() работает нормально.
Почему так?

Answer 1

[Евгений Мамонов]

Предполагаю, что после замены os.Open(fileDir.Name()) на os.Open(path.Join(DirFiles, fileDir.Name())) всё заработает.

Думаю, что проблема вот в чём.
Скорее всего у вас структура файлов близка к этой
./files/file1
main.go
Вы получаете список файлов из папки ./files/, и в os.Open передаётся только название файла (file1.go), т.е. os.Open пытается открыть его в текущей папке, а не в папке ./files/, по этому и не может его открыть.
А если передать в os.Open("./files/file1") - тогда всё заработает (при условии, что есть права на это)

Comments

[Vadim Rublev]

Да, была такая мысль... Но не знал как разрулить.
Walk() же сам следит за директориями, поэтому в нем работает.
Проверяю...

[Vadim Rublev]

А может проще в Walk() ограничить просмотр только указанной_директории(ий)? Можно это сделать?
Тогда уберу вообще функцию ReadDir().

[Евгений Мамонов]

Walk проходит рекурсивно и возвращает полный путь вида "./files/file1".
Если вам нужен рекурсивный проход по папкам - можно использовать Walk, если не нужен - тогда ReadDir будет хорошим выбором

[Vadim Rublev]

Ясно. Я думал можно в Walk() ограничить обрабатываемые директории.
Нужны оба варианта.

[Евгений Мамонов]

Можете описать в чём именно проблема/сложность?
Также прошу написать чем не подходит вариант замены os.Open(fileDir.Name()) на os.Open(path.Join(DirFiles, fileDir.Name()))?

[Vadim Rublev]

Вариант подходит. Только не Join() нужен там. Ищу подходящее...

[Евгений Мамонов]

а чем Join не подходит?

[Vadim Rublev]

Ему в 1-м аргументе список нужен, между элементами которого он вставляет второй аргумент. Насколько я понимаю.
Или по-другому записать надо...

[Vadim Rublev]

DirFiles - это директория с обрабатываемым файлом.
Надо просто конкатенировать ее с именем файла (во 2-м аргументе указан).

Вот такая строка у меня:
my_File, err := os.Open(DirFiles.Join(DirFiles, fileDir.Name()))

[Vadim Rublev]

Просто + сделал. )
Но по вашей идее.

[Евгений Мамонов]

+ не правильно, т.к. будет не кроссплатформенно (в Windows \, а в Lixux /), для конкатенации папок и имён файлов лучше использовать функцию Join из пакета path https://pkg.go.dev/path#Join как я написал в примере, т.е. заменить
os.Open(fileDir.Name()) на os.Open(path.Join(DirFiles, fileDir.Name()))

[Vadim Rublev]

> будет не кроссплатформенно (в Windows \, а в Lixux /)
Хм.... У меня локально на Винде работает. Путь с обычными слешами - "DirFiles" : "../test_html_files/"

Я ж не против Join(), но выражение с ним дает ошибку (см. выше скрин). "Не с чем работать" ему.

[Евгений Мамонов]

у вас оно написано не так, как у в примере
у вас DirFiles.Join, а в примере path.Join(DirFiles
по этому и не работает :)

[Vadim Rublev]

На path он сказал: undefined.
Тогда может быть поясните, что за path? И как он меняет работу Join()? В документации https://pkg.go.dev/strings#Join не вижу его.

[Евгений Мамонов]

Вот этот метод https://pkg.go.dev/path#Join
Работу Join он никак не меняет, Join - это разные функции в разных пакетах.
https://pkg.go.dev/strings#Join - склеивает строки.
https://pkg.go.dev/path#Join - склеивает пути.
Это никак не связанные между собой Join, каждый Join выполняет свою функцию.

В документации к нему есть хороший пример

package main

import (
	"fmt"
	"path"
)

func main() {
	fmt.Println(path.Join("a", "b", "c"))
	fmt.Println(path.Join("a", "b/c"))
	fmt.Println(path.Join("a/b", "c"))

	fmt.Println(path.Join("a/b", "../../../xyz"))

	fmt.Println(path.Join("", ""))
	fmt.Println(path.Join("a", ""))
	fmt.Println(path.Join("", "a"))

}

Вывод

a/b/c
a/b/c
a/b/c
../xyz

a
a

[Евгений Мамонов]

для склеивания строк нужно использовать strings.Join
для склеивания путей path.Join
так будет надёжнее

[Vadim Rublev]

аа.... Сразу бы сказали! )
Не знал про эту функцию.

[Евгений Мамонов]

я не ожидал, что это будет не понятно, таки надо было написать в ответе, что нужно сделать import "path".
в след. раз таки напишу про импорт :)

[Vadim Rublev]

Да просто пояснить: функция path.Join(), а не strings.Join(). :)