Разбор кода DAO Layer на Java

Продолжение серии баек про Java. На этот раз я разбираю код одного моего студента и комментирую частые ошибки
Полный плейлист: • Основы архитектуры при...
Учебный центр: bit.ly/2N4xZUm

[@andriizarazka8671]

Здравствуйте Сергей! Отличное видео, но неплохо было б если бы Вы выложили это исходник. Хотелось посмотреть как всё организовано, так как в инете нет толковых примеров.

[@AlexPa1986]

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

[@kekuh1]

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

[@maxlich9139]

За обзор кода респект. Это ультра-удобно, наглядно, полезно, хорошо запоминается и позволяет хорошо понять тему, разобраться в ней. Продолжайте в том же духе, Сергей.

[@аавыф-б4о]

Спасибо, всё понятно и системно. Объясняется так продуманно и прозрачно, что даже мне, 1Снику (на жаве никогда не писал) всё воспринимается как родное. И куда, и что, и зачем именно так, и почему, и что будет, и как ты потом будешь читать пустые логи, с внезапно и взрывообразно переполненными коннекшнами на скуле, посреди ночи... У нас только переменные длиннее, мы не паримся, у нас это даже в гайдлайнах. Типа, пусть занимают пол-экрана. Читается бегло текст любого размера (по кол-ву букв) - имеет значение только количество перечисленных сущностей. Но в жаве областями видимости управлять гораздо удобнее - появляется возможность сократить имена и при этом не пересечься. Поэтому у нас длиннее. Умножить количество всех объектов что отображены в БД, на количество возможных DAO-имплементаций для любого возможного скуля... Нуда, такое. Где ORM-объектов 1500 штук в системе - там легко может стать и 5000 через полгода, за счет реализаций дополнительных хотелок заказчика. Было 7500 DAO-объектов (по 5 на каждый ORM-объект)... Стало 25000. Мда...

[@TheHNKNTOS]

Сергей большое вам спасибо за ваш труд. Формат видео действительно отличный.

[@АлександрДворцов-ш1б]

Сергей, давайте продолжение! Материал у вас действительно уникальный!

[@woodscrock]

Из 2020 -> актуально :) Спасибо!!!!

[@user-sankarsana]

Очень познавательная лекция! Спасибо! Есть несколько вопросов: - Каждый раз (в каждом методе) нужно создавать и закрывать connect??? Это же затратно по времени и ресурсам? - Будет ли продолжения про БД, леера, тесты (очень важно с реальными примерами, как здесь)? - Посоветуйте, пожалуйста исходный код приложения, для примера. Уровня hello world, как Вы говорите. Где есть ui, бизнес логика, база данных. И где всё правильно организованно. Очень не хватает подобного уровня лекций. Большое спасибо!

[@аавыф-б4о]

Мы, например, передавали коннект в параметрах некоторых методов, например конструкторов, и он мог быть равен null (т.е. еще не установлен). А конструктор брал уже установленный, либо устанавливал новый коннект (если null) и приравнивал его в параметр. Т.е. т.о. мы как бы "закешировали" коннект в параметре между вызовами. Потом мы привязали конеект к пользовательской сессии. 1 пользователь (точнее сессия) - 1 коннект. Это энтерпрайз был, не веб.

[@maksimpetrov109]

Отличный формат. Очень понравилось. Все наглядно.

[@аавыф-б4о]

Сергей, очень (ну прям ОЧЕНЬ) нравится как вы проводите аудит кода, смотрится от корки до корки не перематывая. А еще такие ролики есть/будут на канале? Вот прям с IDE-шкой, кодом, и вами?

[@skisportsergievposad5600]

Сергей,вы как всегда Немчинский , и как всегда лучший!)

[@abdullahabbas3729]

Sergey очень доступно объясняете

[@mmaltsau]

