BLOG-56 refactor: make use cases stateless

backend/feature/post/src/adapter/delivery/post_controller.rs

@@ -1,5 +1,3 @@
-use std::sync::Arc;
-
 use async_trait::async_trait;
 
 use crate::application::{
@@ -23,14 +21,14 @@ pub trait PostController: Send + Sync {
 }
 
 pub struct PostControllerImpl {
-    get_all_post_info_use_case: Arc<dyn GetAllPostInfoUseCase>,
-    get_full_post_use_case: Arc<dyn GetFullPostUseCase>,
+    get_all_post_info_use_case: Box<dyn GetAllPostInfoUseCase>,
+    get_full_post_use_case: Box<dyn GetFullPostUseCase>,
 }
 
 impl PostControllerImpl {
     pub fn new(
-        get_all_post_info_use_case: Arc<dyn GetAllPostInfoUseCase>,
-        get_full_post_use_case: Arc<dyn GetFullPostUseCase>,
+        get_all_post_info_use_case: Box<dyn GetAllPostInfoUseCase>,
+        get_full_post_use_case: Box<dyn GetFullPostUseCase>,
     ) -> Self {
         Self {
             get_all_post_info_use_case,

backend/feature/post/src/framework/web/post_web_routes.rs

@@ -1,17 +1,10 @@
+use std::sync::Arc;
+
 use actix_web::{HttpResponse, Responder, web};
 
 use crate::{
-    adapter::delivery::{
-        post_controller::{PostController, PostControllerImpl},
-        post_info_query_dto::PostQueryDto,
-    },
-    application::{
-        error::post_error::PostError,
-        use_case::{
-            get_all_post_info_use_case::GetAllPostInfoUseCase,
-            get_full_post_use_case::GetFullPostUseCase,
-        },
-    },
+    adapter::delivery::{post_controller::PostController, post_info_query_dto::PostQueryDto},
+    application::error::post_error::PostError,
 };
 
 pub fn configure_post_routes(cfg: &mut web::ServiceConfig) {
@@ -23,15 +16,9 @@ pub fn configure_post_routes(cfg: &mut web::ServiceConfig) {
 }
 
 async fn get_all_post_info_handler(
-    get_all_post_info_use_case: web::Data<dyn GetAllPostInfoUseCase>,
-    get_full_post_use_case: web::Data<dyn GetFullPostUseCase>,
+    post_controller: web::Data<Arc<dyn PostController>>,
     query: web::Query<PostQueryDto>,
 ) -> impl Responder {
-    let post_controller = PostControllerImpl::new(
-        get_all_post_info_use_case.into_inner(),
-        get_full_post_use_case.into_inner(),
-    );
-
     let is_published_only = query.is_published_only.unwrap_or_else(|| true);
     let result = post_controller.get_all_post_info(is_published_only).await;
 
@@ -45,15 +32,9 @@ async fn get_all_post_info_handler(
 }
 
 async fn get_full_post_handler(
-    get_all_post_info_use_case: web::Data<dyn GetAllPostInfoUseCase>,
-    get_full_post_use_case: web::Data<dyn GetFullPostUseCase>,
+    post_controller: web::Data<Arc<dyn PostController>>,
     path: web::Path<i32>,
 ) -> impl Responder {
-    let post_controller = PostControllerImpl::new(
-        get_all_post_info_use_case.into_inner(),
-        get_full_post_use_case.into_inner(),
-    );
-
     let id = path.into_inner();
     let result = post_controller.get_full_post(id).await;
 

backend/server/src/container.rs

@@ -1,18 +1,20 @@
 use std::sync::Arc;
 
 use post::{
-    adapter::gateway::post_repository_impl::PostRepositoryImpl,
+    adapter::{
+        delivery::post_controller::{PostController, PostControllerImpl},
+        gateway::post_repository_impl::PostRepositoryImpl,
+    },
     application::use_case::{
-        get_all_post_info_use_case::{GetAllPostInfoUseCase, GetAllPostInfoUseCaseImpl},
-        get_full_post_use_case::{GetFullPostUseCase, GetFullPostUseCaseImpl},
+        get_all_post_info_use_case::GetAllPostInfoUseCaseImpl,
+        get_full_post_use_case::GetFullPostUseCaseImpl,
     },
     framework::db::post_db_service_impl::PostDbServiceImpl,
 };
 use sqlx::{Pool, Postgres};
 
 pub struct Container {
-    pub get_all_post_info_use_case: Arc<dyn GetAllPostInfoUseCase>,
-    pub get_full_post_use_case: Arc<dyn GetFullPostUseCase>,
+    pub post_controller: Arc<dyn PostController>,
 }
 
 impl Container {
@@ -22,12 +24,14 @@ impl Container {
         let post_repository = Arc::new(PostRepositoryImpl::new(post_db_service.clone()));
 
         let get_all_post_info_use_case =
-            Arc::new(GetAllPostInfoUseCaseImpl::new(post_repository.clone()));
-        let get_full_post_use_case = Arc::new(GetFullPostUseCaseImpl::new(post_repository.clone()));
+            Box::new(GetAllPostInfoUseCaseImpl::new(post_repository.clone()));
+        let get_full_post_use_case = Box::new(GetFullPostUseCaseImpl::new(post_repository.clone()));
 
-        Self {
+        let post_controller = Arc::new(PostControllerImpl::new(
             get_all_post_info_use_case,
             get_full_post_use_case,
-        }
+        ));
+
+        Self { post_controller }
     }
 }

backend/server/src/main.rs

@@ -51,10 +51,10 @@ fn create_app(
         Error = Error,
     >,
 > {
-    let container = Container::new(db_pool);
+    let container = Container::new(db_pool.clone());
 
     App::new()
-        .app_data(web::Data::from(container.get_all_post_info_use_case))
-        .app_data(web::Data::from(container.get_full_post_use_case))
+        .app_data(web::Data::new(db_pool))
+        .app_data(web::Data::new(container.post_controller))
         .configure(configure_post_routes)
 }