<h2>Ошибка 1</h2>
using namespace::std;
<h3>Верно использовать пространство имён таким образом:</h3>
using namespace std;
<h2>Ошибка 2</h2>
Вы объявляете итератор ещё перед циклом, причём вместе с массивом. Надо понимать, что итератор - это счётчик, int. А массив - это совокупность таких частичек int. Не нужно всё под одну крышу загонять. Объявляйте итераторы (если они Вам не нужны, например, для результирующих счётов или прочего - по ситуации) в самом цикле, не мучайте глаза программистов.
<h3>Правильно объявлять так:</h3>
int arr[] = {1,2,3,4,5,6,7,8,9};
for(int i = ..)
<h2>Ошибка 3</h2>
Вы начинаете цикл с 1, хотя Вам нужно пройтись по всему массиву, начиная с самого первого элемента. Поскольку всё строится на десятичной системе счисления (от 0 до 9), то первый элемент будет иметь номер 0. Следовательно, цикл следовало бы начать с нуля.
<h3>Правильно объявить цикл нужно так:</h3>
for(int i = 0; ..)
<h2>Ошибка 4</h2>
Поскольку вы не указываете размер массива, а потом считаете 10 элементов (хотя это уже неправильно, цикл возьмёт один лишний, уже выходящий из заданного Вами массива кусочек и отнимет от него 20)
<h3>Правильно было бы объявлять условие цикла так:</h3>
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++)
Разница в том, что при каждой итерации цикл считает заново размер массива - а это лишние операции.
<h2>Ошибка 5</h2>
Так как Вы хотите вывести каждое результатирующее число через пробел, Вам нужно выводить его через каждую итерацию цикла, а не после него. Ваша ошибка будет заключаться в том (если не учитывать предыдущие), что цикл инкрементирует переменную i до 10 и выведет на консоль arr[i], то есть arr[10] элемент, которого как раз-таки нет в массиве (объяснил в 3 ошибке)
<h3>Правильно было бы выводить так:</h3>
for(int i = 0; i < arraySize; i++){
arr[i] = arr[i] - 20;
cout << arr[i] << " ";
}
<h2>Замечание 6</h2>
Функция типа int main() должна всегда возвращать что-то. Это стандарт языка.
<h3>То есть, правильно было бы в конце добавить:</h3>
return 0;
Итоговый код и проверка представлены на в приложении:
