Но соединение надо установить всего 1 раз

Не мог бы глянуть мой код и если можешь прокомментировать основные косяки: https://github.com/bubnenkoff/Geoportal2 Просто метод же у меня только разво при старте приложения вызывается или все же нет?

Я тебе даю некие общие рекомендации исходя из своего опыта ООП проектирования и разработки. А в частном случае сам думай :)

А тоесть это частный случай? Понял

Но не мог бы ты или кто нить другой сделать небольшой код-ревью проекта. Хоть какие-то явные косяки чтобы указали мне

Ты дай ссылку на конкретный файл, а то ты просто на проект скинул

Я так понял что твое приложение не работает без БД, тогда тебе во всех местах где catch (Exception e) { надо в конце добавить throw e чтобы дальше пробрасывать исключение

И типа в самом верхнем catch его можно было поймать?

Ну да

Но можешь даже и не ловить, просто вывалит трейс в консоль )

И у пользователя будет больше информации о том что произошло.

так, ну вот допустим файл адекватно написан?

https://github.com/bubnenkoff/Geoportal2/blob/master/source/app.d

Есть какие-то сильные косяки?

Да

https://github.com/bubnenkoff/Geoportal2/blob/master/source/app.d#L102 вот этот метод отработает в любом случае но программа при этом не упадет, хотя должна

типа если не смогли что-то там, то нужно эксепшен кинуть?

да, и выйти из приложения, дальнейшая работа бессмысленна

так сделать: catch(Exception e) { writeln("Can't setup DB"); writeln(e.msg); throw e; }

@DmitryBubnenkov очень странно, что ты не понимаешь в чём смысл исключений, потому что это достаточно простая концепция Пример: ты переезжаешь в другую квартиру и заказываешь машину и грузчиков отдельно. Грузчики приезжают, а машина нет — исключительная ситуация. Что могут делать грузчики? Грузить в несуществующую машину?)) Могут конечно просто перекладывать мебель из одного места в другое, но это бессмысленно. Но если, например, приехала машина, но не приехали грузчики, ты можешь обработать эту ситуацию и попросить помочь тебе соседей самостоятельно перетащить мебель.

Соответственно каждая исключительная ситуация образуется когда что-то идёт не по плану

Мне просто опыта не хватает — мне никто не показывал как правильно

так сделать: catch(Exception e) { writeln("Can't setup DB"); writeln(e.msg); throw e; }

Да так

Но вообще это обобщенное исключение можно ловить на уровне выше

и там в main все обернуть блоком try

зачем?

если программа не может работать, например, без подключения к базе, то какой смысл оборачивать в main всё?

что она сделает то в итоге?

м... а ну хотя наерно смысла нет... тут же получается throw приложение просто прервет?

только сообщение выведет?

ну да я думал хотя бы сообщение

и так исключение сообщение выбросит

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

и приложение просто прекратит работу. Просто я думал работу приложения вкидыванием голого throw прерывать нельзя

и приложение просто прекратит работу. Просто я думал работу приложения вкидыванием голого throw прерывать нельзя

почему нельзя?

не стильно

но не нельзя

Ну я читал что там с GC какая то заморочка, но это без отношение к thow

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

так что gc, rc вообще без разницы

хотя некоторые ресурсы (не подконтрольные системе) высвободить стоит, но тут совсем другая история)

я понял, но почему не void? Ведь мы просто значение переменной ставим

Олег, не мог бы глянуть вот этот момент про хэндл

хотя тут всё-таки стоит высвобождать всё, вспомнил ситуацию: пробовал работать с несколькими вебками одновременно в одном приложении и если оно падало драйвер не мог потом эти камеры отдать (которые по usb к ноуту были подключены), а потом так и вообще всё легло с kernel panic (баг в дровах был для usb3)

Олег, не мог бы глянуть вот этот момент про хэндл

что где?

Файлы тоже хорошо бы закрывать

ну там мне Павел говорит, про какой-то хэндлер

Я не понимаю вообще что это такое

Файлы тоже хорошо бы закрывать

ну, вроде, с этим система справляется норм, не?

Не уверен

в любом случае это более правильный подход