Здравствуйте, Сергей. Спасибо за видео. Появился такой вопрос: Вы говорите, что уровень бизнес логики не должен иметь зависимости на всякие sql штуки. В моем проекте появилась необходимость организовать транзакции. Чтобы реализовать это, необходимо иметь один connection для нескольких DAO классов, т.к надо для connection установить autocommit(false), после выполнения запросов сделать connection.commit(), а в случае ошибки сделать connection.rollback(). Если каждый dao класс будет получать свой connection то никакое connection.rollback() не сработает. Вот и получается что у меня connection создается на уровне бизнес-логики и устанавливается через конструктор в классы dao. Как можно красиво решить эту задачу, чтобы бизнес-логика не зависела от connection?

[@georgii6324]

Сергей, я заметил, что в этом примере, в Domain Layer'e находятся классы, которые вытаскиваются из DAO. Эти классы что из себя представляют? DTO или Domain Models? Если Domain Models, то они же должны доставаться из Repository? А если DTO, то значит они не содержат никакой бизнесс логики и нету им места в Domain Layer'e? Я немного запутался :) И еще один вопрос, если есть Domain Models, то это значит, что приложение построено по архитектуре DDD или необязательно? В обычном 3/4-layer'ном приложении вроде нет Domain Models, там всей бизнесс логикой заправляют "процедурные сервисы". P.S. Сорри за много букав :)

[@Alf_GS]

Добрый день, Сергей! Подскажите как в реализации с DAO решить такую проблему (вынесем за скобки актуальность паттерна и подхода в целом, в моём случае он актуален :) ). Добавляем/меняем две реляционно-связанных сущности, и получается, что нужно обратиться к двум разным DAO-классам для их инсерта/апдейта. 1. На каком уровне это делать? На доменном? Или инкапсулировать в DAO одной из сущностей добавление/изменение второй? Не сильно ли тогда DAO одной сущности распухнет? 2. Как корректней объединить это в одну транзакцию? Тогда нужно выносить коннект и понимание транзакции из DAO выше, в доменную логику? 3. Если в транзакции участвует 3-я сущность, которая реляционно не связана с предыдущими двумя, на каком слое манипуляции с ней должны происходить в случае если в ответе на первый вопрос была инкапсуляция в DAO? В этих двух постах под этим видео детальней схожие вопросы звучат, но ответов, к большому сожалению, не сохранилось :( kzbin.info/www/bejne/nJ2wd3x5pbWZmas&lc=Ugj7RHi45uXgCXgCoAEC.84w0D5FXbAo7-KxAXGYTlr kzbin.info/www/bejne/nJ2wd3x5pbWZmas&lc=UgiKK3fbo7VNgHgCoAEC

[@КонстантинКонашенков-и2ф]

А не проще было бы сделать DAOFactory -> PostgreSqlDAOFactory. И уже PostgreSqlDAOFactory будет отдавать коннекшен ?

[@ОльгаЛисицына-у1ж]

Присоединяюсь к вопросу

[@АлексейМешков-з8э]

Тоже интересно

[@SergeyNemchinskiy]

извините. а зачем? Чтобы что?

[@viktor1463]

Супер спасибо , жду про тестирование :)

[@АлександрБондарев-ъ7ф]

Хороший формат. Понятная подача информации.

[@lnnnq]

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

[@alexpepper2146]

на 9:45 где говорится про ексепшин из препаред стетмента, мы не попадём в final где resultSet null, но мы туда попалибы если бы был ексепшин при получении resultSet

[@СергейДедков-ц9э]

Здравствуйте, Сергей. Спасибо за видео, очень интересно, изложение радует) Есть пожелание услышать про такие паттерны, как Паттерны архитектуры источников данных Row Data Gateway (Шлюз к данным записи) Active Record (Активная запись) Table Data Gateway (Шлюз к данным таблицы) Data Mapper () Паттерны обработки Объектно-Реляционных метаданных Metadata Mapping (Распределение на основе метаданных) Query Object (Объект-запрос) Repository (Репозиторий) Паттерны логики сущности Transaction Script (Сценнарий транзакции) Domain Model (Модель области определения) Table Module (Обработчик таблицы) Service Layer (Сервисный уровень) Планируете ли вы рассказывать про это?

[@222.2АндрійЛяшенко]

Супер інформативне!)

[@ВасилийГоловко-д9и]

