[C++] Tablica z liczbami nieparzystymi

[MrFreezZ]

Witam mam do napisania program który mówi żeby utworzyć tablicę zawierającą \(\displaystyle{ n}\) liczb całkowitych nieparzystych. Problem w tym że nie wiem jak wypełnić tą tablicę liczbami nieparzystymi. W tym momencie mam napisane tylko tyle ale wywala mi jakieś liczby z kosmosu.

Kod: Zaznacz cały

#include <iostream>
#include <cmath>
#include <math.h>
#include <cstdlib>
#include <ctime>

using namespace std;

int main()
{
    int n;
		cout<<"Podaj ilosc elementow tablicy"<<endl;
			cin>>n;
				int* tab = new int[n];
				
		for(int i=0; i<n; i++)
			cout<<tab[i]<<endl;	
			
		for(int i=0; i<n; i++)
		{
			if(tab[i]%2==1)
				cout<<tab[i]<<endl;
		}	
	system("pause");
	return 0;
}

Co mam zrobić jak to poprawić żeby działało?

[mortan517]

Przeczytaj dokładnie o dynamicznym alokowaniu pamięci, bo masz wyciek.

[kalwi]

To raz, a dwa - że ty tej tablicy niczym nie wypełniasz, więc wyświetlają się randomowe liczby z pamięci.
Poza tym:

Kod: Zaznacz cały

#include <cmath>
#include <math.h>

Te dwa to jest jedno i to samo (ale w cpp wybiera się pierwszy wariant).
Po drugie - po co inkludować całą resztę, skoro jedyne co jest potrzebne to wyświetlanie i pobieranie tekstu?

Po trzecie - wyciek pamięci. Jak nie wiesz jak działają tablicę, to za dynamiczne alokowanie pamięci nie bierz się tym bardziej.

Po czwarte - skoro tego nie wiesz, to użyj vectora. Tak jak Bjarne Stroustrup radzi początkującym.

Po piąte - naucz się formatować kod. To jest ważniejsze niż Ci się wydaje.

I po szóste:

Kod: Zaznacz cały

#include <iostream>
#include <vector>

using namespace std;

int main()
{
   int n;
   cout << "Podaj ilosc elementow tablicy" << endl;
   cin >> n;

   vector <int>tab(n);
   for(int i = 0; i < n; ++i)
     tab[i] = 2 * i + 1; 
   for(int i = 0; i < n; ++i)
     cout << tab[i] << endl;         
         
   return 0;
}

[Gouranga]

Kod: Zaznacz cały

#include <cstdio>

int main(void){
  printf("Podaj ilosc elementow: ");
  int n;
  scanf("%d", &n);
  int i, tab[n];
  for (i=0; i<n; i++) tab[i] = 2*i+1;
return 0;
}

W treści nie ma mowy o wyświetleniu tablicy, więc to jest najprostszy tworzący tablicę wypełnioną liczbami.