Yarn: Solte o prefixo 'get' para métodos

Criado em 12 dez. 2019  ·  5Comentários  ·  Fonte: FabricMC/yarn

Procurando feedback sobre isso, já que as bibliotecas padrão parecem não estar mais usando get para seus métodos mais recentes.

discussion refactor vote

Comentários muito úteis

Acredito que o padrão do JDK seja imutável sem prefixo, enquanto mutável é prefixado - isso pode ser visto um pouco com os novos registros não tendo getters ou setters, pois seus dados são imutáveis.

Todos 5 comentários

Portanto, os votos atuais são 2 :+1: e 7 :-1:. Levará isso como resultado se não houver votos ou comentários adicionais em outras 24 horas. Discórdia espreita a mente dê uma olhada.

Eu apoio a mudança, mas ela precisa de algumas convenções escritas, pois às vezes é necessário um prefixo get .

Idealmente espelhando como o JDK aborda isso.

O jdk tem o prefixo get quando há um setter? Podemos precisar dividi-los em quando há um setter versus quando não há.

Por exemplo, métodos getter de registro não têm prefixos get , mas não têm setters e não têm potencial confusão de nomes.

Na verdade, alguns novos getters jdk têm prefixos get enquanto outros não. Um exemplo notável está em java.lang.invoke.CallSite , introduzido há relativamente pouco tempo no jdk 7, frequentemente usado pelo atributo de método bootstrap da instrução invokedynamic bytecode, onde há um MethodType type() getter e um MethodHandle getTarget() getter ( o tipo um é imutável enquanto o alvo é mutável; existe um setTarget(MethodHandle) também). Além disso, algumas outras APIs mais recentes também usam o prefixo get , como Class.getModule do java 9.

Acredito que o padrão do JDK seja imutável sem prefixo, enquanto mutável é prefixado - isso pode ser visto um pouco com os novos registros não tendo getters ou setters, pois seus dados são imutáveis.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

asiekierka picture asiekierka  ·  23Comentários

copygirl picture copygirl  ·  14Comentários

Runemoro picture Runemoro  ·  23Comentários

darkevilmac picture darkevilmac  ·  14Comentários

OroArmor picture OroArmor  ·  16Comentários