T
Size: a a a
2021 March 10
VP
ну так то можно любой код переписать
VP
тут скорее вопрос к code style
KC
зато разделено принятия рещения на основе длинны списка и его контента
VP
у меня просто боль от коллеги, который seq-и на not-empty правит, потому что "новички в кложе не поймут о чём речь" (а документацию им не завезли), хотя новичков в кложе у нас нет уже больше полугода
сколько живу, столько вижу холиваров о пустоте коллекций)
по мне - честно, да используйте
какой нибудь хаскелл (может быть, я не шарю) может и сам свернул бы все правильно, только кодер бы этого не увидел
а так появляются семантические костыли (иначе никак не назвать), которые плохо ложатся на голову
по мне - честно, да используйте
(not (empty? ..)), здесь действительно семантика появляетсякакой нибудь хаскелл (может быть, я не шарю) может и сам свернул бы все правильно, только кодер бы этого не увидел
а так появляются семантические костыли (иначе никак не назвать), которые плохо ложатся на голову
VP
сколько живу, столько вижу холиваров о пустоте коллекций)
по мне - честно, да используйте
какой нибудь хаскелл (может быть, я не шарю) может и сам свернул бы все правильно, только кодер бы этого не увидел
а так появляются семантические костыли (иначе никак не назвать), которые плохо ложатся на голову
по мне - честно, да используйте
(not (empty? ..)), здесь действительно семантика появляетсякакой нибудь хаскелл (может быть, я не шарю) может и сам свернул бы все правильно, только кодер бы этого не увидел
а так появляются семантические костыли (иначе никак не назвать), которые плохо ложатся на голову
по поводу перфоманса - не думаю, что здесь появились бы проблемы, разве только в случаях больших коллекций, состоящих из коллекций, где этот
(not (empty?)) дергался бы миллионы разKC
про перформанс вообще речи не шло вроде ...
VP
а сделать
(def not-empty? (comp boolean seq)) в пределах проекта тоже не шибко сложноIL
Для строк лучше использовать str/blank?
VP
про перформанс вообще речи не шло вроде ...
я ж о чем) если нет проблемы с перфомансом, то первоочередная задача - понимание команды.
VP
Не каждый поймет, что seq - это в конкретнх случаях проверка на пустоту.
Но каждый поймет это с
Но каждый поймет это с
(not empty?)KC
каждый поймет если прочтет документацию )
VP
ну согласитесь, что семантика в первую очередь напрашивается из названия
в каком то смысле, имя функции - часть ее документации
уж на что ф-ции, возвращающие bool, просят в кложе заканчивать знаком вопроса)
в каком то смысле, имя функции - часть ее документации
уж на что ф-ции, возвращающие bool, просят в кложе заканчивать знаком вопроса)
VP
здесь получается кейс, когда документация противоречит названию
VP
каждый поймет если прочтет документацию )
конечно, с этим не спорю)
KC
здесь получается кейс, когда документация противоречит названию
документация не противоречит, seq не возврашает boolean
VP
ну знак вопроса здесь просто для примера
противоречие в том, что функция, которая используется для возврата новой коллекции, используется для проверки на пустоту
противоречие в том, что функция, которая используется для возврата новой коллекции, используется для проверки на пустоту
KC
ну знак вопроса здесь просто для примера
противоречие в том, что функция, которая используется для возврата новой коллекции, используется для проверки на пустоту
противоречие в том, что функция, которая используется для возврата новой коллекции, используется для проверки на пустоту
но ведь seq не для возврата новой коллекции а для возврата Sequence или nil, второе как раз и используется как индикатор что входящая коллекция пустая просто потому что в кложе nil это пустая коллекция
VP
но ведь seq не для возврата новой коллекции а для возврата Sequence или nil, второе как раз и используется как индикатор что входящая коллекция пустая просто потому что в кложе nil это пустая коллекция
и получается мешанина - функция делает больше, чем требуется для проверки на пустоту
Sequence - это же коллекция тоже
Sequence - это же коллекция тоже
VP
это как использовать экскаватор, чтобы вырыть ямку для цветочка