fix(utils): use Set for containsAnyOf character check (#20)

* fix: use Set for containsAnyOf character check

Change character search in containsAnyOf to use a Set of runes.
This matches the documentation's claim of O(1) character lookup and improves lookup speed.

* chore: bump version to 1.0.1 and update changelog

Bump library version to 1.0.1 in pubspec.yaml and document the containsAnyOf performance fix in CHANGELOG.md.

Author: dhruvanbhalara

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.0.1
+
+- **Fix**: Make character set lookup faster in `containsAnyOf`.
+  - Used a Set of runes instead of checking characters linearly.
+  - Added unit tests for empty inputs, mismatches, and Unicode characters.
+
 ## 1.0.0
 
 - Initial release of `password_engine`.

lib/src/utils/password_string_extensions.dart

@@ -29,8 +29,9 @@ extension PasswordStringX on String {
   /// Uses a [Set] for O(1) lookup per character instead of O(m) string search.
   bool containsAnyOf(String charSet) {
     if (charSet.isEmpty) return false;
-    for (var i = 0; i < length; i++) {
-      if (charSet.contains(this[i])) return true;
+    final set = charSet.runes.toSet();
+    for (final rune in runes) {
+      if (set.contains(rune)) return true;
     }
     return false;
   }

pubspec.yaml

@@ -1,6 +1,6 @@
 name: password_engine
 description: "A comprehensive and extensible password generation library for Dart and Flutter"
-version: 1.0.0
+version: 1.0.1
 homepage: https://github.com/dhruvanbhalara/password_engine
 repository: https://github.com/dhruvanbhalara/password_engine
 issue_tracker: https://github.com/dhruvanbhalara/password_engine/issues

test/utils/password_string_extensions_test.dart

@@ -45,5 +45,24 @@ void main() {
       expect(''.hasUnicode, isFalse);
       expect(('a' * 1000 + '\u03C0').hasUnicode, isTrue);
     });
+
+    group('containsAnyOf', () {
+      test('returns false on empty charset pool', () {
+        expect('abc'.containsAnyOf(''), isFalse);
+      });
+
+      test('returns false when no characters match', () {
+        expect('abc'.containsAnyOf('xyz'), isFalse);
+      });
+
+      test('returns true when character matches', () {
+        expect('abc'.containsAnyOf('cxz'), isTrue);
+      });
+
+      test('handles unicode characters correctly', () {
+        expect('abc\u{1F60A}'.containsAnyOf('\u{1F60A}'), isTrue);
+        expect('abc\u{1F60A}'.containsAnyOf('\u{1F60B}'), isFalse);
+      });
+    });
   });
 }