Я чувствую, что многие люди попадают в «волну лучших практик», где всегда лучше следовать тому, что говорят элитные программисты.Вот мое мнение:
Если оно работает и оно довольно элегантно, используйте его.
Пока ваше решение не является "хакерским" и будет масштабироваться в зависимости от объема использования, которое видит ваше приложениеувеличивается, что бы вы ни придумали, все в порядке.
С точки зрения того, что будет наиболее приемлемым, у вас есть два принципиально разных представления: сетка и список.Если вы предполагаете, что вам понадобятся два отдельных фрагмента HTML для каждого, если вы не дублируете код вверх и вниз, то это нормально.Если вы видите, что дублируете много кода, создаете приватную функцию в PersonDisplay
и централизуете там логику.
Честно говоря, кажется, что этот фрагмент кода не будет сильно растискажем, тысяча строк кода (макс.)Используйте здравый смысл и проверяйте реальность время от времени.Если вы видите, что создаете слишком много функций, объедините некоторые функции.Если вы пишете повторяющийся код, переместите этот код в отдельную функцию.Вы действительно не можете испортить это слишком много.
Как говорится, "не парься по мелочам".Вы можете позволить себе время от времени иметь немного несовершенный код.