TGViewer
Open Source: PHP Open Source: PHP @open_source_php · 422 subscribers
Post #34 922

Forwarded from Alexander Vorobyev

Нужен совет/рекомендации/доводы/мысли.

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

Тем не менее scrutinizer-ci . Присвоил классу значек "F" и отображает его красным. С пояснением
Total Complexity 60 , Total Lines 827 .

Как подходить в таких случаях? Действительно ли стоит следовать его рекомендациям в таких случаях?

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

Речь идет вот об этом классе https://github.com/Voral/vs-version-incrementor/blob/master/src/Config.php

#semver #changelog
GitHub vs-version-incrementor/src/Config.php at master · Voral/vs-version-incrementor A tool for semantic versioning and changelog generation in Composer projects based on Git commits. - Voral/vs-version-incrementor
More from @open_source_php
  1. Sep 17, 2026Простая задача, чтобы вкатиться в ряды контрибьюторов Testo https://github.com/php-testo/t…
  2. Nov 25, 2025True Async PHP Edmond Dantes призывает всех к обсуждению особенностей реализации нашумевше…
  3. Nov 11, 2025Сегодня состоялся релиз open-source админки MoonShine v4! Вкратце по нововведениям рассказ…
  4. Oct 21, 2025🤎 Драйвер для NATS от Thesis! Каждый, кто пишет на Golang, наверняка слышал про NATS. Это…
  5. Oct 13, 2025Есть задачка посложнее. Нужно добавить поддержку игнорирования мутаторов: - infection-igno…
  6. Oct 13, 2025📣 Infection – библиотека для мутационных тестов Infection довольно круто инструменты для…
Threads Profile ViewerView any public Threads profile without an account.Open ThreadLook →Writing with AI? Make it sound human.Metric37 rewrites AI drafts so they read naturally. Free AI detector, 1,500 words free.Try Metric37 →