将EJB注入JAX-RS(RESTful服务)


72

我试图通过注释将无状态EJB注入到JAX-RS Web服务中。不幸的是,EJB是正义的nullNullPointerException当我尝试使用它时我得到了。

@Path("book")
public class BookResource {

    @EJB
    private BookEJB bookEJB;

    public BookResource() {
    }

    @GET
    @Produces("application/xml")
    @Path("/{bookId}")
    public Book getBookById(@PathParam("bookId") Integer id)
    {
        return bookEJB.findById(id);
    }
}

我究竟做错了什么?

以下是有关我的机器的一些信息:

  • 玻璃鱼3.1
  • Netbeans 6.9 RC 2
  • Java EE 6

你们能举个可行的例子吗?

Answers:


113

我不确定这是否行得通。所以:

选项1:使用注入提供程序SPI

实现一个提供程序,它将执行查找并注入EJB。看到:

com.sun.jersey:jersey-server:1.17的示例:

import com.sun.jersey.core.spi.component.ComponentContext;
import com.sun.jersey.core.spi.component.ComponentScope;
import com.sun.jersey.spi.inject.Injectable;
import com.sun.jersey.spi.inject.InjectableProvider;

import javax.ejb.EJB;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.ws.rs.ext.Provider;
import java.lang.reflect.Type;

/**
 * JAX-RS EJB Injection provider.
 */
@Provider
public class EJBProvider implements InjectableProvider<EJB, Type> {

    public ComponentScope getScope() {
        return ComponentScope.Singleton;
    }

    public Injectable getInjectable(ComponentContext cc, EJB ejb, Type t) {
        if (!(t instanceof Class)) return null;

        try {
            Class c = (Class)t;
            Context ic = new InitialContext();

            final Object o = ic.lookup(c.getName());

            return new Injectable<Object>() {
                public Object getValue() {
                    return o;
                }
            };
        } catch (Exception e) {
            e.printStackTrace();
            return null;
        }
    }
}

选项2:使BookResource成为EJB

@Stateless
@Path("book")
public class BookResource {

    @EJB
    private BookEJB bookEJB;

    //...
}

See:

Option 3: Use CDI

@Path("book")
@RequestScoped
public class BookResource {

    @Inject
    private BookEJB bookEJB;

    //...
}

See:


Dear @Pascal Thivent Is that making JAX-Resources as @Stateless the only way to go?
Jin Kwon

@Pascal Option 3 can also look like this: @EJB private BookEJB bookEJB and it will work (GF 3.1.2)
NBW

2
"I am not sure this is supposed to work." - Why? I would expect you could just inject an EJB without playing a bunch of games
Ryan

@Ryan: no it's not easy at all because of Glassfish 4.1 HK2 bugs. I faced the same issue today.
Viacheslav Dobromyslov

@Pascal Thivent: Are there any disadvantages of 2nd option? like.. it will increase beans in jboss?
nilesh

15

This thread is rather old, nevertheless i fought the same problem just yesterday. Here is my solution:

Just make the BookResource a managed bean through @javax.annotation.ManagedBean at class level.

For this to work you need to enable CDI with a beans.xml:

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns="http://java.sun.com/xml/ns/javaee"
       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
       xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/beans_1_0.xsd">
</beans>

This file needs to be in WEB-INF if the BookResource is part of a war file. If the BookResource is packaged with the ejbs put it into META-INF.

If you want to use @EJB you're done. If you want to inject the EJB through @Inject than a beans.xml must be put into the ejbs jar file into META-INF as well.

What you're doing: You're just telling the container that the resource should be container managed. Therefor it supports injection as well as lifecycle events. So you have your business facade without promoting it to an EJB.

You don't need to extend javax.ws.rs.core.Application for this to work. BookResource is as a root resource automatically request scoped.

Tested with Glassfish 3.1.2 and a maven project.

Happy coding.


I know this answer is extremely old but: META-INF/beans.xml doesn't have anything to do with the Java EE @ManagedBean annotation; you shouldn't need it. I too use @ManagedBean to get my resource class to permit @EJB references inside it and didn't need META-INF/beans.xml involved. It's worth noting as well that if you do this you are hosed if you want to override the @EJB annotations later in a deployment descriptor, because there isn't one for @ManagedBeans.
Laird Nelson

I had the same problem. I needed beans.xml to get it running. My environment was JBoss EAP 6.4 with jersey rest service. I ran into JBAS011859. But at the end it was running.
Christian13467

10

You shall be able to do injection in JAX-RS resource without making it EJB or CDI component. But you have to remember that your JAX-RS resource must not be singleton.

So, you setup your application with this code. This makes BookResource class per-request JAX-RS resource.

@javax.ws.rs.ApplicationPath("application")
public class InjectionApplication extends javax.ws.rs.core.Application {
  private Set<Object> singletons = new HashSet<Object>();
  private Set<Class<?>> classes = new HashSet<Class<?>>();

  public InjectionApplication() {
    // no instance is created, just class is listed
    classes.add(BookResource.class);
  }

