Вывод значения элементов массива

[valdemar593]

Граждане всем доброго времени суток.
Вопрос по коду. Почему при выводе значения элементов все они равны -858993460.

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <windows.h>
using namespace std;
void main()
{
	srand( time(NULL) );
	::SetConsoleCP(1251);
	::SetConsoleOutputCP(1251);
	int ms[5][3];
	int mss[5];
	int znachenie[10000];
	for (int i = 0; i < 10000; i++)
		znachenie[i] = i;
	for (int i = 0; i < 5; i++)
	{
		int j = 0;
		while (j < 3)
		{	
			int a = rand()%10000;
			if (a > 1000) 
			{	
				ms[i][j] = znachenie[a];
				j++;                                   // Сдесь 
				cout << ms[i][j] << " ";  
			}
		}
		cout << endl;
	}
	char n;
	cout << "Введите номер столбца с вашим числом" 
		 << endl;
	cin >> n;
	switch(n)
	{
		case '1':
		for (int i = 0; i < 5; i++)
			{
				ms[i][0] = mss[i];
			}
	
		case '2':
		for (int i = 0; i < 5; i++)
			{
				ms[i][1] = mss[i];
			}
		case '3':
	
		for (int i = 0; i < 5; i++)
			{
				ms[i][2] = mss[i];
			}
	}
	cout << "Ваш номер в этом столбце"
	       << endl;
	for (int i = 0; i < 5; i++)
	{
		cout << mss[i] << endl;             // И сдесь соответственно
	}
}

[netrino]

j++;                                   // Сдесь 
cout << ms[i][j] << " ";

Сначала увеличиваете j, а потом читаете число им адресованное ) проблема в том, что оно ещё не инициализировано, потому там мусор - поменяйте местами эти две инструкции(вывод и инкремент j) и тогда должно нормально работать.
Только один вопрос, зачем заводить массив из 10000 элементов, а затем случайным образом брать оттуда один из них? О_О
Не проще ли так:

ms[i][j] = rand() % 9000 + 1000;

?

[valdemar593]

И вправду так гораздо проще. Только осваиваю функцию rand() cпасибо =). C выводом частично сработало. Первый раз cout нормально отображает матрицу. А вот второй раз после присваивания второму массиву значения элементов выбранного столбика происходит опять фишка с -858993460.

char n;
	cout << "Введите номер столбца с вашим числом" 
		 << endl;
	cin >> n;
	switch(n)
	{
		case '1':
		for (int i = 0; i < 5; i++)
			{
				ms[i][0] = mss[i];
			}
	
		case '2':
		for (int i = 0; i < 5; i++)
			{
				ms[i][1] = mss[i];
			}
		case '3':
	
		for (int i = 0; i < 5; i++)
			{
				ms[i][2] = mss[i];
			}
	}
	cout << "Ваш номер в этом столбце"
	       << endl;
	for (int i = 0; i < 5; i++)
	{
		cout << mss[i] << endl;             // Вот тут
	}

[netrino]

Опять те же грабли %)
Вы же выводите неинициализированный вектор mss!

int n;
cin >> n;
for(int i = 0; i < 5; i++)
    mss[i] = ms[i][n-1];   // вместо switch. и обратите внимание, что теперь mss[i] слева от "="!
...

[valdemar593]

О теперь всё просто и понятно стало, спасибо большое =)

[valdemar593]

Новая неприятность. После ввода номера столбика программа завершается ошибкой
"Run-Time Check Failure #2 - Stack around variable 'ms' was corrupted."

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <windows.h>
using namespace std;
void main()
{
	srand( time(NULL) );
	::SetConsoleCP(1251);
	::SetConsoleOutputCP(1251);
	int ms[5][3];
	int mss[5];
	for (int i = 0; i < 5; i++)
	{
		int j = 0;
		while (j < 3)
		{	
			int a = rand()%9000 + 1000;
			{	
				ms[i][j] = a;
				cout << ms[i][j] << " ";
				j++;
			}
		}
		cout << endl;
	}
	char n;
	cout << "Введите номер столбца с вашим числом" 
		 << endl;
	cin >> n;      // СДЕСЬ
	cout << "Вашь номер в этом столбике" << endl;
	for (int i =0; i < n; i++)
	{
		mss[i] = ms[i][n - 1];
		cout << mss[i];
	}
}

[netrino]

я в своём посте исправил, n должна быть типа int, или нужно дополнительно отнимать от неё 0x30, но так будет некрасиво смотреться )
Кстати, раз уж mss[i] больше нигде не используется, то есть ли смысл сохранять значения столбца в неё?

int n;
cout << "введите номер столбца с вашим числом" << endl;
cin >> n;
for(int i = 0; i < n; i++)
    cout << ms[i][n-1];

[valdemar593]

Логично и просто =)
Спасибо.