Глава 1 | Помощь - Искуственный интелект
1.1: Readme
1. Идеальный текст, нет скачков, четкие разграничения ответственности, хорошо подобранные слова, хотя версия только 1.0.0.
2. Смайлики, неккоторые не соответствуют логике, переставлены или же, просто запутаны, типичная ошибка UI-AI
3. Ответственность, архитектура, лицензирование. Во первых, ты сам подчеркнул что -
данный плагин только вышел в релиз.
то есть сертификации у него быть не может, только самопровозглашение. 2. Архитектура, а вот тут мое любимое)) Я - architecture type fullstack developer, server engineer, system administrator, уровень не назову, просто такой же дев как ты, просто знаю чуть чуть больше, просто в другом слою мышления. Так вот распределяют архитектуру либо опытные разработчики, либо
обученный на опытных репозиториях)
1.2: Coder enthusiasm and contradiction
Идея - прекрасная, но вот посмотри на свое яркое readme
А теперь посмотри на свое резюме:
Не видишь противоречий и несостыковок дизайна? Опытный дизайнер придерживается одного стиля везде, и если он уже начал заниматься косметикой, он будет это делать и в коде, и в тексте, и в readme, посмотри сюда,
Это моя карта будующего плагина, это только рабочая личная карта для таких же дэвов
1.1 Map - Рабочая карта
1.2 Map - Архитектурная карта
P.S: А вот это моя гордость, мои сложные архитектурные карты, которые понимают только такие же архитекторы
Теперь посмотри на мой код:
1.3 Code - Код инициализации системы без Bootstrap
Видишь общую стандартизацию? Все лежит красиво и дополняет друг друга, все сливается в общую архитектурную картну, и она уже работает на тебя и без твоего участия, это то - что не нужно чинить в будующем, это то что нужно дополнять выедь есть понимание что система надежна, и выполнит все твои требования.
Глава 2 | Оветственность - Искуственный интелект
2.1 - Архитектура
Это для меня больная тема. Нет обработчика, команды - бизнес логика, нет защиты инвариантости, и нет архитектуры от слова совсем. Между данными может проскочить conditional race, потому что ты не знаешь как они вызовутся, потому что ты не держишь архитектуру и не понимаешь ее правил. Ты играешь как научили или "показали" но не как безопасно. За 6 лет опыта если они реальны, ты уже подпалил системы которые делал.
Где архитектура? Здесь только смешанные методы, тут нарушены в таком проекте основные правила SOLID, нарушена структура OOP, архитектура классов слишком превосходит планку пред коммит, почему диагностика и рекомендации не выведены в отдельный folder, почему логика и ответственность с ownership размазана? Ты потерял контроль буквально над всем.
2.2 - Код
Есть понятие синтаксиса, он - на высшем уровне как и API, но вот проблема, с тем что я видел ранее - это не имеет под собой основы понимаешь? Твой плагин, это мир с отдельными правилами и установками, отпустишь один конец как ответственность и он со свистом полетит вниз. Ownership размазана, масштабируемости нет, плагин опять же зажат своей же релизацией, и платить по счетам когда придет время будет очень больно и трудно..
Давай разберем вместе со мной вот этот кусок полностью
✔️Передача через конструктор прекрасное решение компизиции а не наследования.
❗Проверка runtime флага вручную, это уже не очень, забудется, не снимется = плагин сломается.
❌ Использование Scheduler для снятия состояния, вот тут и нужно понимание runtime tick
❌ У операции нет владельца состояния.
❌ Могут побежать race между данными.
✔️ DiskSnapshot прекрасное решение, как snapshot, просто идеальное.
DiskSnapshot snapshot = new DiskSnapshot(
❌ Ленивый интерфейс Excpeption, ИИ путает lazy code с понятием ленивых интерфейсов, запомни на тебя - ИИ не хочет тратить свои ресурсы, и экономит при неправильных вопросах, как может крутиться И ОБОБЩАЕТ то же самое было в твоем резюме.
} catch (Exception e) {
plugin.getLogger().warning("Ошибка при проверке диска: " + e.getMessage());
}
❌ Ручное управление не приветствуется
public void start() {
running = true;
}
public void stop() {
running = false;
}
Дальше но изволь не стану, представь это только 1 ПОВЕРХНОСТНО разобранный метод, а теперь представь, как думаешь, что у тебя в системе? И так - везде, потому что ИИ это инструмент а не исполнитель, у него нет ни ответственности ни гарантий, вот зачем нужен Architecture God/Senior Lead.
2.3 GitHub
Что это такое?
Глава 3 | Ответственность за проект и общая оценка. Ответственности нет от слова совсем, слова чужие, эмоции свои, тут скорее всего хотелось сделать нормально и побыстрее, так что за идею по противоречию spark 10/10, реализация 5/10, но это не твоя ошибка, ты просто использовал не то что нужно здесь и ты бы знал как это тупо смотриться здесь, так что лучше учись, да сложно, да не очень то и весело, но если хочешь - сделаешь.
❗Опять же, проект ну просто имба, у тебя шикарные идеи и есть толк, только вот исполнение из-за ИИ хромает, но что поделать нынешняя мета такая, что таким старикам и остается смотреть на то как все уходит в новое пространство.