  @Override
  public Set<Class<?>> getClasses() {
    return classes;
  }

  @Override
  public Set<Object> getSingletons() {
    return singletons;
  }
}

With this setup, you are letting JAX-RS to instantiate BookResource for you on per-request basis and also inject all the required dependencies. If you make BookResource class singleton JAX-RS resource, this is, you put in getSingletons

public Set<Object> getSingletons() {
  singletons.add(new BookResource());
  return singletons;
}

then, you created instance which is not managed by JAX-RS runtime and nobody in container cares to inject anything.


The 'singleton' thing is not relevant with @EJB problem, right?
Jin Kwon

In my case, my REST resource was singleton. So even if I inject session bean (using @EJB or @Inject), by bean instance was always null, causing NullPointerException. Your last hint regarding singleton, helped me to resolve this issue. I made sure that my REST resource is not singleton.
user613114

Old post but thank you so much @Martin! I was struggeling with this problem since yesterday. Your hint with singleton made it.
peez80

5

Unfortunately, my answer is too long for a comment, so here goes. :)

Zeck, I hope that you are aware of what exactly you are doing by promoting your bean to an EJB, as suggested by Pascal. Unfortunately, as easy as it is nowadays with Java EE to 'make a class an EJB', you should be aware of the implications of doing so. Each EJB creates overhead along with the additional functionality it provides: they are transaction aware, have their own contexts, they take part in the full EJB life cycle, etc.

What I think you should be doing for a clean and reusable approach is this: extract the access to your servers services (which hopefully are accessed through a SessionFacade :) into a BusinessDelegate. This delegate should be using some kind of JNDI lookup (probably a ServiceLocator - yes, they are still valid in Java EE!) to access your backend.

Okay, off the record: if you really, really, really need the injection because you do not want to write JNDI access manually, you could still make your delegate an EJB, although it ... well, it just feels wrong. :) That way at least it will be easy to replace it later with something else if you do decide to switch to a JNDI lookup approach...


Errr, that is what you get when you do not read the question completely. In the case of your RESTful service, arguably, the service itself is your BusinessDelegate. So there you have it: the first and last part of my statement are still totally valid, while I would probably not create a BusinessDelegate for the BusinessDelegate... :)
LeChe

I wasn't comfortable making my api endpoints EJBs neither, but you can just use @ManagedBean. See my answer, if you like
Michael Simons

Your offer of JNDI is developed in Option 1: Use the injection provider SPI example.
Viacheslav Dobromyslov

2
People often talk about EJB "overhead" as if it were a well-established fact that they are heavyweight, but most developers have never bothered to measure it. Adam Bien showed that the performance difference between EJBs and POJOs was small (< 10%) in 2008 (adam-bien.com/roller/abien/entry/is_it_worth_using_pojos), and performance has only improved since then. In short, if you're trying to optimize your application for performance by comparing POJOs to EJBs, you're looking in the wrong place.
DavidS

1
I think you're misinterpreting that 10% number, LeChe. It's not "my application is now 10% slower", it's one specific type of operation in the application that is 10% slower. Using EJBs instead of POJOs doesn't make network, DB, file IO, and rendering take 10% more time, and these operations make up the bulk of many systems. The overall performance impact of using EJBs versus POJOs on a real application is probably negligible. As for the latter part of your comment, I agree there are good reasons to not make everything an EJB: fortunately Pascal suggested two alternatives.
DavidS

3

I was trying to do the exact same thing. I'm using EJB 3.1 and have a deployed my app as an EAR with separate EJB project. As Jav_Rock pointed out, I use context lookup.

@Path("book")
public class BookResource {

  @EJB
  BookEJB bookEJB;

  public BookResource() {
    try {
        String lookupName = "java:global/my_app/my_ejb_module/BookEJB";
        bookEJB = (BookEJB) InitialContext.doLookup(lookupName);
    } catch (NamingException e) {
        e.printStackTrace();
    }
  }

  @GET
  @Produces("application/xml")
  @Path("/{bookId}")
  public Book getBookById(@PathParam("bookId") Integer id) {
    return bookEJB.findById(id);
  }
}

See the link below for very useful JNDI look up tips

JNDI look up tips


I did use this method and it did work.. However, the same code works in jboss well... So I am really not sure why this happens..
mwangi

I had to use this for an legacy app. yet in jboss 6.4 i found that only the complete jndi name worked: java:global/completeEar/completeEar-ejb-1.0-SNAPSHOT/TestEJBImpl!personal.ejb.TestEJBRemote
cabaji99

1

Arjan is right. I created another class to initialize the EJB instead of creating a bean for RS

@Singleton
@LocalBean
public class Mediator {
    @EJB
    DatabaseInterface databaseFacade;

to avoid null pointer with:

@Path("stock")
public class StockResource {
    @EJB
    DatabaseInterface databaseFacade;
...

it actually works on GF


0

I have the same problem, and I solved it calling te EJB by a context lookup (the injection was impossible, I had the same error NullPointerException).

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.