Why is this not giving me random pairs of numbers

[Ponomous]

Hey, so im trying to make a memory game using 2d arrays.

This is my code to randomly generate the rows and columns but its not giving me pairs, could someone tell me why, i really dont know.

for (int r=0; r<4; r++)
    {
        for (int c=0; c<4; c++)
        {
            cards[r][c]=rand()%8+1;
            cout<<cards[r][c];
        }
        cout<<endl;
    }

[pecet]

You forget to initialize random seed:

srand ( time(NULL) );

and this:

cards[r][c]=rand()%8+1;

gives you just one random number.

[dgr231]

pecet is correct. Also, don't forget to #include <ctime> so that you can use the time() function.

-D

[Ponomous]

so i have

srand((unsigned)time(NULL));

already

so what would be the easiest way to generate pairs of random numbers (single digit from 1-9) then?

[Ponomous]

yep i have ctime already as well, this is just a snippit of my code for generating pairs but im really not sure why it doesnt work

[sfuo]

OK first off I would like to say this is not that great of a way to do it but it works. This generates pairs of numbers from 1-8 at random locations in your matrix (I picked 1-8 because you have a 4x4 matrix) and thats pretty much it.

#include <iostream>
#include <time.h>

using namespace std;

int main()
{
	int cards[4][4];
	bool makeNext = false;
	memset(cards, 0, sizeof(cards));
	srand(time(NULL));
	
	for( int i = 1; i < 9; i++ )
	{
		makeNext = false;
		while(!makeNext)
		{
			int x1, y1, x2, y2;
			x1 = rand()%4;
			y1 = rand()%4;
			x2 = rand()%4;
			y2 = rand()%4;
			if( (x1 != x2 || y1 != y2) && (cards[x1][y1] == 0 && cards[x2][y2] == 0) )
			{
				cards[x1][y1] = i;
				cards[x2][y2] = i;
				makeNext = true;;
			}
		}
	}
	
	for( int i = 0; i < 4; i++ )
	{
		for( int c = 0; c < 4; c++ )
		{
			cout << cards[i][c] << " ";
		}
		cout << endl;
	}

	
	
     
	system("PAUSE");
	return 0;
}

[restrictment]

why not do something like this:

#include<iostream>
#include<time.h>
using namespace std;
int main()
{
	int random[2];
	srand ((unsigned)time(0));
	for(int x=0;x<2;x++)
	{
		random[x] = (rand() % 8) + 1;
		cout << random[x] << " ";
	}
	cout << endl;
}

Edit: Oops, I read it to fast, you want random PAIRS, sorry. Here is the modified code that will print out pairs.

#include<iostream>
#include<time.h>
using namespace std;
int main()
{
	int random[2];
	for(int x=0;x<2;x++)
	{
		srand ((unsigned)time(0));
		random[x] = (rand() % 8) + 1;
		cout << random[x] << " ";
	}
	cout << endl;
}

In this code random[0] and random[1] are two different numbers. If you would like more pairs simply copy and past with random2[0] and random2[1].

[Ponomous]

So i have the thing almost completed but since i have been looking at it for the past 12 hours i know im missing something stupid....

Why is it that the code just outputs zeros even when you pick two matching values

#include <iostream> #include <cstdlib> #include <ctime> using namespace std; int main() { char comma; int row1, column1, row2, column2, cards[4][4], cards_unflipped[4][4] = {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0} , counter(0); srand((unsigned)time(NULL)); //fill board with pairs bool makeNext = false; bool nullexists = true; memset(cards, 0, sizeof(cards)); srand(time(NULL)); for( int i = 1; i < 9; i++ ) { makeNext = false; while(!makeNext) { int x1, y1, x2, y2; x1 = rand()%4; y1 = rand()%4; x2 = rand()%4; y2 = rand()%4; if( (x1 != x2 || y1 != y2) && (cards[x1][y1] == 0 && cards[x2][y2] == 0) ) { cards[x1][y1] = i; cards[x2][y2] = i; makeNext = true;; } } } for( int i = 0; i < 4; i++ ) { for( int c = 0; c < 4; c++ ) { cout << cards[i][c] << " "; } cout << endl; } //display board while (nullexists = true) { //selection cout<<"Please select the first card row and column seperated by a comma.\n"; cin>>row1>>comma>>column1; cout<<"Please select the second card row and column seperated by a comma.\n"; cin>>row2>>comma>>column2; if (cards[row1][column1]==cards[row2][column2]) { cards_unflipped[row1][column1] = cards[row1][column1]; cards_unflipped[row2][column2] = cards[row2][column2]; counter = counter++; if (counter == 8) { bool nullexists = false; } } //reveal the flipped cards cout<<" 1 2 3 4\n"; cout<<" "; for (int i=0; i<=8; i++) {
        cout<<"-"; } cout<<endl; for (int r=0; r<4; r++) {
        cout<<r+1<<" | ";
        for (int c=0; c<4; c++)
        {
        
        cout<<cards_unflipped[r][c]<<" ";
        } cout<<endl;
        }
        } cout << "You Won!" << endl; return 0; }

[vb6exp32]

quoting restrictment

1.
#include<iostream>
2.
#include<time.h>
3.
using namespace std;
4.
int main()
5.
{
6.
int random[2];
7.
for(int x=0;x<2;x++)
8.
{
9.
srand ((unsigned)time(0));
10.
random[x] = (rand() % 8) + 1;
11.
cout << random[x] << " ";
12.
}
13.
cout << endl;
14.
}

thanx for that source ... it works fine in dev c++ beta 5
though for a console app ... is there any way i could know about keeping the console window open until a key is pressed to close it
and a key to toggle the random number ? thanx in advance.