Zle liczy/wywala program

Question

Witam.Czy mógłby ktoś pomóc mi znaleść błąd w tym program.On ma odwrócić tablicę.T=liczba tablic    n=liczba zmiennych w tablicy.

#include <iostream>

using namespace std;


int main ()
{
	int t,schowek;
	int n,tab[10000],nn;
	cin>>t;
	for (int i=0;i<t;i++)
	 {
		cin>>n;
		nn=n;
		for (int j=0;j<n;j++)
      {
      	cin>>tab[j];
	  }	
for (int ii=0; ii!=n-1 ;n--,ii++)
	  {
	  schowek=tab[ii];
	  tab[ii]=tab[n-1];
	  tab[n-1]=schowek;
      }
	  for (int iii=0;iii<nn;iii++)
	  {
	  	cout<<tab[iii]<<endl;
	  }

	  }
	
	return 0;
	
	
}

 

Comments

A ty badz madry i pisz ladny kod

---

On liczy dobrze dla n=nieparzysta liczba.Ładny kod.

Answer 1

Kod jest okropny. Brak jakiegokolwiek formatowania(jeśli nie umiesz to przed wstawieniem kodu na forum użyj: http://format.krzaq.cc/ ), słabe nazwy zmiennych. Co do problemu, tutaj:

for (int ii=0; ii!=n-1/*niepotrzebnie odejmujesz 1*/ ;n--,ii++)
//powinno być
for (int ii=0; ii<(n/2) ;n--,ii++)

btw. dlaczego do odwracania tablicy nie użyjesz po prostu funkcji reverse( http://en.cppreference.com/w/cpp/algorithm/reverse )

Comments

Ja tez dziekuje, bo szukalem tej stronki do formaowania tak dlugo, plusik leci

---

Dzięki.Jakiego formatowania?

---

Kodu.

---

Co robi to formatowanie?Porządkuje kod?

---

Tak, robi cos czego ty nie robisz

---

https://pl.wikipedia.org/wiki/Formatowanie_kodu

---

Tak w ogóle  ten kod robi błędnie program w dalszym ciągu.Dobrze zamienia ale później coś kombinuje i jest żle.Nie potrafię znaleść błędu.Bardziej bym powiedział ,że nie zamienia tablicy momentami a momentami zamienia

Answer 2

On liczy dobrze dla n=nieparzysta liczba