From fad8b55d81713f5d8a88e16ac546e9cf97b3465b Mon Sep 17 00:00:00 2001
From: rito528 <39003544+rito528@users.noreply.github.com>
Date: Wed, 25 Sep 2024 23:32:45 +0900
Subject: [PATCH] =?UTF-8?q?refactor:=20=E6=A4=9C=E7=B4=A2=E3=81=99?=
 =?UTF-8?q?=E3=82=8B=E3=81=A8=E3=81=8D=E3=81=AB=E3=82=AF=E3=82=A8=E3=83=AA?=
 =?UTF-8?q?=E3=81=AE=E6=89=80=E6=9C=89=E6=A8=A9=E3=82=92=E8=A6=81=E6=B1=82?=
 =?UTF-8?q?=E3=81=97=E3=81=AA=E3=81=84=E3=82=88=E3=81=86=E3=81=AB?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/repository/search_repository.rs       | 12 +++++-----
 .../infra/resource/src/database/components.rs | 12 +++++-----
 server/infra/resource/src/database/search.rs  | 24 +++++++++----------
 .../src/repository/search_repository_impl.rs  | 12 +++++-----
 server/usecase/src/search.rs                  | 12 +++++-----
 5 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/server/domain/src/repository/search_repository.rs b/server/domain/src/repository/search_repository.rs
index fc7f6813..79e06ae9 100644
--- a/server/domain/src/repository/search_repository.rs
+++ b/server/domain/src/repository/search_repository.rs
@@ -10,10 +10,10 @@ use crate::{
 #[automock]
 #[async_trait]
 pub trait SearchRepository: Send + Sync + 'static {
-    async fn search_users(&self, query: String) -> Result<Vec<User>, Error>;
-    async fn search_forms(&self, query: String) -> Result<Vec<Form>, Error>;
-    async fn search_labels_for_forms(&self, query: String) -> Result<Vec<Label>, Error>;
-    async fn search_labels_for_answers(&self, query: String) -> Result<Vec<Label>, Error>;
-    async fn search_answers(&self, query: String) -> Result<Vec<Answer>, Error>;
-    async fn search_comments(&self, query: String) -> Result<Vec<Comment>, Error>;
+    async fn search_users(&self, query: &str) -> Result<Vec<User>, Error>;
+    async fn search_forms(&self, query: &str) -> Result<Vec<Form>, Error>;
+    async fn search_labels_for_forms(&self, query: &str) -> Result<Vec<Label>, Error>;
+    async fn search_labels_for_answers(&self, query: &str) -> Result<Vec<Label>, Error>;
+    async fn search_answers(&self, query: &str) -> Result<Vec<Answer>, Error>;
+    async fn search_comments(&self, query: &str) -> Result<Vec<Comment>, Error>;
 }
