в D нет множественного наследования и есть один базовый класс Object, от которого неявно наследуется любой класс

Throwable это интерфейс?

Или абстрактный класс?

я же скинул ссылку

прям на этот класс

а класс судя по ней

ну только по ней и надо судить ?

это как бы официальный рантайм языка)

И еще вопрос. Чем лучше исключения кодов возврата?

С кодами возврата много мороки. Каждая функция возвращает код, тебе после этого нужно данный код проверить. И так всё время. Плюс, функции в таком случае не могут возвращать результат, получается нужно постоянно использовать указатели.

С кодами возврата много мороки. Каждая функция возвращает код, тебе после этого нужно данный код проверить. И так всё время. Плюс, функции в таком случае не могут возвращать результат, получается нужно постоянно использовать указатели.

либо использовать tuple, которые в D не самым удачным образом сделаны

Согласен о tuple.

Так есть фунция copy2FTP В ней есть проверка кода ответа при копировании: if(rs.code == 452) { writeln("Not Enough Free Space of FTP Server"); } Т.е. продолжать нет смысла. Имеет ли мне смысл тут бросить исключение и поймать его в main или просто return сделать, что меня тоже в main вернет. Как праивльнее?

return просто вернёт в main и твоя программа исполнится дальше не сообщив, что файл не скопирован? если это нормальное поведение (забить на то, что требовалось выполнить), то, в принципе, можешь и вернуть...

Тоесть логичнее: if(rs.code == 452) throw new Exception("Not Enough Free Space of FTP Server"); и дальше в main поймать. ок понял

в этом случае у тебя это будет обрабатывать код, отвечающий за обработку ошибок, что по сути логичней но если говорить в общем, то ты как архитектор всегда делаешь выбор между возвращаемым значением и исключением — нет единого правильного подхода

сейчас у меня сделано вот так: http://www.everfall.com/paste/id.php?wstqbilx01jp По идее мне в случае любой ошибки нужно вызвать: updateFlags как быть то? А то тут оно не вызовется при нехватке места

может наоборот при успешной загрузке оно не вызовется?

при успешной должно вызываться с флагом 11, а при неуспешной с 13

ну так и напиши так

вообще тут не особо нужны исключения...

вот именно в этой ситуации

тк у тебя выброс исключения подразумевается в блоке try

Немного не логично проверять код, бросать исключение и сразу же его ловить. Для чего тратить зря время, если можно сразу по коду всё понять?

Немного не логично проверять код, бросать исключение и сразу же его ловить. Для чего тратить зря время, если можно сразу по коду всё понять?

вот да

ай блин, тоесть я тут же его и поймаю получается?

А как ты думал?

блин, я почему-то подумал, что уровнем выше)

Не в коде, который ты привёл?

да щас то я понял что там же его и поймаю где кину

