Les noms de méthode «plus» et «moins» sont-ils appropriés?

Java SE 8 est livré avec un nouveau mécanisme pour les dates, l' introduction LocalDate, LocalTimeet des LocalDateTimeclasses pour représenter instants de temps. Pour manipuler ces instants, un ensemble de méthodes sont données: LocalDate.plusDays(...), LocalDate.minusDays(...)et ainsi de suite.

J'ai toujours pensé que la bonne pratique consistait à nommer les méthodes après des verbes décrivant leur objectif, car les méthodes sont, en fait, des opérations à exécuter, quelque chose qui exécutera une action. Il suffit de mentionner, si l' on considère les classes comme StringBuilder, par exemple, les noms de méthodes sont append, insert, delete...

C'est pourquoi pour moi cela ne sonne pas juste de nommer une méthode plusDaysau lieu de sumDays, minusDaysau lieu de subtractDays. C'est juste que je trouve ça très ennuyeux? Qu'est-ce que tu penses?

La seule raison pour laquelle je peux penser est que les dates sont des objets immuables, donc en appelant, plusDaysvous n'ajoutez pas de jours à l'objet d'origine mais en créez un nouveau avec de nouvelles propriétés, mais c'est très très subtil.

La seule raison pour laquelle je peux penser est que les dates sont des objets immuables, donc en appelant plusDays, vous n'ajoutez pas de jours à l'objet d'origine mais en créez un nouveau avec de nouvelles propriétés, mais cela varie très subtilement.

C'est exactement la raison. Imaginez que vous disposiez d'une sorte d'API pour manipuler des plages de dates à des fins de planification. Il peut exposer des méthodes vous permettant de faire une déclaration comme:

var workdaySchedule = initialSchedule.withoutWeekends();

Cela se lit de manière très similaire à la déclaration en anglais: "L'horaire de travail est l'horaire initial sans week-end". Cela n'implique pas de modifier l'horaire initial, cela implique que l'horaire de travail est une chose nouvelle et différente.

Imaginez maintenant qu'il s'appelle:

var workdaySchedule = initialSchedule.removeWeekends();

Ceci est déroutant. L'horaire initial est-il en train d'être modifié? Cela ressemble certainement à cela, car il semble que nous en supprimions les week-ends. Mais alors pourquoi l'attribuons-nous à une nouvelle variable? Bien que ces deux schémas de dénomination soient très similaires, celui-ci évoque beaucoup moins clairement ce qui se passe. Ce serait plus approprié si removeWeekends cela changeait le calendrier initial et renvoyait le vide - auquel cas ce withoutWeekendsserait une option déroutante.

Il s'agit essentiellement d'une distinction déclarative vs impérative. Est -ce que nous annonçons que la workdayScheduleest une chose particulière, ou sommes - nous menons une liste des impératifs instructions pour faire cette chose en particulier (comme « enlever »)? Habituellement, la dénomination impérative a plus de sens lorsque vous mutez des valeurs et déclarative a plus de sens avec des valeurs immuables, comme le montre l'exemple ci-dessus.

Dans votre cas, vous avez exactement la même chose. Si je voyais:, tomorrow.plusDaysje n'imaginerais pas que cela tomorrowest en train de muter, alors que tomorrow.addDaysje pense que c'est possible. C'est quelque peu subtil, mais pas nécessairement dans le mauvais sens. Sans avoir à y penser trop fort, cette dénomination oriente naturellement votre réflexion dans la bonne direction en termes de mutation. Pour clarifier cette distinction entre ces styles impératifs et déclaratifs: "ajouter" (et "supprimer") sont des verbes , tandis que "plus" (et "sans") sont des prépositions .

Dans .NET, la dénomination est différente bien que le résultat soit exactement le même. Au lieu de:

tomorrow = LocalDateTime.plusDays(1);

il y a:

tomorrow = DateTime.Now.AddDays(1);

Cela signifie seulement que les différences entre la compréhension de «plus» et «ajouter» sont devenues une question d'opinion personnelle. Réjouissez-vous, vous n'êtes pas seul, au moins vous pouvez choisir la langue qui vous convient le mieux :)

Il s'agit probablement d'un artefact que Java utilise .Methodpour toutes les méthodes, à la fois celles qui modifient l'objet et celles qui ne le font pas.

Imaginez un langage qui possède également une object=>methodsyntaxe, qui donnerait methodune copie de l'objet à travailler. Maintenant, dans une telle langue, startDate=>plusDays(5)est clairement sans ambiguïté. Il prend la date d'origine et crée une nouvelle date qui est 5 jours plus tard.

Sur une note indépendante, cela sumDaysn'a pas de sens ici. LocalDateest un point temporel , pas une durée . Vous pouvez additionner n'importe quel nombre de durées (et le résultat est une autre durée), et vous pouvez ajouter un point temporel et une durée (le résultat est un autre point temporel), mais vous ne pouvez pas additionner des points temporels.