web-dev-qa-db-ja.com

ソートされた配列の二分探索

このバイナリ検索コードを使用して、降順でソートされた配列を検索しようとしています。ただし、並べ替えて検索しようとすると、結果が返されず、無限ループがあるかのように消えることのない読み込みアイコンが表示されます。コードが論理的に見えるので、何が問題なのかわかりません。

これは、4.0フレームワークを使用したaspx、c＃です。前もって感謝します！

    protected void Button2_Click(object sender, EventArgs e)
    {
        String item = TextBox1.Text;
        int target = Convert.ToInt16(item);
        int mid, first = 0, last = mynumbers.Length - 1;

        //for a sorted array with descending values
        while (first<=last)
        {
            mid = (first + last) / 2;
            if (target < mynumbers[mid])
                first = mid + 1;
            if (target > mynumbers[mid])
                last = mid - 1;
            else
                Label11.Text = "Target " + item + " was found at index " + mynumbers[mid];

        }
c#arraysbinary-search
12
Emmanuel Buckley

Arrayクラスにはバイナリ検索があります：

int index = Array.BinarySearch(mynumbers, target);

降順の場合、これはReverseComparerを使用して簡単に実行できます。

    public class ReverseComparer<T> : IComparer<T>
    {
        public int Compare(T x, T y)
        {
            return Comparer<T>.Default.Compare(y, x);
        }
    }

次に：

int index = Array.BinarySearch(numbers, 7, new ReverseComparer<int>());

これが学術的な演習であり、カスタム検索を使用する必要がある場合は、もちろんこれは適用されません。それがクラスのカスタムアルゴリズムである必要がある場合、問題はループが検出されたときにループから抜け出さなければならず、インデックスがmynumbers[mid]ではなくmidにあることです。

    //for a sorted array with descending values
    while (first<=last)
    {
        mid = (first + last) / 2;

        if (target < mynumbers[mid])
        {
            first = mid + 1;
        }

        if (target > mynumbers[mid])
        {
            last = mid - 1;
        }

        else
        {
            // the index is mid, not mynumbers[mid], and you need to break here
            // once found or it's an infinite loop once it finds it.
            Label11.Text = "Target " + item + " was found at index " + mid;
            break;
        }
    }

そして実際には、アルゴリズムを純粋に保つために代わりにブールフラグを設定し、検索結果と出力の懸念を混同しないようにします。これにより、not foundでループを終了した場合に何が起こったのかが簡単にわかります。

    bool found = false;

    //for a sorted array with descending values
    while (!found && first<=last)
    {
        mid = (first + last) / 2;

        if (target < mynumbers[mid])
        {
            first = mid + 1;
        }

        if (target > mynumbers[mid])
        {
            last = mid - 1;
        }

        else
        {
            // You need to stop here once found or it's an infinite loop once it finds it.
            found = true;
        }
    }

    Label11.Text = found 
        ? "Item " + item + " was found at position " + mid
        : "Item " + item + " was not found";
20
James Michael Hare

当て推量：

if (target < mynumbers[mid]) 
   first = mid + 1; 
else if (target > mynumbers[mid]) 
   last = mid - 1; 
else 
{
    ....
    break;
}

Mid + 1とmid-1の間を行き来しているのではないかと思います

1
n8wrl

これは正しい1つです。

もしtarget< mynumbers[mid]次に、下位から範囲1、つまりmid-1で検索する必要があるため、lastからmid-1を使用する必要があります。

while (first<=last)
        {
            mid = (first + last) / 2;
            if (target == mynumbers[mid])
            Label11.Text = "Target " + item + " was found at index " + mynumbers[mid];

            else if (target < mynumbers[mid])
                last = mid - 1;
            else (target > mynumbers[mid])
                first = mid + 1;

            }
1
Ravindra Bagale
//this works fine with these Test cases    
// has to check if (target == mynumbers[mid])    
// this is for an array with ascending order.
class Program
{

    static void Main(string[] args)
    {
        // TEST cases:
        // for 8: item 8 was not found
        // for 4: item 4 found at Position 3
        // for 1: item 1 found at position 0
        // for 0: item 0 was not found


        int target =8;
        int searchkey = target;

        int[] mynumbers = { 1, 2, 3, 4, 5 };

        int mid=0, first = 0, last = mynumbers.Length - 1;

        bool found = false;

        //for a sorted array with ascending values
        while (!found && first <= last)
        {
            mid = (first + last) / 2;

            if (target == mynumbers[mid])
                found = true;
            else
            {

                if (target > mynumbers[mid])
                {
                    first = mid + 1;
                }

                if (target < mynumbers[mid])
                {
                    last = mid - 1;
                }

            }

        }

        String foundmsg = found
            ? "Item " + searchkey + " was found at position " + mid
            : "Item " + searchkey + " was not found";
        Console.WriteLine(foundmsg);
     }
}
0
Jean

それは私のために働いた

public static int search(int[] arr, int value)
{
    Debug.Assert(arr.Length>0);
    var left = 0;
    var right = arr.Length - 1;

    while (((right - left)/2) > 0)
    {
        var middle = left + (right - left)/2;
        if (arr[middle] < value)
            left = middle + 1;
        else
            right = middle;
    }
    return arr[left] >= value ? left : right;
}
0
dprotopopov
        int num = 2;
        int[] value = { 1, 2, 3, 4, 5, 6,7,8,9,10,11,12,13,14,15,16,17,18,19,20 };
        int s = 0;
        int e = value.Length - 1;
        int m = (s + e) / 2;
        bool find = false;
        do
        {
            if (value[m] == num)
            {
                Console.WriteLine("We have found the given number");
                find = true;
            }
            else if (num < value[m])
            {
                e = m - 1;
            }
            else 
            {
                s = m + 1;
            }
            m = (s + e) / 2;
        } while (find != true);
0
Elvis Sylejmani