Дефектный бинарный поиск в C

Question

В настоящее время у меня есть программа, которая реализует сортировку вставок, а затем использует бинарный поиск для ее поиска (массив целых).Кажется, у меня в данный момент ошибка 1 off.

Моя сортировка при вставке должна сортироваться в порядке убывания.Сейчас кажется, что значение, сохраненное в последней позиции, отсутствует.

#include <stdio.h>
void insertionSort(int nums[], int size)
{
    int i, key, j;
    for (i = 1; i < size; i++)
    {
        key = nums[i];
        j = i - 1;

        /* Move elements of arr[0..i-1], that are
        greater than key, to one position ahead
        of their current position */
        while (j >= 0 && nums[j] > key)
        {
            nums[j + 1] = nums[j];
            j = j - 1;
        }
        nums[j + 1] = key;
    }
}

int binarySearch(int nums[], int size, int searchVal)
{
    int l = 0, r = size - 1;

    while (l <= r)
    {
        int m = l + (r - l) / 2;

        // Check if x is present at mid
        if (nums[m] == searchVal)
            return m;

        // If x greater, ignore left half
        if (nums[m] < searchVal)
            l = m + 1;

        // If x is smaller, ignore right half
        else
            r = m - 1;
    }

    // if we reach here, then element was
    // not present
    return -1;
}

int main()
{
    int n;
    printf("Enter the number of elements (between 1 and 50) in the array: \n");
    scanf("%d", &n);
    int i, nums[n];
    printf("Enter %d positive integers: \n", n);
    for (i = 0; i < n; i++)
    {
        scanf("%d", &nums[i]);
    }
    int x = 0;
    insertionSort(nums, n);
    printf("Enter a positive integer or -1 to quit: \n");
    scanf("%d", &x);
    do
    {
        int ind = binarySearch(nums, n, x);
        if (ind > 0)
        {
            printf("Found\n");
        }
        else
        {
            printf("Not Found\n");

        }
        printf("Enter a positive integer or -1 to quit: \n");
        scanf("%d", &x);
    } while (x != -1);

    return 0;
}

Результаты:

Enter the number of elements (between 1 and 50) in the array:
9
Enter 9 positive integers:
7
4
10
49
6
12
32
17
Enter a positive integer or -1 to quit:
4
Not Found
Enter an positive integer -1 or to quit
12
Found
Enter a positive integer or -1 to quit:
5
Not Found
Enter a positive integer or -1 to quit:
49
Found
Enter a positive integer or -1 to quit:
-1

Как вы видите, все работает, но первый тест, в котором я проверяю число 4. Кто-нибудь знает, почему меня отключили на 1?

Answer 1

#include <stdio.h>

void insertionSort (int nums[], int size)
{
    int i, key, j;
    for (i = 1; i < size; i++) {
        key = nums[i];
        j = i - 1;

        /* Move elements of arr[0..i-1], that are
           greater than key, to one position ahead
           of their current position */
        while (j >= 0 && nums[j] > key) {
            nums[j + 1] = nums[j];
            j = j - 1;
        }
        nums[j + 1] = key;
    }
}

int binarySearch (int nums[], int size, int searchVal)
{
    int l = 0, r = size - 1;

    while (l <= r) {
        int m = l + (r - l) / 2;

        // Check if x is present at mid
        if (nums[m] == searchVal)
            return m;

        // If x greater, ignore left half
        if (nums[m] < searchVal)
            l = m + 1;

        // If x is smaller, ignore right half
        else
            r = m - 1;
    }

    // if we reach here, then element was
    // not present
    return -1;
}

int main ()
{
    int n;
    printf
        ("Enter the number of elements (between 1 and 50) in the array: \n");
    scanf ("%d", &n);
    int i, nums[n];
    printf ("Enter %d positive integers: \n", n);
    for (i = 0; i < n; i++) {
        scanf ("%d", &nums[i]);
    }
    int x = 0;
    insertionSort (nums, n);
    printf ("Enter a positive integer or -1 to quit: \n");
    scanf ("%d", &x);
    do {
        int ind = binarySearch (nums, n, x);
        if (ind >= 0) {
            printf ("Found\n");
        } else {
            printf ("Not Found\n");

        }
        printf ("Enter a positive integer or -1 to quit: \n");
        scanf ("%d", &x);
    } while (x != -1);

    return 0;
}

Попробуйте это только одна ошибка, которая решается, если (ind> = 0)

Answer 2

Хотя вы обнаружили, что проверка (ind > 0) приводит к невозможности найти элементы по индексу 0, а (ind >= 0) предоставляет решение, давайте рассмотрим короткий тестовый пример, который проверяет, binarySearch() находит все элементы вашегоотсортированный массив, при котором пользователю не нужно постоянно вводить или перенаправлять информацию в вашу программу.

Каждый раз, когда вы тестируете алгоритмы, предоставляет простую структуру проверки, которая проверяет все элементы, числа и т. д. без требования ввода пользователя.Возьмите ваши 9 номеров, например.Простое включение инициализированного массива и сортировка, а затем цикл по всем значениям подтвердит, работает ли binarySearch(), как задумано.(это также не спасет вас от горя).Короткий код, выполняющий поиск по всему массиву, может быть простым:

int main (void) {

    int nums[] = { 7, 4, 10, 49, 6, 12, 32, 17, 21 },
        n = sizeof nums / sizeof *nums;

    insertionSort (nums, n);
    for (int i = 0; i < n; i++)
        printf (" %2d  (%s)\n", nums[i], 
                binarySearch (nums, n, nums[i]) >= 0 ? "found" : "not found");
}

Пример использования / Вывод

$ ./bin/bsearchtest
  4  (found)
  6  (found)
  7  (found)
 10  (found)
 12  (found)
 17  (found)
 21  (found)
 32  (found)
 49  (found)

Каждый элемент теперь указан как"found" или "not found" и код завершается самостоятельно.