Skip to content

Базы данных#15

Open
lepfel wants to merge 3 commits intoiaulitin:masterfrom
lepfel:master
Open

Базы данных#15
lepfel wants to merge 3 commits intoiaulitin:masterfrom
lepfel:master

Conversation

@lepfel
Copy link

@lepfel lepfel commented Oct 21, 2019

Реализован доп. класс Users для хранения полученных из базы пользователей, а также выполнены задания по получению данных из базы в MainJava.
Также выполнено задание по написанию метода сравнения баз данных без использования циклов в классе CompDatabase. Как работает метод можно увидеть в CompAppl.

…елей, а также выполнены задания по получению данных из базы в MainJava.

Также выполнено задание по написанию метода сравнения баз данных без использования циклов в классе CompDatabase. Как работает метод можно увидеть в CompAppl.
…елей, а также выполнены задания по получению данных из базы в MainJava.

Также выполнено задание по написанию метода сравнения баз данных без использования циклов в классе CompDatabase. Как работает метод можно увидеть в CompAppl.
@iaulitin
Copy link
Owner

Хорошо. Стало почти совсем хорошо.
Теперь давай перейдем к следующему шагу.

Насчет ДЗ4

  1. Пробегись еще раз по коду и проверь, что camelCase-именование соблюдено. Местами остались огрехи.
  2. В 4х методах из 5 ты одинаково выбираешь пользователей в ResultSet, затем бежишь по ResultSet. Есть такой принцип - DRY - don't repeat yourself. Обычно среда разработки подсказывает, что у тебя код дублируется. Повторяющиеся куски функционала следует выносить в отдельные функции. Сделай, пожалуйста.
  3. При сортировке списка пользователей ты со скобочками переборщила.
  4. Почитай про JavaDoc. Оформи комментарии к методам в виде JavaDoc. можно на русском.

Насчет ДЗ5

  1. Замечания по форматированию (операции во вложенных лямбдах было бы неплохо вынести на новые строки)
  2. Слушай, а зачем тебе выполнять select *? Если в таблице 100 колонок, а тебе нужно сравнить только по одной, то ты получишь большой оверхэд и по чтению из БД (с диска компьютера) и по затраченной памяти.
  3. Помнишь, мы говорили о том, что не нужно лишний код заворачивать в try-catch? Вот в твоем вложенном try-catch ты допускаешь ошибку. Почитай, в каких случаях ResultSet может выбросить SQLException и сверься с твоим сообщением, выводимым при поимке ошибки.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments