use reified loader #29
Conversation
| @@ -10,5 +10,5 @@ import org.springframework.stereotype.Component | |||
| class NamespaceResolver : GraphQLResolver<Namespace> { | |||
|
|
|||
| fun affiliation(namespace: Namespace, dfe: DataFetchingEnvironment) = | |||
| dfe.loader(Affiliation::class).load(namespace.affiliationId) | |||
| dfe.loader<Affiliation>().load(namespace.affiliationId) | |||
There was a problem hiding this comment.
Dette må være AffiliationDataLoader, hvis vi ikke setter "${simpleName}DataLoader" i loader() funksjonen.
| @@ -23,7 +23,7 @@ class ApplicationDeploymentResolver( | |||
| Namespace(applicationDeployment.namespaceId, applicationDeployment.affiliationId) | |||
|
|
|||
| fun details(applicationDeployment: ApplicationDeployment, dfe: DataFetchingEnvironment) = | |||
| dfe.loader(ApplicationDeploymentDetails::class).load(applicationDeployment.id) | |||
| dfe.loader<ApplicationDeploymentDetails>().load(applicationDeployment.id) | |||
There was a problem hiding this comment.
Jeg må innrømme at jeg egentlig ikke syntes dette er bedre. Det er ikke alltid det passer å bruke reification og generics for å overføre typeinformasjon som parameter. Etter min vurdering leser
dfe.loaderFor(ApplicationDeploymentDetails::class)
bedre enn
dfe.loader<ApplicationDeploymentDetails>() eller evt. dfe.loaderFor<ApplicationDeploymentDetails>()
Å bruke generics her er bare en konvoluted måte å sende inn en parameter på. En måte å vurdere om generics passer eller ei på er å sjekke om metodesignaturen passer ved å gå over til flertall. F.eks.
val loaders = dfe.loadersFor(ApplicationDeploymentDetails::class, DeploymentSpec::class)
En slik endring vil ikke kunne støttes med bruk av generics.
No description provided.