Неправильный вывод?

Question

[Slavon7]

Когда я вывожу максимальный элемент - выводит число 9 (что не есть верным) Через дебагер тоже пробовал: всё равно число 9; раньше работало, а через время перестало; кто сможет найти ошибку, буду очень благодарен.

int main()
{
    const int N = 10; ///Константа, кол-во єлєментів
    int max = 0; /// Змінна 
    int a,b,e = 0; /// Змінні 
    int c = 0;/// Змінна 
    
    int s[10] = {1,5,6,1,2,6,6,2,4,3};

    int *ptr = s;
    

    for(int *ptr = s[0];ptr <= N;ptr++){ /// Цикл перерахунку масива 
        if(ptr > max){ /// Находження максимальног єлємента
            max = ptr;/// Присвоення
        }
    }
    printf("%d", max);
}

Comments

[Армянское Радио]

Объясните это место
for(int *ptr = s[0];ptr <= N;ptr++)

[Seityaya]

Когда я вывожу максимальный элемент - выводит число 9(что не есть верным)

Нужно найти максимальный элемент, или все таки последний?

Answer 1

[res2001]

Вы не правильно инициализируете указатель ptr: int *ptr = s[0];
Обращение s[0] просто возвращает первый элемент массива (в вашем случае 1). Это не указатель.
Правильная запись была бы:

int *ptr = &s[0];
или
int *ptr = s;

Если бы вы поставили максимальный уровень сообщений компилятора, то в этом месте он бы вам ругнулся.
В целом рекомендую в своих проектах всегда выставлять максимальный уровень сообщений и исправлять ошибки, на которые указывает компилятор. В большинстве случаев компилятор выдает абсолютно правильные рекомендации. Конечно бывают и исключения.

Answer 2

[Antony]

for(int *ptr = s[0];ptr <= N;ptr++)

Что с чем вы сравниваете в условии?

Десятый элемент находится по адресу s + N, а не N
Тут же сравнение адреса (который почти всегда сильно больше десяти) с числом. Наверное все-таки как-то так:
for(int *ptr = s; ptr-s <= N; ptr++)
или
for(int *ptr = s; ptr <= s+N; ptr++)

Ну и плюс еще ошибка на которую вам Дмитрий указал

Плюс у вас бесполезная строка int *ptr = s;
Зачем вы объявляете переменную ptr, которая больше нигде не используется? (зато в цикле вы объявляете еще одну переменную с таким же именем, которая не та же самая переменная, что объявлено раньше)

К тому же почему вы считаете ответ 9 не верным? Ваш код выводит максимальное значение, это именно 9.

Comments

[Slavon7]

Да вы чертовский бог

Answer 3

[Дмитрий]

1) зачем использовать указатель? можно же без него
2)

for(int *ptr = s; ptr <= N; ptr++){ /// Цикл перерахунку масива 
        if(*ptr > max){ /// Находження максимальног єлємента
            max = *ptr;/// Присвоення
        }
    }

Comments

[Slavon7]

Задание такое ,хех)
А с этим кодом выводит 0

[CityCat4]

Slavon7, А, тренируемся бегать по массиву указателями? Очень полезный навык. Тогда наверное вот так:

for(int *ptr = &s[0];ptr <= N;ptr++){ /// Цикл перерахунку масива 
        if(ptr > max){ /// Находження максимальног єлємента
            max = ptr;/// Присвоення
        }

Ну и можно инициализацию рабочих переменных цикла делать прямо в стартовой секции - так читается лучше.

[Дмитрий]

int main() {
    const int N = 10; ///Константа, кол-во єлєментів
    int max = 0; /// Змінна 
    
    int s[10] = {1,5,6,1,2,6,9,2,4,3};
    int *ptr = s;
    
    for(int i = 0; i < N; i++){ /// Цикл перерахунку масива 
        if(*ptr > max){ /// Находження максимальног єлємента
            max = *ptr;/// Присвоення
        }
        ptr++;
    }
    
    std::cout << max << "\n";
    return 0;
}

[Дмитрий]

CityCat4, ты сравниваешь указатель с N: ptr <= N

[CityCat4]

Дмитрий, Не, я ничего ни с чем не сравниваю :) Я все свои лабы сделал много-много лет назад...

Answer 4

[profesor08]

максимальный элемент 9

for(int *ptr = s; ptr != s + N; ptr++) {
        if(*ptr > max) {
            max = *ptr;
        }
    }

последний элемент 3 - *(s + (N - 1))