siva rockeyboy Ответов: 3

Что такое ошибка в этой программе я использовал VOID указатель и функцию для поиска наибольшего из элементов массива

#include <stdio.h>
int compare(void *p,int a[],int n);
int main()
{
    void *p;
    int n,i,j,c;
    scanf("%d",&n);
    int a[n];
    for(i=0;i<n;i++)
    {
        scanf("%d",&a[i]);
    }
    compare(&p,a,n);
    for(i=0;i<n;i++)
    {
        printf("%d",*(int*)p+i);
    }
    
    return 0;
}
int compare(void *p,int a[],int n)
{
    int i,j;
    p = a;
    int f = 0;
    for(i=0;i<n;i++)
    {
        for(j=i+1;j<n;j++)
        {
            if(*(int*)p+i > *(int*)p+j)
            {
                f = 1;
            }
        }
    }
    if(f==1)
    {
        for(i=0;i<n;i++)
        {
            return *(int*)p+i;
        }
    }
}


Что я уже пробовал:

Я попытался получить какой-то вывод, но он не возвращает правильное значение.

ПОМОГИ МНЕ С ЭТИМ.

Richard MacCutchan

Не используйте void*, используйте правильный тип, который в вашем случае является int*.

KarstenK

ваш p не имеет правильного значения, когда вызывается compare? Остальные абсолютно правы в остальном.

3 Ответов

Рейтинг:
25

OriginalGriff

Какой смысл передавать указатель void в вашу функцию, если первое, что вы собираетесь сделать с ним, это выбросить его?

int compare(void *p,int a[],int n)
{
    int i,j;
    p = a;

Серьезно, 1 минута с отладчиком показала бы вам это: так что привыкайте к нему - это самый мощный инструмент, который у вас есть!

Рейтинг:
2

John R. Shaw

Следующее не будет компилироваться, потому что значение 'n' должно быть известно во время компиляции (а не во время выполнения).

scanf("%d",&n);
int a[n];
Вы должны выполнить выделение памяти во время выполнения 
    scanf("%d",&n);
    int *a = malloc(sizeof(int) * n);
... // work with array, then free it when finished
    free(a);
    return 0;
Ваша функция сравнения принимает указатель (сравните(void *p, int a[], int n)) в качестве первого параметра и вы передаете ему адрес указателя (сравните(&p, a, n)), который является указателем на указатель (**p). Поэтому я заключаю, что ваша функция сравнения должна принимать указатель на указатель (сравните(void **p, int a[], int n)).

Это изменит способ обращения к переменной 'p' в функции сравнения:
вместо
*(int*)p
вы бы использовали 
*(int*)*p
Кроме того, ваша функция сравнения либо возвращает не определено значение, если 'f' не равно '1' или любое другое значение хранится в[0].

Вам нужно вернуться к чертежной доске и выяснить, чего именно вы пытаетесь достичь с помощью этого кода. Потому что, что бы это ни было, это не оно.

Напишите подробное описание функции сравнения. Каковы ожидаемые входные и выходные данные? Это то, что вам нужно проверить на уровне вызова функции.

Возьмите каждую линию отдельно, чтобы увидеть, что она делает. Напишите описание линии на английском языке (или языке по выбору). Если вы можете это сделать, тогда вы понимаете это; в противном случае вы этого не сделаете.

Рейтинг:
0

Patrice T

Цитата:
Какая ошибка в этой программе я использовал VOID POINTER

Я уже давно не использую C, но вы уверены в указателе void?

Цитата:
Я попытался получить какой-то вывод, но он не возвращает правильное значение.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.