refactor(codec): streamline entry lookup methods

- Replaced manual entry search loops in `find_entries`, `find_entry`, and `find_entry_mut` with calls to new `find_entry` and `find_entry_mut` methods in the `Resource` struct.
- This change enhances code readability and maintainability by encapsulating entry lookup logic within the `Resource` struct.

Author: WendellXY

langcodec/src/codec.rs

@@ -154,10 +154,8 @@ impl Codec {
     pub fn find_entries(&self, key: &str) -> Vec<(&Resource, &Entry)> {
         let mut results = Vec::new();
         for resource in &self.resources {
-            for entry in &resource.entries {
-                if entry.id == key {
-                    results.push((resource, entry));
-                }
+            if let Some(entry) = resource.find_entry(key) {
+                results.push((resource, entry));
             }
         }
         results
@@ -187,10 +185,7 @@ impl Codec {
     /// }
     /// ```
     pub fn find_entry(&self, key: &str, language: &str) -> Option<&Entry> {
-        self.get_by_language(language)?
-            .entries
-            .iter()
-            .find(|entry| entry.id == key)
+        self.get_by_language(language)?.find_entry(key)
     }
 
     /// Finds a mutable entry by its key in a specific language.
@@ -219,10 +214,7 @@ impl Codec {
     /// }
     /// ```
     pub fn find_entry_mut(&mut self, key: &str, language: &str) -> Option<&mut Entry> {
-        self.get_mut_by_language(language)?
-            .entries
-            .iter_mut()
-            .find(|entry| entry.id == key)
+        self.get_mut_by_language(language)?.find_entry_mut(key)
     }
 
     /// Updates a translation for a specific key and language.

langcodec/src/formats/android_strings.rs

@@ -454,6 +454,29 @@ World
         assert_eq!(some_non_ascii.name, "some_non_ascii");
         assert_eq!(some_non_ascii.value, "你好");
         assert_eq!(some_non_ascii.translatable, None);
+
+        let resource = Resource::from(format);
+        assert_eq!(resource.entries.len(), 5);
+        let entry = &resource.entries[0];
+        assert_eq!(entry.id, "hello");
+        assert_eq!(entry.value, Translation::Singular("Hello".to_string()));
+        assert_eq!(entry.status, EntryStatus::Translated);
+        assert_eq!(entry.comment, None);
+
+        let entry = resource.find_entry("hello").unwrap();
+        assert_eq!(entry.value, Translation::Singular("Hello".to_string()));
+        assert_eq!(entry.status, EntryStatus::Translated);
+        assert_eq!(entry.comment, None);
+
+        let entry = resource.find_entry("multiple_lines").unwrap();
+        assert_eq!(entry.value, Translation::Singular("Hello\\n\\n\\nWorld\\n    ".to_string()));
+        assert_eq!(entry.status, EntryStatus::Translated);
+        assert_eq!(entry.comment, None);
+
+        let entry = resource.find_entry("some_non_ascii").unwrap();
+        assert_eq!(entry.value, Translation::Singular("你好".to_string()));
+        assert_eq!(entry.status, EntryStatus::Translated);
+        assert_eq!(entry.comment, None);
     }
 
     #[test]

langcodec/src/types.rs

@@ -50,6 +50,14 @@ impl Resource {
         self.entries.push(entry);
     }
 
+    pub fn find_entry(&self, id: &str) -> Option<&Entry> {
+        self.entries.iter().find(|e| e.id == id)
+    }
+
+    pub fn find_entry_mut(&mut self, id: &str) -> Option<&mut Entry> {
+        self.entries.iter_mut().find(|e| e.id == id)
+    }
+
     pub fn parse_language_identifier(&self) -> Option<LanguageIdentifier> {
         self.metadata.language.parse().ok()
     }