HW 2.1 by matiko-ma · Pull Request #2 · matiko-ma/cv_builder

matiko-ma · 2025-10-11T20:55:22Z

Изменения

HTML/CSS:

Доработка мелочей
Адаптация под планшеты и телефоны

JS:

Извлечение информации из различных полей
Функция парсинга карточек

To be done

Доработать drop-down list согласно макету
Доработать меню в версии для телефонов согласно макету
Добавить функционал карточке (удаление, перемещение)
Добавить возможность редактировать содержимое карточки
Добавить функционал раздела Публикации
Добавить логику/ограничения/валидацию
Сверстать финальный шаблон резюме
Добавить функцию генерации PDF

matiko-ma · 2025-10-15T23:17:47Z

Я не знала, что если коммитить в ту же ветку, то не получится сделать отдельный pull request :(
Но да и ладно, в коммите hw2.2 было сделано следующее:

Доработано меню в версии для телефонов согласно макету
Добавлен функционал карточке (удаление, перемещение)
Сделан общий класс CardList со всеми функциями карточек

To Be Done:

Доработать drop-down list согласно макету
Доработать функцию перемещения для планшетов и телефонов
Добавить возможность редактировать содержимое карточки
Добавить функционал раздела Публикации
Добавить логику/ограничения/валидацию
Сверстать финальный шаблон резюме
Добавить функцию генерации PDF

vsovgir · 2025-11-15T15:14:29Z

@@ -1,3 +1,5 @@
+import './menu_activeBtn.js'


Используются модули

vsovgir · 2025-11-15T15:15:43Z

+
 document.addEventListener('DOMContentLoaded', function () {

    const posEl = document.getElementById('posName');


Понятные имена переменных, например, здесь ясно, что это поле с позицией, для которой создается резюме

vsovgir · 2025-11-15T15:21:59Z

+        console.log(sessionStorage.getItem('position'));
+    });
+
+    const photoEl = document.getElementById('photo_input');


Очень классно, что делаешь предпросмотр загруженного фото

vsovgir · 2025-11-15T15:26:33Z

Код очень понятный, легко читается: каждый блок отвечает за одно поле + все данные складываются в sessionStorage

Мне кажется, что раз здесь похожие блоки, то можно оптимизировать код, написав вспомогательную функцию. Например:

const bindInputToSession = (id) => { const el = document.getElementById(id); if (!el) return; el.addEventListener('change', () => { sessionStorage.setItem(id, el.value); }); }; bindInputToSession('name'); bindInputToSession('surname'); bindInputToSession('email'); bindInputToSession('phone'); bindInputToSession('about');

vsovgir · 2025-11-15T15:29:28Z

+        const edStart = document.getElementById('ed_start').value;
+        const edEnd = document.getElementById('ed_end').value;
+
+        const formatMonthYear = (dateStr) => {


Чуть-чуть поехали отступы

vsovgir · 2025-11-15T15:31:14Z

+        const workCurrent = document.getElementById('w_cur').checked;
+        let cardSmallText = "";
+
+        const formatMonthYear = (dateStr) => {


Функция используется ещё в cv_3, поэтому можно вынести в отдельный модуль

vsovgir · 2025-11-15T15:32:37Z

+        };
+        const  cardSmallText = `${formatMonthYear(edStart)} — ${formatMonthYear(edEnd)}`;
+
+        const extraData = {


Удобный способ хранения данных

vsovgir · 2025-11-15T15:36:14Z

+        this.init();
+    }
+
+    init() {


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

vsovgir · 2025-11-15T15:36:42Z

+            }
+        });
+
+        document.addEventListener('mouseup', () => {


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

vsovgir · 2025-11-15T15:57:23Z

+        console.log(sessionStorage.getItem('surname'));
+    });
+
+    const emailEl = document.getElementById('email');


Для телефона и почты можно добавить валидацию

vsovgir · 2025-11-15T16:22:54Z

Саммари по критериям

Читаемость и понятность кода

Код читается легко: файлы логично разделены по шагам приложения (cv_1-cv_5, CardList, меню), имена переменных и функций отражают их смысл
Логика обработчиков событий линейная и понятная, даже без комментариев ясно, что где происходит

Отсутствие ошибок

Синтаксических ошибок в проекте нет, работа со всеми элементами реализована корректно

Модульность и переиспользуемость кода

Проект хорошо структурирован на модули: класс CardList переиспользуется на нескольких шагах, а логика меню вынесена в отдельный модуль
В cv_5.js видно удачное обобщение через addCardHandler, что уменьшает дублирование и делает код гибче
formatMonthYear можно вынести в отдельный модуль, чтобы ещё улучшить архитектуру

Соответствие стандартам форматирования и кодирования

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

Соответствие требованиям задания

Используется динамическое создание и изменение DOM-элементов. Карточки создаются, удаляются и переставляются динамически через JS
Данные хранятся в JavaScript-структурах (массивы, объекты). Данные карточек собираются в объекты и массивы и дополнительно сохраняются в sessionStorage
Есть обработка пользовательских событий (клики, отправка форм, ввод текста). Обрабатываются клики по кнопкам, изменения в полях формы и действия при перетаскивании карточек
Код организован в модули (ES modules). Проект разделён на 7 модулей: cv_1.js-cv_5.js, CardList.js, menu_activeBnt.js.
Приложение корректно работает и не выдаёт ошибок в консоли

lyaplyap · 2025-12-23T15:03:32Z

Ревью дз по javascript

Критерии выполненного задания:

Используется динамическое создание и изменение DOM-элементов [ok]
Данные хранятся в JavaScript-структурах (массивы, объекты) [ok]
Есть обработка пользовательских событий (клики, отправка форм, ввод текста) [ok]
Код организован в модули (ES modules) [mostly ok]
Приложение корректно работает и не выдаёт ошибок в консоли [ok]

Итого: 9/10

matiko-ma added 3 commits October 6, 2025 04:06

hw1

8bfa74c

hw2

33e2fac

hw2.2

ee1323f

matiko-ma mentioned this pull request Nov 2, 2025

HW1 #1

Open

vsovgir reviewed Nov 15, 2025

View reviewed changes


		document.addEventListener('DOMContentLoaded', function () {

		const posEl = document.getElementById('posName');

Conversation

matiko-ma commented Oct 11, 2025

Изменения

To be done

Uh oh!

matiko-ma commented Oct 15, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vsovgir commented Nov 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Саммари по критериям

Uh oh!

lyaplyap commented Dec 23, 2025

Ревью дз по javascript

Критерии выполненного задания:

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vsovgir commented Nov 15, 2025 •

edited

Loading