java - Springブートでインスタンス化されるクラスのAutowiringリポジトリ

Question

今、私はリポジトリに接続しようとしているAPIクライアントタイプのクラスを持っているので、MySQLデータベースにデータを格納できます。

私が抱えている問題は、APIクライアントクラスが新しいオブジェクトをインスタンス化するため、Autowiringが正しく機能しないということです。私はこの問題の回避策を見てきましたが、いくつかのオプションがありましたが、問題にどのように適用するのか混乱しています。

GeniusApiClient.java：

@Component 
public final class GeniusApiClient { 
    private final OkHttpClient client = new OkHttpClient(); 

    @Autowired 
    private ArtistDao artistDao; 

    public static void main(String[] args) throws Exception { 
     GeniusApiClient geniusApiClient = new GeniusApiClient(); 
     String artistId = (geniusApiClient.getArtistId("Ugly Duckling")); 

     ArrayList<String> artistSongIds = geniusApiClient.getArtistSongIds(artistId); 
     System.out.println(geniusApiClient.getAllSongAnnotations(artistSongIds, artistId)); 
    } 

    public String getAllSongAnnotations(ArrayList<String> songIds, String artistId) { 
     Artist artist = new Artist("test name for now", "string123", "223"); 
     artistDao.save(artist); 
     return "finished"; 
    } 

} 

ArtistDao.java：

@Transactional 
public interface ArtistDao extends CrudRepository<Artist, Long> { 
    public Artist findByGeniusId(String geniusId); 
} 

ArtistController.java：

は参考のため、ここでは関係のファイルのいくつかの部分であり、

@Controller 
public class ArtistController { 
    @Autowired 
    private ArtistDao artistDao; 

    /** 
    * GET /create --> Create a new artist and save it in the database. 
    */ 
    @RequestMapping("/create") 
    @ResponseBody 
    public String create(String name, String annotations, String genius_id) { 
     String userId = ""; 
     try { 
      genius_id = genius_id.replaceAll("/$", ""); 
      Artist artist = new Artist(name, annotations, genius_id); 
      artistDao.save(artist); 
      userId = String.valueOf(artist.getId()); 
     } 
     catch (Exception ex) { 
      return "Error creating the artist: " + ex.toString(); 
     } 
     return "User succesfully created with id = " + userId; 
    } 

    /** 
    * GET /get-by-email --> Return the id for the user having the passed 
    * email. 
    */ 
    @RequestMapping("/get") 
    @ResponseBody 
    public String getByEmail(String genius_id) { 
     String artistId = ""; 
     try { 
      Artist artist = artistDao.findByGeniusId(genius_id); 
      artistId = String.valueOf(artist.getId()); 
     } 
     catch (Exception ex) { 
      return "User not found"; 
     } 
     return "The user id is: " + artistId; 
    } 

} 

問題は、getAllSongAnnotationsメソッドのGeniusApiClient.javaでは、artistDaoにアクセスしようとするとnullポインタ例外が発生することです。私はこのクラスのインスタンス化がオートワイヤリングを台無しにしていることを理解していますが、これを修正する最善の方法が何であるか不思議です。

新しいメソッドをインスタンス化する必要がないように、クラス内のすべてのメソッドを静的にすることを検討しましたが、これはうまくいくとは思われません。助言がありますか？

おかげ

EDIT：

は、わかりやすくするために、いくつかの無関係なコードを削除しました。

EDIT2：あなたのcatchブロックでやろうとしている何を追加しましたArtistController.java

Answer 1

オブジェクトをautowire/injectできるようにするには、そのオブジェクトはSpring Beanでなければなりません。

ここでは、ArtistDaoは、Beanではないため、オートワイヤできません。 Beanにするにはいくつかのアノテーションオプションがありますが、この場合のスーツは@Repository注釈です。 GeniusApiClientクラスで使用した@Componentの特殊バージョンです。

ので、

@Repository 
@Transactional 
public interface ArtistDao extends CrudRepository<Artist, Long> { 
    public Artist findByGeniusId(String geniusId); 
} 

は動作するはずです。

私はあなたが読むことをお勧めしたい：参考資料を読むことあなたに怖い聞こえるhttp://docs.spring.io/spring/docs/current/spring-framework-reference/html/beans.html

場合は、コア春Spring in Actionの部分を見てみることができます。

Answer 2

が私には明確ではない

、あなたはそれを修正し、任意で撮影することが目的のアクションでそれを交換する必要が例外発生。

Answer 3

最終的にGeniusApiClient.classを作成しないでください。SpringはCGLIBを使用して、プロキシを作るためにクラスを動的に拡張します。そして、CGLIBが動作するための要件は、あなたのクラスを非最終的にすることです。ここでは、この上の

より：Make Spring Service Classes Final?