В блоке finally отдельно делаем для каждого close блок try? Для coonection, для statement и для resultSet. Т.к. если будет один блок try для их закрытие, и например выпадет exception для connection, то try прервется и не выполнится закрытие для statement и для resultSet

[@tzofeolam]

Закрывать ResultSet самому обычно не надо. Он прекрасно закрывается автоматически во время закрытия Statement или PreparedStatement. В свою очередь Statement и PreparedStatement так же закрываются автоматически после закрытия Connection. Разница тут в том, что обычно мы хотим продолжать использовать Connection после окончания работы с конкретным Statement или PreparedStatement (поэтому их нам нужно закрывать руками) но после получения ResultSet старый Statement или PreparedStatement нам обычно уже не нужен и мы хотим закрыть их оба, что можно сделать закрыв лишь Statement или PreparedStatement.

[@tedirensmusic]

Добрый день! Подскажите. В проекте используется jpa. Одна из сущностей содержит поле, которое в случае какого-то действия должно увеличиваться или уменьшаться на указанное число. При использовании SQL это выглядело бы как-то так: SET field = field + 1. Как подобное реализовать используя ORM? Если я просто вызову "setter" на сущности, то в случае если кто-то в это же время попытается изменить эту сущность в другом потоке, то я потеряю часть данных ибо "last commit wins". Интернеты рекомендуют использовать версионирование. Но это просто приводит к исключению StaleEntityException в случае коллизии. А мне нужно просто увеличить поле. Спасибо!

[@user-olegmish]

@Sergey Nemchinskiy Спасибо за видео! После просмотра сюжета возник следующий вопрос. Может более изящьным решением было разделение DAO слоя на два слоя. Слой DAO и слой Service? В слое ДАО мы работаем с SQL, а в сервис слое управляем транзакцией.

[@аавыф-б4о]

нет, Сергей сказал что более изящным решением было бы реализовать 1 универсальный (независимый от вида скуля) DAO-объект на каждый 1 ORM-объект, и сделать их так чтоб не менять ни один из них при переезде на другой скуль. Т.е. не использовать специфические для данного скуля фишки, а только универсальные, чтоб на любом скуле работало.

[@Alellas]

Сергей, возможно я еще не все видео посмотрел, но во многих вы обходите стороной такую, на мой взгляд, замечательную технологию, как myBatis (в прошлом iBatis). Сразу скажу, что свой уровень как разработчика я не считаю высоким. И у вас я именно интересуюсь мнением, а не вступаю в спор. Но тем не менее у меня есть опыт перевода целого проекта с JPA на myBatis именно из-за читаемости, необходимости тюнинга и поддержки. Так же вы неоднократно говорили про необходимость знания SQL - т.е. мы по умолчанию считаем что человек, зовущийся Java программистом может без проблем использовать myBatis. Не кажется ли вам, что myBatis более оптимальный и читаемый вариант? Есть конечно косяки с тем что знаки больше и равно он так просто не съест, но это уже мелочи. Спасибо.

[@Alellas]

+Sergey Nemchinsky ясно, учту. Спасибо!

[@maxlich9139]

А есть что-нибудь по юнит-тестированию и логированию? Если нет, то расскажите, пожалуйста об этом, Сергей.

[@adelkhalitov5984]

По поводу дао, касаемо метода дао и открытие соединении там очень интересно. Но только что если это транзакционный запрос в рамках одного инстанта и нескольких дао сущностей. Тем более любой запрос на чтение тоже в рамках данной транзакции. Я к тому что это накладно по поддержке. Я понимаю транзакция в рамках 2х микросервисов, можно что то подумать, там стоит наверное акцентировать внимание на 2х разных транзакциях в разных бд. Но в рамках одного инстанта, зачем??? Можно больше ценности пожалуйста, очень интересно. И если запрос синглтон почему бы не отдать соединение синглтон?

[@EdwardNorthwind]

А это не слишком медленно, при таком подходе - на каждый чих подымать и вырубать конекшн? Я бы еще понял, если бы у нас был пул конекшинов, и мы из него брали уже поднятый свободный. Но вот так? Будут заметные задержки при каждом подключении. Зайдите в любой менеджер СУБД и сделайте конект, вот прикиньте почти также каждый раз.

