Dlaczego c++ wyrzuca mi "smieci''. Jak to naprawić? Co zmienić?

Question

Generalnie zaczynam się uczyć i jestem na etapie tablic. Mam problem z tym że program po kompilacji wyrzuca mi jakieś śmieci z systemu? Nie wiem jak to nazwac. Co zmienić w kodzie? Co naprawić? Dlaczego tak się dzieje? A może totalnie robie źle. Będę wdzięczny za pomoc i wytłumaczenie dlaczego tak a nie inaczej. Załączam jeszcze ss aby było wiadomo o co chodzi.

 

 

 

 Napisz program wczytujący z klawiatury 5 liczb całkowitych. Wczytane liczby
 należy wydrukować na ekranie w odwrotnej kolejności.

#include <iostream>

using namespace std;

int main() {

  int a, b, c, d, e;

  cout << "Wczytaj pierwszą liczbę: ";
  cin >> a;
  cout << "Wczytaj drugą liczbę: ";
  cin >> b;
  cout << "Wczytaj trzecią liczbę: ";
  cin >> c;
  cout << "Wczytaj czwarta liczbę: ";
  cin >> d;
  cout << "Wczytaj piątą liczbę: ";
  cin >> e;

  int numbers[] = {a, b, c, d, e};
  for (int i = 5; i >= 0; i--) {
    cout << numbers[i] << endl;
  }
}

 

Comments

To nie prościej napisać:

  int numbers[] = {e, d, c, b, a};

i potem już iterować normalnie,. do przodu.

---

 for (int i = 5; i >= 0; ) {
    cout << numbers[--i] << endl;

@Micky, może tak :--i, zamiast i--;

 for (size_t i = 4; i <5; i--) { //tak chyba najpoprawniej by było
    cout << numbers[i] << endl;

 

Answer 1

Przeanalizuj ten kod. Moim zdaniem po tej modyfikacji bez problemu znajdziesz czemu drukuje śmieci. Mam nadzieję że kod działa. 

#include <iostream>
 
using namespace std;
 
int main() {
 
  int a, b, c, d, e;
 
  cout << "Wczytaj pierwszą liczbę: ";
  cin >> a;
  cout << "Wczytaj drugą liczbę: ";
  cin >> b;
  cout << "Wczytaj trzecią liczbę: ";
  cin >> c;
  cout << "Wczytaj czwarta liczbę: ";
  cin >> d;
  cout << "Wczytaj piątą liczbę: ";
  cin >> e;
 
  int numbers[] = {a, b, c, d, e};
  for (int i = 5; i >= 0; i--) {
    cout << "numbers[" << i << "] = " << numbers[i] << endl;
  }
}

 

Answer 2

Tablice są indeksowane od 0 (zera), dlatego musisz wziąć to pod uwagę, przyjęło się, że do iterowania po tablicy bierze się 

długość tablicy - 1

Czyli w Twoim przypadku

for (int i = 4; i >= 0; i--) ...

 

//      element   1  2  3  4  5      array.length = 5
//   indeks (i)   0  1  2  3  4      iterate - array.length - 1
int numbers[] = { a, b, c, d, e };