Skip to content

Commit dbaefa5

Browse files
author
victor.scherbakov
committed
fix incorrect CompareTo usage
1 parent abc4933 commit dbaefa5

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

DataTanker/Core/AccessMethods/BPlusTree/BPlusTree.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,13 @@ internal class BPlusTree<TKey, TValue> : IBPlusTree<TKey, TValue>
3232
return null;
3333
}
3434

35-
if (entries[0].Key.CompareTo(key) == 1)
35+
if (entries[0].Key.CompareTo(key) > 0)
3636
{
3737
index = 0;
3838
return null;
3939
}
4040

41-
if (entries[last - 1].Key.CompareTo(key) == -1)
41+
if (entries[last - 1].Key.CompareTo(key) < 0)
4242
{
4343
index = last;
4444
return null;
@@ -745,7 +745,7 @@ private bool CheckNodeEntriesAreOrdered(IBPlusTreeNode<TKey> node, out string me
745745
message = string.Empty;
746746
for (int i = 0; i < node.Entries.Count - 1; i++)
747747
{
748-
if (node.Entries[i].Key.CompareTo(node.Entries[i + 1].Key) != -1)
748+
if (node.Entries[i].Key.CompareTo(node.Entries[i + 1].Key) >= 0)
749749
{
750750
message = $"Disordered entries in node: {node.Index}";
751751
return false;
@@ -778,7 +778,7 @@ private bool CheckNodeRanges(IBPlusTreeNode<TKey> node, out string message)
778778
var smallestKey = node.Entries.First().Key;
779779
var largestKey = previous.Entries.Last().Key;
780780

781-
if (previous.Entries.Last().Key.CompareTo(node.Entries.First().Key) != -1)
781+
if (previous.Entries.Last().Key.CompareTo(node.Entries.First().Key) >= 0)
782782
{
783783
message =
784784
$"Smallest key ({smallestKey}) of node: {node.Index} is smaller than the largest key ({largestKey}) of previous node: {node.PreviousNodeIndex}";

0 commit comments

Comments
 (0)