[@GAVVVR]

Я правильно понял, что лучше не стоит создавать DAO интерфейсы, как в этом проекте, а лучше сразу делать DAO классы? (по крайней мере для такого игрушечного проекта). Просто в интернете я видел много примеров с интерфейсами DAO. За видео спасибо, очень познавательно, подписался.

[@GAVVVR]

+Sergey Nemchinsky я сейчас делаю по сути такой же HelloWorld проект на сервлетах, только другая предметная область. И тоже думал, что в таком игрушечном проекте никак не буду зависеть от базы данных (использую H2). У меня три типа пользователей и один тип имеет отдельные поля. В Java-коде используется наследование. А в SQL я воспользовался неким шаблоном под названием TablePerType, чтобы "имитировать" наследование. И вот теперь не могу понять, как вставить одну запись в обе таблицы, потому что ID в основной таблице-предке, генерируется последовательностью, а во второй таблице этот ID должен совпадать с ID в первой для данной записи. На мой вопрос на SO (stackoverflow.com/questions/34964062/how-should-i-insert-new-data-with-tablepertype-table-structure), как быть, люди отвечают, что такое SQL выражение будет привязано к конкретной реализации РСУБД, так что, возможно, придется писать H2xxxDao классы.

[@sedilpioestma8802]

Сергей, можете пожалуйста объяснить какой смысл паттерна DAO в данном контексте? Везде пишут что DAO - это паттерн инкапсулирующий хранилища и механизмы доступа к ним: благодаря зависимости от интерфейсов мы можем подставлять файловые хранилища, различные виды баз данных, XML и так далее. Посмотрев ваше видео и почитав комментарии я понял вас так: мы создаем классы DAO и классы Entity, при этом мы избавляемся от интерфейсов, фабрики DAO и множества реализаций типа PostreSQLDAO и так далее. А что осталось в сухом остатке? Просто обычные классы, которые инкапсулируют SQL запросы и являются точкой взаимодействия с БД. Это уже не дотягивает до паттерна...

[@Otruiniy_Grybochok]

такой вопрос если нам не нужно обрабатывать исключения которые могут вылететь при закрытии rs, можно ли просто сделать catch по Exception и тогда по идее все исключения которые могут вылететь из rs.close(); будут перехвачены .

[@HowItWorks]

Будете смеяться, но на моей практике было пару раз так, что сервер ложился из-за лог файлов и бекапов. Тупо заканчивалось свободное место. Вроде бы не много места занимает, но накапливается и совершенно внезапно выясняется что больше писать некуда.

[@аавыф-б4о]

Присоединяюсь. Часто кончалось (где-то) место из-за логов. Но расследование показало, что место периодически кончалось не из-за размеров логов само по себе (ну кончалось бы оно не каждые 2 недели - а каждый месяц-полтора - но всё так же неожиданно и аварийно - если б сократили логирование в нексолько раз) - а из-за отсутствия автоматической обрезки логов (месячной давности они всёравно не нужны, допустим) и плюс что вообще никто не следил за свободным местом на сервере, т.е. никто не отвечал за это (это если более глобально).

[@GAVVVR]

05:15 я правильно понял, что проблемы будут только если Connection сделать статическим полем? так то Connection будет отдельным для каждого объекта класса DAO.

[@lEpIvIaKl]

Жесть, Сергей когда-то код писал оО

[@ИмяФамилия-б3л7б]

А можно тоже самое но на самом начальном уровне??? для "самых маленьких" ))))

[@аавыф-б4о]

А смысл, "то же самое"? Для маленьких нужно совсем другое.

[@wepko]

Это был я. Ставь лайк

[@salaleser]

Как обычно, все круто

[@Чонжинди-в2о]

здраствуйте Сергей.могли-ли вы сделать коротенькое видео по алгоритмам?сортировка или что вроде этого...

[@Чонжинди-в2о]