всё аккуратненько освободить, закрыть и разъеденить

Надо поэкспирементировать, но в любом случае правильнее будет закрывать, а не надеяться что система всё всегда сделает как надо. А то это как вёрста под ие6 с незакрытыми тегами "браузер сам закроет"

ну там мне Павел говорит, про какой-то хэндлер

я тоже пока не понял)

А как ты считаешь void надо делать или как павел предлагает через return?

наверное не считаю никак))) потому что это конкретный случай реализации архитектуры и общие какие-то правила я бы не стал советовать

я бы сделал return если бы это было удобней конкретно мне в каком-то конкретном случае

хотя, наверное в твоей ситуации сделал бы return

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

геттер это делал бы за меня

если бы инициализация поля происходила бы всегда в конструкторе и нельзя было бы создать объект без создания этого поля, то я бы не делал геттер

блин не понимаю разницу: "потому что тогда бы не парился о наличии объекта при попытке его использовать" если же его нет, то я его создаю. Какую роль тут return играет

А на сколько у меня этот файл адекватно написан? https://github.com/bubnenkoff/Geoportal2/blob/master/source/dbconnect.d

catch(Exception e) { writeln(e.msg); }

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

как быть если к примеру метод: getBaseMapVectorLayers упал ? что возвращать и куда?

Но программе все равно упадет

ERROR: S client not available

Только с более непонятной ошибкой, типа cant access to object because it is null

во да только что упала! ты как в воду смотрел!

object.Error@(0): Access Violation 0x0044E752 in vibe.data.json.Json dbconnect.GeoDataBase.getBaseMapVectorLayers(vibe.data.json.Json)

?

Так, ну вот как мне быть-то? Что делать если один из методов обращения к БД падает?

Типа оборачивать каждый метод в try ?

и там прокидывать исключение уровнем выше?

машины нет, что делать?)

ничего не делать)))

если нет подключения к базе, то прога должна упасть

дык я не могу понять почему блок подключения обернутый 10 раз catch ничего не ловит

он ловит, ошибка вне его возникает

когда код пытается использовать переменную в которой нету раельного коннекта

а почему вне него? Или оно не последовательно работает? Я думал так: прога проверяет есть ли коннект, потом запускает метод получения данных из него

А тут не понятно почему кооект с неправильными реквизитами ничего не падает на них

типа неправильное подклчение заполняет объект коннекта?

Statement stmt; try { stmt = conn.createStatement(); } catch(SQLException e) { writeln("Снова пытаемся поймать и не получается: ", e.msg); // 0x0044E76B in vibe.data.json.Json dbconnect.GeoDataBase.getBaseMapVectorLayers(vibe.data.json.Json) dbconnect.d(235) }

Statement stmt; try { stmt = conn.createStatement(); } catch(SQLException e) { writeln("Снова пытаемся поймать и не получается: ", e.msg); // 0x0044E76B in vibe.data.json.Json dbconnect.GeoDataBase.getBaseMapVectorLayers(vibe.data.json.Json) dbconnect.d(235) }

У тебя здесь object.Error@(0): Access Violation, а не SQLException

А что это значит?

Это значит что ты ловишь исключения не того типа, который там выкидывается

И судя по всему у тебя там null проскакивает

При парсинге json каких-то полей судя по всему не хватает, а дальше уже как оно там сделано: может SQL запрос тупо выдает null при ошибке, может исключение, может он вобще ничего не делает, а можт БД не заполнена

Отладчиком смотрел?

С отладчиком можно все catch поубирать, и просто дебажить во все поля

"Это значит что ты ловишь исключения не того типа, который там выкидывается" А базоый Exception все типы ловит или нет?

В документации D должно быть написано, но наврное да

У меня пароль к ней не верно указан и я не могу понять почему оно так странно себя ведет

хотя и с правильным паролем что-то падает так же

Отладчиком его

Каким проще всего? Рассчет именно на новичка

Ребята в чате подскажут

"Это значит что ты ловишь исключения не того типа, который там выкидывается" А базоый Exception все типы ловит или нет?

Нет. Есть ещё еррор. Эксепшн и еррор всё от фроубл