feat: use lexicographic string comparisons

Author: esentis

lib/src/nullable_string_extensions.dart

@@ -82,10 +82,10 @@ extension MiscExtensionsNullable on String? {
     if (this.isBlank) {
       return '';
     }
-    if (s.isBlank) {
+    if (s == null || s.isBlank) {
       return this!;
     }
-    return this!.replaceAll(s!, '');
+    return this!.replaceAll(s, '');
   }
 
   /// Returns the average read time duration of the given `String` in seconds.
@@ -1894,14 +1894,20 @@ extension MiscExtensionsNullable on String? {
   /// String s = "esentis".wrap("AA", after: "BB"); // returns "AAesentisBB";
   /// ```
   String wrap(String? before, {String? after}) {
-    before = before.ifBlank("");
-    if (after.isBlank) {
-      if (before.isCloseWrapChar()) {
-        before = before.getOppositeChar();
+    var resolvedBefore = before?.ifBlank("") ?? "";
+    var resolvedAfter = after?.ifBlank("");
+
+    if (resolvedAfter == null) {
+      if (resolvedBefore.isEmpty) {
+        return "${this ?? ''}";
+      }
+      if (resolvedBefore.isCloseWrapChar()) {
+        resolvedBefore = resolvedBefore.getOppositeChar() ?? resolvedBefore;
       }
-      after = before.getOppositeChar();
+      resolvedAfter = resolvedBefore.getOppositeChar();
     }
-    return "$before${this as String}${after.ifBlank(before)}";
+
+    return "$resolvedBefore${this ?? ''}${resolvedAfter}";
   }
 
   /// Returns the opposite wrap char of the `String` if possible, otherwise returns the same `String`.

lib/src/string_extensions.dart

@@ -81,10 +81,10 @@ extension MiscExtensionsNonNullable on String {
     if (this.isBlank) {
       return '';
     }
-    if (s?.isBlank == true) {
+    if (s == null || s.isBlank) {
       return this;
     }
-    return this.replaceAll(s!, '');
+    return this.replaceAll(s, '');
   }
 
   /// Returns the average read time duration of the given `String` in seconds.
@@ -1804,14 +1804,20 @@ extension MiscExtensionsNonNullable on String {
   /// String s = "esentis".wrap("AA", after: "BB"); // returns "AAesentisBB";
   /// ```
   String wrap(String? before, {String? after}) {
-    before = before?.ifBlank("");
-    if (after?.isBlank == true) {
-      if (before?.isCloseWrapChar() == true) {
-        before = before?.getOppositeChar();
+    var resolvedBefore = before?.ifBlank("") ?? "";
+    var resolvedAfter = after?.ifBlank("");
+
+    if (resolvedAfter == null) {
+      if (resolvedBefore.isEmpty) {
+        return "$this";
+      }
+      if (resolvedBefore.isCloseWrapChar()) {
+        resolvedBefore = resolvedBefore.getOppositeChar();
       }
-      after = before?.getOppositeChar();
+      resolvedAfter = resolvedBefore.getOppositeChar();
     }
-    return "$before${this}${after?.ifBlank(before ?? "") ?? before}";
+
+    return "$resolvedBefore${this}${resolvedAfter}";
   }
 
   /// Returns the opposite wrap char of the `String` if possible, otherwise returns the same `String`.

test/string_extensions_test.dart

@@ -701,13 +701,13 @@ void main() {
       String? string1 = 'employee';
       expect(string1.isUrl, false);
 
-      String? string2 = '1111111.com';
+      String? string2 = 'https://1111111.com';
       expect(string2.isUrl, true);
 
-      String? string3 = 'http://8recommmmmmendation';
+      String? string3 = 'http://';
       expect(string3.isUrl, false);
 
-      String? string4 = 'www.google.com';
+      String? string4 = 'https://www.google.com';
       expect(string4.isUrl, true);
 
       String? string5 = 'ftp://www.google.com';