Amadeus/guice
1
0
Fork 0
mirror of https://github.com/google/guice.git synced 2024-04-21 12:32:36 +00:00
Code Issues Releases Wiki Activity
Page: AvoidStaticState
Pages
3rdPartyModules AOP AppsThatUseGuice AssistedInject Avoid Injecting Closable Resources AvoidCallingProvideMethodsAndInjectConstructors AvoidConditionalLogicInModules AvoidStaticState BeCarefulAboutIoInProviders BestPractices BindingAnnotations BindingResolution Bindings Bootstrap BoundFields BuiltInBindings ClassLoading CustomInjections CustomScopes CyclicDependencies DocumentPublicBindings DontReuseAnnotations Errors ExtendingGuice ExtensionSPI ExternalDocumentation FrequentlyAskedQuestions GettingStarted GoogleAppEngine Grapher Guice10 Guice20 Guice30 Guice40 Guice41 Guice42 Guice421 Guice422 Guice423 Guice500 Guice501 Guice510 Guice600 Guice700 GuiceDiscussions GuiceInKotlin GuicePersist GuicePersistMultiModules Home InjectOnlyDirectDependencies InjectingProviders InjectingTheInjector InjectionPoints Injections InspectingModules InstanceBindings JPA JSR330 JustInTimeBindings KeepConstructorsHidden LinkedBindings MentalModel MinimizeMutability ModulesShouldBeFastAndSideEffectFree Motivation Multibindings OSGi OptionalAOP OrganizeModulesByFeature PreferAtProvides ProviderBindings ProvidesMethods RestrictedBindingSource Scopes ServletExtensionSPI ServletModule ServletRegexKeyMapping Servlets SpringComparison Struts2Integration ThrowingProviders ToConstructorBindings Transactions UntargettedBindings UseNullable
Clone
3
AvoidStaticState
Copybara-Service edited this page 2020-08-06 10:09:21 -07:00
Table of Contents

Avoid static state

Static state and testability are enemies. Your tests should be fast and free of side-effects. But non-constant values held by static fields are a pain to manage. It's tricky to reliably tear down static singletons that are mocked by tests, and this interferes with other tests.

requestStaticInjection() is a crutch. Guice includes this API to ease migration from a statically-configured application to a dependency-injected one. New applications developed with Guice should not use this API.

Although static state is bad, there's nothing wrong with the static keyword. Static classes are okay (preferred even!) and for pure functions (sorting, math, etc.), static is just fine.