• Есть массив, не получается от каждого элемента отнять 20?
    Сделать на с++

    #include "pch.h"
    #include

    using namespace::std;

    int main()
    {


    int i, arr[] = {1,2,3,4,5,6,7,8,9};

    for (i = 1; i < 10; i++) {
    arr[i] = arr[i] - 20;

    }
    cout << arr[i] << " ";

    }

Ответы 4

  • Не понимаю к чему тут некоторые исправления ошибок... 1. Условий тут никаких нету , есть только проблема которую просят решить 2. Я взял скопировал код который прикрепили( и который имеет ошибки,которые автор не просит исправить) и исправил проблему(допуская при этом свои ошибки)3. Тут задача для студентов, а не для школьников и я не знаю кем надо быть, что бы учиться на программиста и не переписать код с готовым алгоритмом под себя.
  • 1 - с этим никто и не спорит2 - именно, из-за ваших ошибок и удалили3 - так это и сделали)
  • всем спасибо сам уже ришил смотрите на новую задачу
  • Ошибка 1

    using namespace::std;

    Верно использовать пространство имён таким образом:

    using namespace std;

    Ошибка 2

    Вы объявляете итератор ещё перед циклом, причём вместе с массивом. Надо понимать, что итератор - это счётчик, int. А массив - это совокупность таких частичек int. Не нужно всё под одну крышу загонять. Объявляйте итераторы (если они Вам не нужны, например, для результирующих счётов или прочего - по ситуации) в самом цикле, не мучайте глаза программистов.

    Правильно объявлять так:

    int arr[] = {1,2,3,4,5,6,7,8,9};

    for(int i = ..)

    Ошибка 3

    Вы начинаете цикл с 1, хотя Вам нужно пройтись по всему массиву, начиная с самого первого элемента. Поскольку всё строится на десятичной системе счисления (от 0 до 9), то первый элемент будет иметь номер 0. Следовательно, цикл следовало бы начать с нуля.

    Правильно объявить цикл нужно так:

    for(int i = 0; ..)

    Ошибка 4

    Поскольку вы не указываете размер массива, а потом считаете 10 элементов (хотя это уже неправильно, цикл возьмёт один лишний, уже выходящий из заданного Вами массива кусочек и отнимет от него 20)

    Правильно было бы объявлять условие цикла так:

    for(int i = 0; i < (sizeof(arr)/sizeof(int)); i++){/*...*/}

    То есть мы берём размер массива в байтах, размер типа int в байтах (по сути это константа - 4 байта, но на разных версиях оси она может соответствовать разным значениям) и делим размер массива на размер типа, массив которого мы создали (в нашем случае - int)

    А лучше всего вынести выражение sizeof(arr)/sizeof(int) в отдельную переменную:

    int arraySize = sizeof(arr)/sizeof(int);

    for(int i = 0; i < arraySize; i++)

    Разница в том, что при каждой итерации цикл считает заново размер массива - а это лишние операции.

    Ошибка 5

    Так как Вы хотите вывести каждое результатирующее число через пробел,  Вам нужно выводить его через каждую итерацию цикла, а не после него. Ваша ошибка будет заключаться в том (если не учитывать предыдущие), что цикл инкрементирует переменную i до 10 и выведет на консоль arr[i], то есть arr[10] элемент, которого как раз-таки нет в массиве (объяснил в 3 ошибке)

    Правильно было бы выводить так:

    for(int i = 0; i < arraySize; i++){

    arr[i] = arr[i] - 20;

    cout << arr[i] << " ";

    }

    Замечание 6

    Функция типа int main() должна всегда возвращать что-то. Это стандарт языка.

    То есть, правильно было бы в конце добавить:

    return 0;

    Итоговый код и проверка представлены на в приложении:

    answer img
  • Добавить свой ответ

Еще вопросы

Войти через Google

или

Забыли пароль?

У меня нет аккаунта, я хочу Зарегистрироваться

How much to ban the user?
1 hour 1 day 100 years