DMdev talks

2022-08-26 17:54:10 DMdev Code Review! Всем спасибо за участие в опросе. Code Review быть! Когда: 20.08.22 (суббота) в 10:00 по Москве Где: прямая трансляция в Google Meet Продолжительность: 1-1.5 часа Количество человек: до 100 Кто хочет стать участником code review - подготовьте… Открыть/Комментировать

2022-08-23 19:41:28 1. Параметр marshaller изменяется внутри метода.
Best practices:
- clean code (всегда есть доступ к исходным параметрам функции)
- pure function/immutable (не изменяешь входные параметры, а значит безопасно вызывать такие функции извне)

2. Объект класса Marshaller - не потокобезопасный. Как и говорил в посте по code review выше - нужно всегда обращать внимание на работу кода в многопоточности. Поэтому лучше сделать вместо параметра - локальную переменную через jaxbContext (локальные переменные и jaxbContext точно потокобезопасны).

3. Многие писали про закрытие потока StringWriter. Но это не обязательно для данной реализации Writer, потому он что содержит внутри обычный объект типа StringBuffer.

4. - тоже не следует делать для объекта сериализации, ибо ограничений у jaxb библиотеки на реализацию этого интерфейса тоже нет.

5. На null все параметры проверять и пробрасывать исключение тоже не имеет смысла - все методы будут громоздкие и неуклюжие. Используй Optional или обычные @NonNull аннотации, которые генерируют подобные проверки автоматически согласно fail fast principle

#dmdev_code_review

1.5K views16:41

Открыть/Комментировать

2022-08-23 11:34:15 #dmdev_code_review
Недавно наткнулся на примерно вот такой код.
Здесь происходит сериализация Java объекта в xml (обычная строка) с помощью библиотеки jaxb.

Как думаешь, что не так с методом serialize? Или все так?
PS. Предположения/идеи пиши в комментариях Открыть/Комментировать

2022-08-19 11:44:25 DMdev Code Review! Всем спасибо за участие в опросе. Code Review быть! Когда: 20.08.22 (суббота) в 10:00 по Москве Где: прямая трансляция в Google Meet Продолжительность: 1-1.5 часа Количество человек: до 100 Кто хочет стать участником code review - подготовьте… Открыть/Комментировать

2022-08-18 17:52:42 Сейчас появилось свободное время из-за того, что заболел Covid.
К сожалению, это время приходится тратить по большей части на сон и отдых.
Тем не менее, удалось дочитать книгу “Не про бег”, которую как раз-таки мне и посоветовал друг, когда я начал заниматься пробежками.

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

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

2. Торопиться жить - это как брать в долг. Если не угадал с темпом в начале дистанции, то вторую половину придётся идти пешком.

Оба пункта на самом деле далеко не только про бег… Открыть/Комментировать

2022-08-10 11:01:47 На канале уже опубликовано первое видео нового курса Bash!

В первом уроке:
- Что нужно знать для прохождения курса
- GUI & CLI
- Командная оболочка shell
- Командная оболочка bash & zsh
- Что будет разбираться в курсе
- Зачем знать bash

Готов подружиться с Bash?
Переходи по ссылке, заваривай чашечку горячего чая и приятного просмотра Открыть/Комментировать

2022-08-06 17:57:31 Best practices in Code Review

Ни один merge своего кода в main ветку не обходится без code review.
А значит каждый разработчик с этим сталкивается при ежедневной работе.

Для хорошего code review, нужно придерживаться определенных правил “на что обратить внимание”.
Эти правила не только разительно улучшают качество его проведения,
но и делают процесс действительно полезным для обоих сторон (owner & reviewer).

Тогда code review не будет сводится к принципу “лишь бы оставить комментарий”.
Все комментарии будут только по делу и не тратят впустую время разработчиков.

1. Код хорошо спроектирован
2. Функциональность легко читать, а значит использовать и поддерживать другим разработчикам
3. Код и его форматирование придерживаются общего стиля всего проекта
4. Нет привнесенной сложности. Код не сложнее, чем требует от того задача
5. Важен не только сам код, но и его контекст
6. Код безопасно работает в многопоточности
7. Код качественно покрыт тестами, которые точно также легко читаются
8. Если и есть комментарии, то они полезны и отвечают на вопрос “почему”, а не “как”

Делись в комментариях о каком пункте было бы интересно узнать более подробно - расскажу в отдельных постах Открыть/Комментировать

2022-08-04 16:12:00 DMdev Code Review!

Всем спасибо за участие в опросе. Code Review быть!

Когда: 20.08.22 (суббота) в 10:00 по Москве
Где: прямая трансляция в Google Meet
Продолжительность: 1-1.5 часа
Количество человек: до 100

Кто хочет стать участником code review - подготовьте ссылку на github с кодом своего небольшого проекта.
В начале стрима я выберу рандомную ссылку. Из практики успеваю разобрать 2-3 человека.

P.S. Не стесняйтесь, это отличный шанс получить обратную связь по своему коду и помочь улучшить свои навыки!

До встречи в эфире Открыть/Комментировать

2022-07-30 18:35:00 P.S. Для тех, кто не в теме, оставляю ссылку на предыдущие выпуски "Code Review" Открыть/Комментировать

2022-07-30 18:34:17 Открыть/Комментировать