NullPointerException estranha do Guice

25 de maio de 2012 17 respostas

Rafael_Guerreiro 25 de maio de 2012

Bom dia.

Na minha aplicação, eu esgtou tendo uma NullPointerException estranha que não causa nenhum problema... :shock: Toda vez que uma requisição termina, é lançada essa exception. Porém o sistema tem funcionamento perfeito. (Essas linhas que começam com "at frontend.rsedw.interceptor." é bem no momento em que meus interceptors fazem o stack.next()...)

1) Error in custom provider, java.lang.NullPointerException
  at br.com.caelum.vraptor.ioc.guice.VRaptorAbstractModule.requestInfoBindings(VRaptorAbstractModule.java:127)
  while locating br.com.caelum.vraptor.http.MutableRequest
  while locating javax.servlet.http.HttpServletRequest
  at br.com.caelum.vraptor.ioc.guice.GuiceComponentRegistry.bindToConstructor(GuiceComponentRegistry.java:110)
  while locating br.com.caelum.vraptor.interceptor.ForwardToDefaultViewInterceptor

1 error
	at com.google.inject.internal.InjectorImpl$4.get(InjectorImpl.java:977)
	at com.google.inject.internal.InjectorImpl.getInstance(InjectorImpl.java:1003)
	at br.com.caelum.vraptor.ioc.guice.GuiceProvider$GuiceContainer.instanceFor(GuiceProvider.java:60)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:47)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.LazyInterceptorHandler.execute(LazyInterceptorHandler.java:61)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.LazyInterceptorHandler.execute(LazyInterceptorHandler.java:61)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.interceptor.ExecuteMethodInterceptor.intercept(ExecuteMethodInterceptor.java:85)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at frontend.rsedw.interceptor.SetFuncionalidadesInterceptor.intercept(SetFuncionalidadesInterceptor.java:39)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:56)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:56)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:56)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.LazyInterceptorHandler.execute(LazyInterceptorHandler.java:61)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at frontend.rsedw.interceptor.NoCacheInterceptor.intercept(NoCacheInterceptor.java:43)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.interceptor.ParametersInstantiatorInterceptor.intercept(ParametersInstantiatorInterceptor.java:89)
	at br.com.caelum.vraptor.core.LazyInterceptorHandler.execute(LazyInterceptorHandler.java:59)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.interceptor.FlashInterceptor.intercept(FlashInterceptor.java:83)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.interceptor.ExceptionHandlerInterceptor.intercept(ExceptionHandlerInterceptor.java:71)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.interceptor.InstantiateInterceptor.intercept(InstantiateInterceptor.java:48)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:56)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at frontend.rsedw.interceptor.CloseSessionInterceptor.intercept(CloseSessionInterceptor.java:30)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.interceptor.ResourceLookupInterceptor.intercept(ResourceLookupInterceptor.java:69)
	at br.com.caelum.vraptor.core.ToInstantiateInterceptorHandler.execute(ToInstantiateInterceptorHandler.java:54)
	at br.com.caelum.vraptor.core.DefaultInterceptorStack.next(DefaultInterceptorStack.java:54)
	at br.com.caelum.vraptor.core.EnhancedRequestExecution.execute(EnhancedRequestExecution.java:23)
	at br.com.caelum.vraptor.VRaptor$1.insideRequest(VRaptor.java:92)
	at br.com.caelum.vraptor.ioc.guice.GuiceProvider.provideForRequest(GuiceProvider.java:76)
	at br.com.caelum.vraptor.VRaptor.doFilter(VRaptor.java:89)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:243)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:224)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:185)
	at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:472)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:151)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:100)
	at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:929)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:118)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:405)
	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:269)
	at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:515)
	at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:302)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.NullPointerException
	at br.com.caelum.vraptor.ioc.guice.VRaptorAbstractModule$2.get(VRaptorAbstractModule.java:130)
	at br.com.caelum.vraptor.ioc.guice.VRaptorAbstractModule$2.get(VRaptorAbstractModule.java:127)
	at com.google.inject.internal.InternalFactoryToProviderAdapter.get(InternalFactoryToProviderAdapter.java:40)
	at com.google.inject.internal.FactoryProxy.get(FactoryProxy.java:54)
	at com.google.inject.internal.InjectorImpl$4$1.call(InjectorImpl.java:968)
	at com.google.inject.internal.InjectorImpl.callInContext(InjectorImpl.java:1021)
	at com.google.inject.internal.InjectorImpl$4.get(InjectorImpl.java:964)
	at br.com.caelum.vraptor.ioc.guice.RequestCustomScope$1.getAttribute(RequestCustomScope.java:58)
	at br.com.caelum.vraptor.ioc.guice.AbstractScope$ScopedProvider.get(AbstractScope.java:46)
	at com.google.inject.internal.InternalFactoryToProviderAdapter.get(InternalFactoryToProviderAdapter.java:40)
	at com.google.inject.internal.InjectorImpl$4$1.call(InjectorImpl.java:968)
	at com.google.inject.internal.InjectorImpl.callInContext(InjectorImpl.java:1014)
	at com.google.inject.internal.InjectorImpl$4.get(InjectorImpl.java:964)
	... 65 more

