2016-08-25 31 views
3

我有一个字符串,简化为"12345"它是排序的。字符串couild包含数字(0-9)或字母(a-z)。如果混合使用自然排序顺序。我需要一种方法来验证这是否属实。检查一个字符串是否排序

尝试使用LINQ技术:

string items1 = "2349"; //sorted 
string items2 = "2476"; //not sorted, 6<>7 

bool sorted1 = Enumerable.SequenceEqual(items1.OrderBy(x => x), items1); //true 
bool sorted2 = Enumerable.SequenceEqual(items2.OrderBy(x => x), items2); //false 

但可能还按降序排序。

是否有更好的方法,然后

string items3 = "4321"; 
bool sorted3 = Enumerable.SequenceEqual(items3.OrderBy(x => x), items3) || Enumerable.SequenceEqual(items3.OrderByDescending(x => x), items3); 

检查一个字符串排序?也许有些内置解决方案?

+0

http://stackoverflow.com/questions/6396378/c-sharp-linq-orderby-numbers-that-are-string-and-you-cannot-convert-them-to-int – Oluwafemi

+0

如果字符串包含,该怎么办你的信?您刚刚向我们展示了最佳案例场景。如果字符不是英文字母(例如'-'或'?')? – user3185569

+0

@ c0rd因此'123abc'和'abc123'和'ab12cd34'被认为是分类的? – user3185569

回答

2

我曾经检查过类似于您的案例,但数据流量很大,因此性能很重要。我想出了这个小的扩展类,它有很好的表现:

public static bool IsOrdered<T>(this IEnumerable<T> enumerable) where T: IComparable<T> 
{ 
    using (var enumerator = enumerable.GetEnumerator()) 
    { 
     if (!enumerator.MoveNext()) 
      return true; //empty enumeration is ordered 

     var left = enumerator.Current; 
     int previousUnequalComparison = 0; 

     while (enumerator.MoveNext()) 
     { 
      var right = enumerator.Current; 
      var currentComparison = left.CompareTo(right); 

      if (currentComparison != 0) 
      { 
       if (previousUnequalComparison != 0 
        && currentComparison != previousUnequalComparison) 
        return false; 

       previousUnequalComparison = currentComparison; 
       left = right; 
      } 
     } 
    } 

    return true; 
} 

使用它,显然是非常简单的:

var items1 = "2349"; 
var items2 = "2476"; //not sorted, 6<>7 
items1.IsOrdered(); //true 
items2.IsOrdered(); //false 
+1

这确实是正确的方法。比任何“简洁”的纯LINQ尝试都要好得多。 DRY和**用法**更加简洁和可读。 –

0
var items = "4321"; 
var sortedItems = items.OrderBy(i => i); // Process the order once only 
var sorted = sortedItems.SequenceEqual(items) || sortedItems.SequenceEqual(items.Reverse()); // Reverse using yield return 
+1

'OrderBy'的结果是IEnumberable,应该在每次调用时处理 – c0rd

6

您的解决方案精细且非常易读。其中一个问题是,它需要订购stringwhich is O(n * log(n)),这可以通过迭代string而不对其进行排序来解决。

例如:

var firstDifs = items1.Zip(items1.Skip(1), (x, y) => y - x); 

Linq项目每2项第一string了一些这表明它们的区别,所以,如果你有items1 = "1245"输出将是:

firstDifs: {1, 2, 1}

现在您所需要做的就是验证德那firstDifs是按升序或降序:

bool firstSorted = firstDifs.All(x => x > 0) || firstDifs.All(x => x < 0); //true 

目前:

  • SkipO(1)由于跳过1个细胞所需的动作量是 恒定。
  • ZipO(n)
  • AllO(n)

所以整个溶液为为O(n)

注意这将是更有效的用一个简单的循环,还当第一All返回false因为第三千四百八十七项目改变其方向(例如:1234567891),第二All会无缘无故跑Zip运行两次以上(直到All要求) - 因为有两个迭代AllLinq懒惰地评估它们。

+0

您确定要迭代多次?是不是因为ZIP只是迭代真正需要的元素,所以如果我在ALL的第三次迭代中发现结果是错误的,并且因此ALL的其余部分没有迭代,其余的也没有压缩? –

+0

@HaraldDutch由于有两个'All'和'Linq'迭代懒惰地评估它们,因此压缩将被执行两次。 –

+0

@TamirVered All()停止处理,如果发现一个无效字符,直到** O(n)**正确? – c0rd

0

我会去简单迭代所有元素:

string str = "whatever123"; 
Func<char, char, bool> pred; 
bool? asc = str.TakeWhile((q, i) => i < str.Length - 1) 
       .Select((q, i) => str[i] == str[i+1] ? (bool?)null : str[i] < str[i+1]) 
       .FirstOrDefault(q => q.HasValue); 
if (!asc.HasValue) 
    return true; //all chars are the same 
if (asc.Value) 
    pred = (c1, c2) => c1 <= c2; 
else 
    pred = (c1, c2) => c1 >= c2; 

for (int i = 0; i < str.Length - 1; ++i) 
{ 
    if (!pred(str[i], str[i + 1])) 
     return false; 
} 
return true; 
+0

__Operator'> ='不能应用于'bool'和'char'类型的操作数__ – c0rd

+0

我总是得到'Func'声明错误,我改了它,请立即尝试 – slawekwin

+0

确定现在可以正常工作,但是返回'“aabcdefghijklmnopqrstuvwxyz”'false – c0rd

3

它需要一个reducer。在C#中,它是Enumerable.Aggregate。这是O(n)算法。

var query = "123abc".Aggregate(new { asceding = true, descending = true, prev = (char?)null }, 
(result, currentChar) => 
    new 
    { 
     asceding = result.prev == null || result.asceding && currentChar >= result.prev, 
     descending = result.prev == null || result.descending && currentChar <= result.prev, 
     prev = (char?)currentChar 
    } 
); 
Console.WriteLine(query.asceding || query.descending); 
1

您可以不必比较所有的做多接受的答案要好得多的元素:

var s = "2349"; 
var r = Enumerable.Range(1, s.Length - 1); 
//var isAscending = r.All(i => s[i - 1] <= s[i]); 
//var isDescending = r.All(i => s[i - 1] >= s[i]); 
var isOrdered = r.All(i => s[i - 1] <= s[i]) || r.All(i => s[i - 1] >= s[i]); 
相关问题