***** .Роберт Лафоре структуры данных и алгоритмы ///////// Алгоритмы на java 4-е издание Роберт Седжвик,Кевин Уэйн....пытаюсь разобраться с алгоритмами в этих книгах.только вот очень трудно по книжке учиться.да и думаю не все же алгоритмы используются.я имел в виду не могли бы вы рассказать самое нужное из этих книг??

[@Чонжинди-в2о]

***** спасибо.тогда я наверное это дело отложу.

[@nikitachizhik8786]

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

[@аавыф-б4о]

скорей всего, у вас медленный интеренет и видео показывалось низкого разрешения. смотрел в 1080 - весь код виден настолько идеально, что скорей всего еще на 1-2 ступеньки ниже - тоже должно читаться (не пробовал).

[@romanreal7]

а какие реалии нынче с этой темой? вот так вот на чистом JDBC никто не работает, а используют как минимум Hibernate, Spring Data, но тем не менее его знания требуют на собеседовании или как?

[@asidorov01]

+Sergey Nemchinsky , Это нужно объяснять студентам, потому что они так начнут кодить реальные проекты. И потом такой код только выкинуть, поскольку поддерживать такой код нереально. Ценность такого кода для клиента резко отрицательная.

[@Anton_Rozhanskii]

666 лайков, прям не хочется менять

[@andrii_popov]

1) Plain JDBC - это жесть; 2) JDBC + try-catch with resources почти такая же жесть; 3) Spring JdbcTemplate - шаг к свету, 4) JPA/Hibernate - это ОК, 5)Spring Data JPA - это космос. 6) Добвьте сами в будущем...

[@Alex11Fox]

JDBC +Repository

[@FlawlessVictory-j8j]

@@Alex11Fox Репозиторий - это вы имеете ввиду интерфейс внутри спринга? Спринг Дату уже назвали

[@Alex11Fox]

@@FlawlessVictory-j8j да, это интерфейс в спринге. Крч это Spring Data JPA. Учи JPA и будет счастье.

[@Alexander-mj3jk]

насчет 9:40 - если вылетет экзепшен, то мы пролетаем мимо finally c resultSet.close() А что насчет аннотации@Clenupиз projectlombok? Помоему удобно, даже удобней java 8, и позволяет избежать этой аццкой простыни.

[@Alexander-mj3jk]

Vadya Chmil если в строках 97-98 вылетает экзепшен - мы не входим в try который в 99 строке, и поэтому пролетаем мимо блока finally строк 111-118. разве что логгер в 100 бросит экзепшен. тогда действительно то finally о котором говориться, не пролетим :)

[@Alexander-mj3jk]

Vadya Chmil При чем тут я? Автор видео моделировал такую ситуацию и утверждал по запарке, что мы туда попадем.

[@Dream1Lord]

Vadya Chmil "Ну а почему autoclosable не используется, то это вопрос" - Потому что Java 6

[@NickRuster]

Ваша продуктивность равняется = kzbin.info/www/bejne/iaWmZpuHicyri80

[@HowItWorks]

Так себе код. Уже очень давно так никто не делает.

[@Otruiniy_Grybochok]

я бы переменную назвал pStatement

[@Otruiniy_Grybochok]

Спасибо, только учусь, буду знать)

[@maxlich9139]

тогда уж prStatement. Но всё же statement звучит лучше.

[@lindx2533]

мелковато конечно, глаза устают всматриваться

[@OlegMarchuk]

PrepareStatement ps; очевидно же

[@Vitsed]

P.S. не очевидно. )))

[@abdullahabbas3729]

Кстати я индус и я не пишу так "st")

[@siriusdiamond]

ведь им разговаривал Ленин

[@EdwardNorthwind]

Вроде что-то показал, но в том то и дело, что что-то... Да, показать типичные ошибки новичка важно, но люди тут именно для того, чтобы понять как правильно связывать слои, что и как в каждом слое делается. Хотя бы ДАО+Домен. А в итоге, вообще не понятно, что обозревали, и как оно взаимодействует.

[@patison8868]

зашел посмотреть как организовать DAO, а тут на те, название, конекшен, эксепшены, логи, ну и в конце "не пишите интерфейсы для дао"

[@maxlich9139]

