final test class есть ли в этом необходимость?

Dima_Mayboroda · 18.Ноябрь.2020 10:33:34

Всем привет.
В общем и целом, такой вопрос. Насколько целесообразно помечать тест класс как final? Влияет ли это каким-то образом на производительность или компиляцию. Может есть какие статьи на эту тему? В гугле похожего ничего не нашел(
Спасибо.

Mihail_Bratuhin · 18.Ноябрь.2020 16:56:48

Скорее не влияет, чем влияет. Лучше не забивать этим голову. Оно не для перфоманса было сделано. Обычно узкие места в совсем иных местах. Вот тут пишут, что делали тест и влияния на производительность не зафиксировано:

Но тесты на производительность очень тонкая материя. Доверять этим выводам или нет - решать вам. К тому же оптимизации могут отличатся в разных реализациях java-машины.

ArtOfLife · 19.Ноябрь.2020 08:11:23

Типичное использование модификатора final на классе хоть как-то обосновано лишь в случае утилитных классов (аля хэлперы). Ибо по факту, таким образом мы запрещаем наследование, которое, к слову, бессмысленно при наличии одних лишь статических методов и приватного конструктора. Т.е. это скорее подсказка людям, использующим этот класс, что он - stateless и это сделано by design.

В случае с тестами, чисто в теории, наследование может иметь смысл при частном случае расширения какой-то области тестирования. Но это скорее логическое разделение, и то же самое можно обыграть на уровне пакетов.

В общем, использовать final на тестовых классах можно, но практического смысла в этом мало. И это скорее вызовет уйму вопросов у ревьюверов: мол, зачем?

Viktor_Yakushenko · 19.Ноябрь.2020 17:24:48

Eсли это еще и ui тесты то оптипизациями на уровне синтаксиса вобще забейте. Вон в Селениуме в основной либе три обращения к апи драйвера при проверке тега элемента, в SelectElement классе, вот про такие оптимизации нужно думать)

github.com

SeleniumHQ/selenium/blob/trunk/dotnet/src/support/UI/SelectElement.cs

// <copyright file="SelectElement.cs" company="WebDriver Committers">
// Licensed to the Software Freedom Conservancy (SFC) under one
// or more contributor license agreements. See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership. The SFC licenses this file
// to you under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//     http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
// </copyright>

using System;
using System.Collections.Generic;

This file has been truncated. show original

Viktor_Yakushenko · 25.Ноябрь.2020 19:06:42

уже пофиксили)

kochetkov-ma · 30.Ноябрь.2020 16:37:47

На скорость компиляции и производительность не повлияет. Если у вас будет миллион final классов и 10 миллионов инстансов этих классов скорость компиляции, загрузки классов, работы с отражениями и объектами по сравнения с открытыми классами не будет.

final повлияет на архитектуру приложения и безопасность.
На основании советов из Эффективное програмирование на Java нужно максимально ограничивать ваш класс, то есть использовать final и открывать для наследования, только если вы уверены в безопасности наследования. Хотя для тестовых классов, которые будет расширять максимум один человек абсолютно не важно final он или нет.