Response rs = rq.post(ftpConnString ~ "/" ~ r.arch_path.baseName ~ "/" ~ f.baseName, cast(ubyte[])read(f)); if (rs.code == /* OK */) { updateFlags(r.route_id, 11); } else { // ... }

А по логике мне какой код возврата проверять? "250 Запрос прошёл успешно" прошел успешно == файл залился? Вот список кодов https://ru.wikipedia.org/wiki/%D0%A1%D0%BF%D0%B8%D1%81%D0%BE%D0%BA_%D0%BA%D0%BE%D0%B4%D0%BE%D0%B2_%D0%BE%D1%82%D0%B2%D0%B5%D1%82%D0%BE%D0%B2_FTP

https://en.wikipedia.org/wiki/List_of_FTP_server_return_codes

Видимо, 250 код.

А там нет никакого enum`а с кодами? А то получается ты magic numbers используешь, что нехорошо.

а чем magic numbers плохи?

а ты как думаешь?)

`if (value == 42) { ... }`о чём может говорить?)

только о том, что это проверка пройдёт, если value будет равно 42

А в каких случаях value будет равно 42?

а что это за 42 такое, которому должно быть равно value из кода не ясно

Вот, да.

Автор в коде вообще так делает: https://github.com/ikod/dlang-requests/blob/261df21f2f548e8cf0aaba91aba4976f8e0d55bf/source/requests/ftp.d#L184

Это трюк.

if (value == MAX_ITEM_COUNT) { ... } тут понятней

Т.к. в HTTP 2XX коды являются success ответами.

Автор в коде вообще так делает: https://github.com/ikod/dlang-requests/blob/261df21f2f548e8cf0aaba91aba4976f8e0d55bf/source/requests/ftp.d#L184

это не трюк, а дичь =)

По-моему я такое в phobos видел.

а как он должен был делать? Через Enum'ы?

По-моему я такое в phobos видел.

это не отменяет того факта, что это дичь)

ну в функцию можно вынести маленькую

if (isSuccessCode(code)) { ... }

Иной вариант: if (code >= 200 && code <= 299)

Вариант с функцией, кстати, тоже хорош. Более читаем.

тем более они в d инлайнятся сами, так что накладных минимум

щас посмотрю как в vibed сделано

не везде где сделано будет не дичь)))

это как "курить плохо"

и все такие "да, да, конечно..."

а потом за угол курить =)

https://github.com/rejectedsoftware/vibe.d/blob/b7c408b98f369920d85454be6156ba13abadcbb6/source/vibe/core/drivers/threadedfile.d#L46

тоесть по уму нужно назначить на все цифры текстовые кода и потом уже проверять их?

вот этот код похож на выжимку из Boost.Asio

Автор в коде вообще так делает: https://github.com/ikod/dlang-requests/blob/261df21f2f548e8cf0aaba91aba4976f8e0d55bf/source/requests/ftp.d#L184

Отсюда же: if ( code == 550 ) { Вот я лично понятия не имею, что 550 значит.

тоесть по уму нужно назначить на все цифры текстовые кода и потом уже проверять их?

ну на все цифры ты не сможешь, но на коды, которые действуют в твоей программе это норм

Отсюда же: if ( code == 550 ) { Вот я лично понятия не имею, что 550 значит.

насколько я знаю 5ХХ это код ошибки сервера, а уж в vibe они могли и свой придумать)

вот этот код похож на выжимку из Boost.Asio

весь vibe или тот кусок?

насколько я знаю 5ХХ это код ошибки сервера, а уж в vibe они могли и свой придумать)

Вот-вот. Придумали свой? А что он значит? Вот в чём проблема magic numbers.

ERROR: S client not available

там, насколько я помню коды ошибок надо было свести из MSDN и POSIX

у них названия одинаковые, а коды отличались

хотя, это бы любой делал, кто кроссплатформенно пишет что-то, что работает с сокетами

а, это не тот файл)

тут с файлами работа, ошибся

в общем, коды ошибок с именами - хорошо, особенно, если наружу это выглядывает

а так я любой код могу свести к операции над числом 42

хотя есть контекст

ну на все цифры ты не сможешь, но на коды, которые действуют в твоей программе это норм

вот даже лучше не на цифры символьные описания, а коды enum'ом, чтобы возвращаемый и передаваймый типы были ограничены в вариантах

http://www.everfall.com/paste/id.php?jsdmsjcvfbia блок try-catch выкинуть можно?

Видимо rq.post никогда не кидает исключений. То можно убрать.

В success блоке можешь добавить continue в конец.

а что оно даст?

А ты мне скажи.

ну по идее оно к следующему файлу перейдет не обрабатываю дальнейшие команды типа: if(rs.code == 452)

Если бы ты else if использовал, то не перешло бы.

А, 452? Так зачем тебе его проверять, если ты уже знаешь, что у тебя 250?

так может так тогда сделать? http://www.everfall.com/paste/id.php?j8iyzwvhmcs3

типа если не 250 то что-то другое

Можно так.

а потом, через год, всё-всё-всё переписать =)

Так. А вот теперь интересный момент. Из БД я беру путь из которого нужно скопировать все файлы. Сейчас мой код делаем апдейт, при копировании единичного файла, а не всей папки. Получается мне как-то нужно обновить состояние на успех при копировании последнего файла, верно?

вопрос только — как это сделать то? Разве в foreach можно как-то отловить последий файл

есть for =)

есть индекс в форыче вроде бы ещё

И переменные вне foreach.

И вообще я ничего не понял.

Я беру из БД путь. Получаю из пути список файлов. файлы по одному копирую на FTP.

https://dlang.org/spec/statement.html#ForeachStatement

https://dlang.org/phobos/std_range_interfaces.html#.InputRange.moveFront кто-нибудь знает, что этот метод должен делать по логике?

https://dlang.org/phobos/std_range_interfaces.html#.InputRange.moveFront кто-нибудь знает, что этот метод должен делать по логике?

Перемещать содержимое.