diff --git a/server/infra/resource/src/database/components.rs b/server/infra/resource/src/database/components.rs
index 2ee76d37..7ac6b392 100644
--- a/server/infra/resource/src/database/components.rs
+++ b/server/infra/resource/src/database/components.rs
@@ -120,10 +120,10 @@ pub trait UserDatabase: Send + Sync {
 #[automock]
 #[async_trait]
 pub trait SearchDatabase: Send + Sync {
-    async fn search_users(&self, query: String) -> Result<Vec<User>, InfraError>;
-    async fn search_forms(&self, query: String) -> Result<Vec<Form>, InfraError>;
-    async fn search_labels_for_forms(&self, query: String) -> Result<Vec<Label>, InfraError>;
-    async fn search_labels_for_answers(&self, query: String) -> Result<Vec<Label>, InfraError>;
-    async fn search_answers(&self, query: String) -> Result<Vec<Answer>, InfraError>;
-    async fn search_comments(&self, query: String) -> Result<Vec<Comment>, InfraError>;
+    async fn search_users(&self, query: &str) -> Result<Vec<User>, InfraError>;
+    async fn search_forms(&self, query: &str) -> Result<Vec<Form>, InfraError>;
+    async fn search_labels_for_forms(&self, query: &str) -> Result<Vec<Label>, InfraError>;
+    async fn search_labels_for_answers(&self, query: &str) -> Result<Vec<Label>, InfraError>;
+    async fn search_answers(&self, query: &str) -> Result<Vec<Answer>, InfraError>;
+    async fn search_comments(&self, query: &str) -> Result<Vec<Comment>, InfraError>;
 }
diff --git a/server/infra/resource/src/database/search.rs b/server/infra/resource/src/database/search.rs
index 732d9ff2..d8ba1e61 100644
--- a/server/infra/resource/src/database/search.rs
+++ b/server/infra/resource/src/database/search.rs
@@ -11,12 +11,12 @@ use crate::database::{components::SearchDatabase, connection::ConnectionPool};
 
 #[async_trait]
 impl SearchDatabase for ConnectionPool {
-    async fn search_users(&self, query: String) -> Result<Vec<User>, InfraError> {
+    async fn search_users(&self, query: &str) -> Result<Vec<User>, InfraError> {
         Ok(self
             .meilisearch_client
             .index("users")
             .search()
-            .with_query(&query)
+            .with_query(query)
             .with_attributes_to_highlight(Selectors::All)
             .execute::<User>()
             .await?
@@ -26,12 +26,12 @@ impl SearchDatabase for ConnectionPool {
             .collect_vec())
     }
 
-    async fn search_forms(&self, query: String) -> Result<Vec<Form>, InfraError> {
+    async fn search_forms(&self, query: &str) -> Result<Vec<Form>, InfraError> {
         Ok(self
             .meilisearch_client
             .index("form_meta_data")
             .search()
-            .with_query(query.as_str())
+            .with_query(query)
             .with_attributes_to_highlight(Selectors::All)
             .execute::<Form>()
             .await?
@@ -41,12 +41,12 @@ impl SearchDatabase for ConnectionPool {
             .collect_vec())
     }
 
-    async fn search_labels_for_forms(&self, query: String) -> Result<Vec<Label>, InfraError> {
+    async fn search_labels_for_forms(&self, query: &str) -> Result<Vec<Label>, InfraError> {
         Ok(self
             .meilisearch_client
             .index("label_for_forms")
             .search()
-            .with_query(query.as_str())
+            .with_query(query)
             .with_attributes_to_highlight(Selectors::All)
             .execute::<Label>()
             .await?
@@ -56,12 +56,12 @@ impl SearchDatabase for ConnectionPool {
             .collect_vec())
     }
 
-    async fn search_labels_for_answers(&self, query: String) -> Result<Vec<Label>, InfraError> {
+    async fn search_labels_for_answers(&self, query: &str) -> Result<Vec<Label>, InfraError> {
         Ok(self
             .meilisearch_client
             .index("label_for_form_answers")
             .search()
-            .with_query(query.as_str())
+            .with_query(query)
             .with_attributes_to_highlight(Selectors::All)
             .execute::<Label>()
             .await?
@@ -71,12 +71,12 @@ impl SearchDatabase for ConnectionPool {
             .collect_vec())
     }
 
-    async fn search_answers(&self, query: String) -> Result<Vec<Answer>, InfraError> {
+    async fn search_answers(&self, query: &str) -> Result<Vec<Answer>, InfraError> {
         Ok(self
             .meilisearch_client
             .index("real_answers")
             .search()
-            .with_query(query.as_str())
+            .with_query(query)
             .with_attributes_to_highlight(Selectors::All)
             .execute::<Answer>()
             .await?
@@ -86,12 +86,12 @@ impl SearchDatabase for ConnectionPool {
             .collect_vec())
     }
 
-    async fn search_comments(&self, query: String) -> Result<Vec<Comment>, InfraError> {
+    async fn search_comments(&self, query: &str) -> Result<Vec<Comment>, InfraError> {
         Ok(self
             .meilisearch_client
             .index("form_answer_comments")
             .search()
-            .with_query(query.as_str())
+            .with_query(query)
             .with_attributes_to_highlight(Selectors::All)
             .execute::<Comment>()
             .await?
diff --git a/server/infra/resource/src/repository/search_repository_impl.rs b/server/infra/resource/src/repository/search_repository_impl.rs
index a6ea5322..a184d9c8 100644
--- a/server/infra/resource/src/repository/search_repository_impl.rs
+++ b/server/infra/resource/src/repository/search_repository_impl.rs
@@ -13,7 +13,7 @@ use crate::{
 
 #[async_trait]
 impl<Client: DatabaseComponents + 'static> SearchRepository for Repository<Client> {
-    async fn search_users(&self, query: String) -> Result<Vec<User>, Error> {
+    async fn search_users(&self, query: &str) -> Result<Vec<User>, Error> {
         self.client
             .search()
             .search_users(query)
@@ -21,7 +21,7 @@ impl<Client: DatabaseComponents + 'static> SearchRepository for Repository<Clien
             .map_err(Into::into)
     }
 
-    async fn search_forms(&self, query: String) -> Result<Vec<Form>, Error> {
+    async fn search_forms(&self, query: &str) -> Result<Vec<Form>, Error> {
         self.client
             .search()
             .search_forms(query)
@@ -29,7 +29,7 @@ impl<Client: DatabaseComponents + 'static> SearchRepository for Repository<Clien
             .map_err(Into::into)
     }
 
-    async fn search_labels_for_forms(&self, query: String) -> Result<Vec<Label>, Error> {
+    async fn search_labels_for_forms(&self, query: &str) -> Result<Vec<Label>, Error> {
         self.client
             .search()
             .search_labels_for_forms(query)
@@ -37,7 +37,7 @@ impl<Client: DatabaseComponents + 'static> SearchRepository for Repository<Clien
             .map_err(Into::into)
     }
 
-    async fn search_labels_for_answers(&self, query: String) -> Result<Vec<Label>, Error> {
+    async fn search_labels_for_answers(&self, query: &str) -> Result<Vec<Label>, Error> {
         self.client
             .search()
             .search_labels_for_answers(query)
@@ -45,7 +45,7 @@ impl<Client: DatabaseComponents + 'static> SearchRepository for Repository<Clien
             .map_err(Into::into)
     }
 
-    async fn search_answers(&self, query: String) -> Result<Vec<Answer>, Error> {
+    async fn search_answers(&self, query: &str) -> Result<Vec<Answer>, Error> {
         self.client
             .search()
             .search_answers(query)
@@ -53,7 +53,7 @@ impl<Client: DatabaseComponents + 'static> SearchRepository for Repository<Clien
             .map_err(Into::into)
     }
 
-    async fn search_comments(&self, query: String) -> Result<Vec<Comment>, Error> {
+    async fn search_comments(&self, query: &str) -> Result<Vec<Comment>, Error> {
         self.client
             .search()
             .search_comments(query)
diff --git a/server/usecase/src/search.rs b/server/usecase/src/search.rs
index ade441a6..ccb7a5c8 100644
--- a/server/usecase/src/search.rs
+++ b/server/usecase/src/search.rs
@@ -9,12 +9,12 @@ pub struct SearchUseCase<'a, SearchRepo: SearchRepository> {
 impl<R: SearchRepository> SearchUseCase<'_, R> {
     pub async fn cross_search(&self, query: String) -> Result<CrossSearchResult, Error> {
         let (forms, users, label_for_forms, label_for_answers, answers, comments) = try_join!(
-            self.repository.search_forms(query.to_owned()),
-            self.repository.search_users(query.to_owned()),
-            self.repository.search_labels_for_forms(query.to_owned()),
-            self.repository.search_labels_for_answers(query.to_owned()),
-            self.repository.search_answers(query.to_owned()),
-            self.repository.search_comments(query.to_owned())
+            self.repository.search_forms(&query),
+            self.repository.search_users(&query),
+            self.repository.search_labels_for_forms(&query),
+            self.repository.search_labels_for_answers(&query),
+            self.repository.search_answers(&query),
+            self.repository.search_comments(&query)
         )?;
 
         Ok(CrossSearchResult {