Я бы сказал - наезд. Про само ДАО правда было мало рассказано. Приходится смотреть в картинку, и додумывать, как там это всё устроено, и как нужно делать.

[@sergeyshestakov4936]

Спасибо, нужно было посмотреть как рисовать uml, в итоге залип и до сюда досмотрел )

[@greegav]

Вот ссылка на Гитхаб с исходниками была бы бесценна. За остальное огромное спасибо.

[@greegav]

Sergey Nemchinsky Перезалить на гитхаб :) тяжко перенабирать показанное, тем более глубоко не всё показано.

[@tzofeolam]

15:48 неверная точка зрения. Логгер никогда не напишет "и ещё столько-то строк" вместо исключения внутри исключения. Он это напишет лишь в случае слишком длинной цепочки вызовов. Вот простой код, который это демонстрирует: package org.example; import org.apache.log4j.Logger; import org.apache.log4j.PropertyConfigurator; public class MainClass { private static final Logger logger = Logger.getLogger(MainClass.class); public static void main(String[] args) { PropertyConfigurator.configure(MainClass.class.getResourceAsStream("/log4j.properties")); MainClass mc = new MainClass(); try { mc.fu(0); } catch (Exception e) { logger.error("WTF?", e); } } public void fu(int n) throws Exception { if (n > 100) { throw new Exception("Final cause exception"); } try { fu(n + 1); } catch (Exception e) { throw new Exception(e); } } } После запуска получится очень длинная цепочка из "Caused by", содержащая все без исключения места бросания исключений. Заканчиваться она будет вот так: Caused by: java.lang.Exception: java.lang.Exception: java.lang.Exception: Final cause exception at org.example.MainClass.fu(MainClass.java:29) at org.example.MainClass.fu(MainClass.java:27) ... 99 more Caused by: java.lang.Exception: java.lang.Exception: Final cause exception at org.example.MainClass.fu(MainClass.java:29) at org.example.MainClass.fu(MainClass.java:27) ... 100 more Caused by: java.lang.Exception: Final cause exception at org.example.MainClass.fu(MainClass.java:23) at org.example.MainClass.fu(MainClass.java:27) ... 101 more Process finished with exit code 0 Дело вовсе не в размере логов, а в их читаемости. Дублирующее логирование исключений приводи к тому, что такие логи труднее читать.

[@artemshpykuliak7842]

Народ, может кто-то знает где можно подобные исходники взять? похожая задача, мне бы просто архитектуру глянуть и что там как с чем вяжеться. Спасибо.)

[@svetopolk]

1. Круто, очень понравился формат 2. Зачем копировать классы с именем постгрес на имя майэскуэль... Наверное можно порефаторить и просто их переименовать? Я понимаю, это тоже не очень хорошо, но всяко лучше, чем копировать код. Кстати еще можно забить на название, будем работать с Mysql, хотя классы будут от postgre. 3. Уроки хорошие, но для краткости, я думаю можно убрать все эти "я поеду в отпуск, я организую встречу". Смотреть это будут и через год. Ты уже 100 раз вернешься из отпуска... зачем это надо?

[@cgstudio2010]

svetopolk У видео есть дата публикации.

[@lexzcq]

svetopolk по-поводу второго вопроса - переименовывать - это тоже не комильфо, нужно писать универсальные классы, которые можно использовать как для работы с MySQL так и для работы с PostgreSQL или Oracle.

[@viorelvior5686]

Откуда могу взять этот код ??? Спасибо за видео урок.

[@eney1975]

а я постоянно пишу название базы в названии классов (((

[@аавыф-б4о]

Это точно неправильно, относится к архитектурным ошибкам. Вы смешиваете слой сервера приложения (где код программы и названия классов и прочие имена переменных) и сервера БД (где адрес базы, название скуля, имена таблиц). Т.о. вы нарушаете трёхзвенку. (Классическая трёхзвенная архитектура: клиент или фронт - сервер приложения - сервер БД).

[@HalfbackRUS]

нах тут вебка на 1/4 экрана и почему такой мелкий шрифт? а зачем автор напялил националистическую символику и вещает на русском языке? лол. классный канал.