17 Respostas

Rafael_Guerreiro 25 de mai. de 2012

Opa, então… Agora o problema piorou um pouquinho… Esse erro só acontece em uma determinada tela, depois que eu passo nela, o sistema trava…
Nenhuma requisção termina…

Eu coloquei try{}catch(NPE e) para tentar pegar essa exception… mas ela não é devolvida… Ela não é lançada das minhas classes…

Eu debuguei, para ver se os parametros passados ao stack.next estavam nulos, mas não estavam…

Lucas_Cavalcanti 26 de mai. de 2012

isso é no getter da session provavelmente… vc faz algum session.invalidate() ?

Rafael_Guerreiro 26 de mai. de 2012

Sim eu faço, quando a pessoa clica em logout, só… Mas quando esse erro começa a disparar, não tem nenhum session.invalidate()…

Lucas_Cavalcanti 27 de mai. de 2012

dúvidas:
-vc executa alguma tarefa em threads separadas da requisição?

esse nullPointer acontece em algum forward?

Rafael_Guerreiro 27 de mai. de 2012

Não tenho nenhuma thread separada (ainda).

Não executo forwards, somente redirects e alguns ofs…

Lucas_Cavalcanti 27 de mai. de 2012

consegue isolar quando acontece o problema? é em alguma lógica específica ou é aleatoriamente?

Rafael_Guerreiro 27 de mai. de 2012

É em uma lógica específica, mas embora eu tenha encapsulado, a exception não acontece na lógica…

Lucas_Cavalcanti 27 de mai. de 2012

me descreve por favor os passos que acontecem nessa lógica pra eu ver o que pode estar acontecendo. não precisa ter a sua lógica da aplicação, mas as interações com o vraptor

Rafael_Guerreiro 27 de mai. de 2012

A lógica toda consiste em algumas chamadas AJAX (Não tem problema nenhum passar a lógica inteira)
Essa lógica é a responsável pela JSP inicial (essa não é acessada por AJAX)

@Get("/visitas/{obj}/{time}")
	public void list(SellerDeepSea obj, DateTime time) {
		if (obj == null)
			throw new IllegalArgumentException(
					"SellerDeepSea 'obj' cannot be null.");
		if (time == null)
			time = new DateTime();

		result.include("sellerDeepSea", obj);
		result.include("time", time);
	}
// Essas duas lógicas sobrecarregam a de cima...
@Get("/visitas/")
	public void listSemSellerSemTimeNaUrl(SellerDeepSea obj, DateTime time) {
		this.list(obj, time);
		result.of(this).list(obj, time);
	}

	@Get("/visitas/{obj}/")
	public void listSemTimeNaUrl(SellerDeepSea obj, DateTime time) {
		this.list(obj, time);
		result.of(this).list(obj, time);
	}

Depois que essa lógica é executada, ocorrem 3 chamadas AJAX (a anotação @Ajax é para evitar que ele leve para a jsp inicial caso o usuário esteja deslogado)
(As IllegalArgumentException são capturadas pela chamada ajax mesmo.)

