- Brak nazwy zmiennej
- Brak średników
- Brak deklaracji zmiennej n przed jej "użyciem"
- W tej postaci kodu zmienna globalna
. . .
const int N=20
int main()
{
. . .
}
jest "niepotrzebna", bo: Difference Between Local and Global Variable
-
Jeżeli "pytasz" użytkownika o: "podaj liczbe elemntow tablicy a"
cout<<"\n podaj liczbe elemntow tablicy a:";
to sensownym jest zadeklarowanie tablicy z zadaną wielkością n, po tym pytaniu.
-
Sensownym jest pytanie w pętli (w tym przypadku while(...)) o kolejne wartości w tablicy.
cin>>a[i];
Propozycja poprawienia kodu. (można to wykonać mniejszą ilością pętli - użyte dla łatwiejszego zrozumienia)
#include <iostream>
using namespace std;
int main()
{
const int N=20;
int i;
int n=0;
cout << "\npodaj liczbe elemntow tablicy a: ";
cin >> n;
if (n<0 && n>N) n=N;
int a[n];
// Wczytywanie elementów do tablicy jednowymiarowej ...
for (i=0; i<n; i++) {
cout << "podaj wartosc dla a[" << (i+1) << "]: ";
cin >> a[i];
}
cout << endl;
// ... wprowadzanie elementów tablicy na ekran ...
for (i=0; i<n; i++)
cout << " a[" << (i+1) << "]=" << a[i] << endl;
cout << endl;
// ... uzupełnij kod o instrukcje które elementy dodatnie tablicy powiększą o 2
for (i=0; i<n; i++)
if (a[i]>=0) a[i]+=2;
// wygląd elementów tablicy po ... dodatnie tablicy powiększą o 2
for (i=0; i<n; i++)
cout << " a[" << (i+1) << "]=" << a[i] << endl;
return 0;
}