потокобезопасным stdout стал, когда его обернули в vibe shared FileLogger(stdout, stderr)

Должен же вайбед понимать когда один из его обработчиков сдох

Должен же вайбед понимать когда один из его обработчиков сдох

обращение по null-pointer не будет отработанно, скорее всего

потому что система прибьёт приложение

Error исключения не перехватываются обычно, что тоже не очень

чтобы понять где падает я бы везде поставил логирование просто в stdout (stderr) и просто запустил бы через systemd, а там бы и время падения и логи посмотрел бы

обычный log из vibe делает это по дефолту

кстати сам класс FileLogger не обеспечивает потокобезопасность

там ручками сделали блокировки

Пасибо, у меня на новой работе щас время нет проверить. Постараюсь выбрать время и затестить все

Дмитрий, а на работе ты на чём кодишь? )

Пока на Питоне тк ArcGIS на него завязан на 100%

Подскажите, как поймать исключение если соединение я создаю в конструкторе: this(Config config) { this.config = config; conn = ds.getConnection(); // в момент коннекта может кинуть исключение }

Не нужно подключаться в конструкторе

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

а ок, пасибо

чnо-то не пойму что ему не нравится:

м... типа это у меня все вне метода получилось и висит в воздухе или как?

хыхы да

try catch вне метода

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

А обработка в конструкторе чем чревата?

да впринципе ты можешь конектится в конструкторе и кидать исключение, но тогда 1) Тяжело будет тестировать класс. Ты не можешь создать объект без подключения к БД 2) Подключение будет неленивым 3) Нельзя создать несколько объектов и расшарить между ними подключение

Конструктор по дизайну хорошего кода не должен содержать никакой логики, только наполнение и инициализация своих свойств.

значит отдельный метод реализовать?

да

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

В тех, где реально требуется лезть в базу

а как его принято называть? Просто Connect?

да как угодно, чтобы понятно было

initConnection(), connectToDb(), dbConnect()

можно назвать getConnection() и там описать логику что если подключения нету то создаем его, иначе возвращаем существующее

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

не к переменной, а к геттеру этой переменной

а геттер уже отвечает за инициализацию подключения

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

ну просто метод который возвращает эту переменную

а... щас попробую и покажу код

http://www.digitalmars.com/d/archives/56.html есть же геттеры

хотя.. там 2001 года сообщения )

что то типа этого?

ага

а что ему не нравится почему ошибка?

о

понял

надо так:

а устанавливать значение через return это хорошая практика?

Да только у тебя не учитвается что метод Connect() могут дернуть несколько раз

и он каждый раз будет пытаться установить соединение

а как это предотвратить?

if (conn != null) { return conn; }

ругается: Error: use '!is' instead of '!=' when comparing with null что-то не пойму почему тут is - она же тип проверяет

я на псевдокоде написал) Правильный синтаксис сам выбери )

я понял, но тут же вроде мы не тип проверяем а значние

или null это тип?

походу да

А что возвращать во всех остальных случаях?

этот метод должен либо вернуть conn либо выбросить исключение

типа такого: else { throw new Exception("Connection is already exists"); }

да нет

Ну это типа синглтон должен получиться

этот метод всегда возвращает соединение, а если оно не установлено, то устанавливает его

не мог бы в коде показать, что-то не допру как это сделать можно

ERROR: S client not available

так я ж написал выше, этот код надо вставить в самое начало метода

как то так? http://codepad.org/44ta5BUs

код ругается на: Error: function dbconnect.GeoDataBase.Connect no return exp;

Connection Connect() // создаем объект подключения { if(conn is null) { conn = ds.getConnection(); } return conn; }

м... тогда метод наверно void надо?

подредактировал

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

вернуть хендл соединения тоже надо

что такое хэндл соединения и что будет если его не вернуть

или null это тип?

нет

просто на null проверка через is

типа стиль

Так а если не делать проверку на то имеется уже ли соединение то откуда оно может повторно то взяться? Ведь по идее экземпляр подключения к бд создается один раз при старте приложения

Так и еще раз напомните. Если я просто ловлю общее исключение вида: Exception e т.е. без указания его типа навроде SQLException то какие минусы у этого подхода?

неудобно

а чем?

Просто я иногда ленюсь искать тип исключения который либа кидает

Это плохо тем что ты все исключения ловишь в одном месте, не всегда для этого подходящем

Если ты работаешь с БД то должен ловить только исключения, касающиеся БД.

Иначе можно поймать не то исключение?

А то допустим ты поймал исключение о том что место на диске закончилось. Твой класс понятия не имеет как это обрабатывать, ведь он только с базой работает

А код который умеет такие случаи обрабатывать, не получит этого исключения

Но ты можешь после того как поймал исключение, кидать его дальше

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

поймал - что-то сделал (вывел ошибку) - дальше бросил это же исключение

А где это бывает полезно?

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

Смысл исключений в том что если ты его поймал, значит весь нормальный код после него не должен выполниться. То есть ты должен вывести сообщение об ошибке, и прекратить выполнять максимальное количество кода в подзадаче.

Не знаю как это сформулировать даже :)

а.... ну понял идею

а не мог бы в продолжении вчерашней темы ответить на вопрос: а если не делать проверку на то имеется уже ли соединение то откуда оно может повторно то взяться? Ведь по идее экземпляр подключения к бд создается один раз при старте приложения

Ты этот метод getConnection() должен дергать каждый раз когда хочешь получить соединение. А обращаться в свойству conn вообще не надо

На то он и геттер.