@Post("/relacionamento/{id}")
	@Ajax
	public void relacionamento(Long id) {
		if (id == null || id &lt; 0L)
			throw new IllegalArgumentException(
					"Long 'id' cannot be null or negative.");
		result.include("accountGroupList", EnumSet.allOf(AccountGroup.class));
		List&lt;RelCusSelAcc&gt; list = relCusSelAccDAO.listSomentePKs(id);
		new RelCusSelAccService().setCustomerCategory(list);
		result.include("relacionamentoList", list);
	}

	@Post("/relacionamento/mktShare/{id}")
	@Ajax
	public void mktShare(Long id) {
		if (id == null || id &lt; 0L)
			throw new IllegalArgumentException(
					"Long 'id' cannot be null or negative.");
		result.include("accountGroupList", EnumSet.allOf(AccountGroup.class));
		List&lt;RelCusSelAcc&gt; list = relCusSelAccDAO.listSomentePKs(id);
		RelCusSelAccService relacionamentoService = new RelCusSelAccService();
		Map&lt;AccountGroup, Map&gt;&lt;Integer, MarketShare&gt;&gt; headersMap = relacionamentoService
				.setCustomerMarketShareAndGetHeader(list);

		result.include("r1", visitaService.getR1PeriodoTabajara());
		result.include("headersMap", headersMap);
		result.include("masterHeadersMap",
				relacionamentoService.getMasterHeaders(headersMap));
		result.include("relacionamentoList", list);
	}

	@Post("/visitas/")
	@Ajax
	public void visitas(Long id, DateTime time, WeekNavigation navigation) {
		if (id == null || id &lt; 0L)
			throw new IllegalArgumentException(
					"Long 'id' cannot be null or negative.");

		time = visitaService.getDateNavigation(time, navigation);

		result.include("accountGroupList", EnumSet.allOf(AccountGroup.class));
		result.include("relacionamentoList",
				relCusSelAccDAO.listSettedVisitsInRelacionamento(id, time));
		result.include("semanaList", visitaService.get13Weeks(time));
		result.include("time", time);
	}

Lucas_Cavalcanti 27 de mai. de 2012

já que vc quer executar o método, pq vc não troca o result.of pelo result.forwardTo?

o que o @Ajax faz?

Rafael_Guerreiro 28 de mai. de 2012

A anotação @Ajax é para evitar que ele leve para a jsp inicial caso o usuário esteja deslogado (é usado somente em um interceptor).

Eu posso usar o forwardTo mesmo… Eu vou mudar aqui…

Lucas_Cavalcanti 28 de mai. de 2012

como é esse interceptor?

Rafael_Guerreiro 28 de mai. de 2012

Quando eu uso essa anotação, significa que eu não estou usando JSON, mas sim compondo a página com o HTML “gerado” na requisição ajax.
Então, para evitar que a página inicial apareça no meio de uma div, eu retorno essa exception…

@Intercepts(after = NoCacheInterceptor.class)
public class UsuarioInterceptor implements Interceptor {

	private final Logger logger = LoggerFactory.getLogger(this.getClass());
	private final Result result;
	private final UserWeb userWeb;

	public UsuarioInterceptor(Result result, UserWeb userWeb) {
		this.result = result;
		this.userWeb = userWeb;
	}

	@Override
	public void intercept(InterceptorStack stack, ResourceMethod method,
			Object resourceInstance) throws InterceptionException {
		if (method.containsAnnotation(Ajax.class)) {
			throw new InterceptionException("AJAX call with offline user.");
		}

		this.logger.info("Offline user being redirected to login page.");
		this.result.redirectTo(MainController.class).login();
	}

	@Override
	public boolean accepts(ResourceMethod method) {
		return ((!method.containsAnnotation(Liberado.class)) && (!this.userWeb
				.getLogado()));
	}
}

Lucas_Cavalcanti 28 de mai. de 2012

aquela nullpointer acontece nos mesmos casos em que dá essa exception?

tenta fazer ao invés disso algo do tipo:

if (method.containsAnnotation(Ajax.class)) {  
   this.result.use(Results.status()).forbidden("AJAX call with offline user.");
} else { 
   this.logger.info("Offline user being redirected to login page.");  
   this.result.redirectTo(MainController.class).login(); 
}

o problema é que vc deu uma exception mas não usou o result… se vc não usa o result o VRaptor tenta mesmo assim dar o forward pra default view.

mandando o forbidden ele retorna o status 403 com a mensagem, assim tb cai no callback de erro do ajax.

Rafael_Guerreiro 28 de mai. de 2012

Eu alterei, acho realmente dessa forma melhor. Mas o erro acontecia quando ele não executava o throw (ou seja, o usuário estava logado)…

Lucas_Cavalcanti 28 de mai. de 2012

Talvez o problema seja que vc está controlando a saída por exceptions… o VRaptor espera que vc controle o fluxo da requisição usando o Result, pode ser que ele esteja perdendo o controle quando as exceptions acontecem, em alguns casos…

talvez seja esse o problema de não conseguir fechar as sessions no guice também, pq o VRaptor executa um código após a requisição, e quando dá exception ele talvez não execute os destroys

Rafael_Guerreiro 28 de mai. de 2012

Hmmmm… Muito plausível mesmo. Eu vou começar a encapsular todas as exception e depois eu verifico se continua dando problemas.

Criado 25 de maio de 2012

Ultima resposta 28 de mai. de 2012

Respostas 17

Participantes 2

17 Respostas

Topicos relacionados