zim1985 - Viewing Profile

zim1985's Profile User Rating:

Reputation: 73 Whiz

Group:: Contributors
Active Posts:: 540 (0.44 per day)
Joined:: 19-February 10
Profile Views:: 8,460
Last Active:: Apr 10 2013 02:05 AM
Currently:: Offline

Previous Fields

Country:: US
OS Preference:: Windows
Favorite Browser:: Chrome
Favorite Processor:: Intel
Favorite Gaming Platform:: PC
Your Car:: Honda
Dream Kudos:: 50

Latest Visitors

Michael26
28 Apr 2013 - 06:21
HopelessDev
27 Aug 2012 - 05:19
simeesta
22 Feb 2012 - 10:21
nuclearfroggy
22 Feb 2012 - 01:47
sasi247
07 Mar 2011 - 02:45
fallenreaper
03 Nov 2010 - 19:55
macosxnerd101
03 Nov 2010 - 12:00
pdkharkar
09 Jun 2010 - 08:52
webpeater
08 Jun 2010 - 06:43
Scottm
07 Jun 2010 - 21:55

zim1985 has set his status

Posts I've Made

In Topic: TicTacToe Game and Private Member Functions

Posted 27 Feb 2012
Hezekiah, on 27 February 2012 - 07:50 AM, said:
First, the return false; has to be moved to after the loop.
```
if(board[i][0] == '+' || board[0][i] == '+')
```
Here you have the same problem you had with the diagWin() function. If board[i][0] is part of the winning row, and it contains '0', but board[0][i] contains '+', the program will say player 1 has won, even though board[0][i] is not part of the winning row.

If the winning row is a column, you should test the value of board[i][0-2], but if it is a row, you should test board[0-2][i]. If you think about it, there is a cell which always falls in both these ranges: board[i][i].

The reason why it always says player 2 has won, is not because there is a row of '0's, but because there is a row of '*'s. You have to implement a test for that.
I get it. All I had to do was make sure that the tests only passed if the rows were + or 0. How I had it before made it so any row that was all equal would pass.

Thank you so much for your help. You are a genius.

In Topic: TicTacToe Game and Private Member Functions

Posted 27 Feb 2012

Hezekiah, on 27 February 2012 - 07:18 AM, said:

You haven't fixed the rowWin() function. Did you understand my explanation (I don't think I would)? Have another look at my code example, and ask if you don't understand.

I fixed the rest of what you said to fix. I ran the program again and got the same error.

I entered 1 3 for Player 1.
Then 2 3 for Player 2.
The game quit. Again, just saying that Player 2 has won.

Maybe I messed something up that you told me to fix.

#include <cstdlib>
#include <iostream>

using namespace std;

class Game
{
	private:
		char board[3][3];

		bool rowWin()
		{
			int count = 0;
			for(int i = 0; i < 3; i++)
			{
				if((board[0][i] == board[1][i] && board[0][i] == board[2][i]) || (board[i][0] == board[i][1] && board[i][0] == board[i][2])) //condition for 3 in a row
				{
					if(board[i][0] == '+' || board[0][i] == '+') cout << "Player 1 has won.";
					else cout << "Player 2 has won.";
					return true;
				}
				else return false;
			}
		}

		bool diagWin()
		{
			if((board[0][0] == board[1][1] && board[0][0] == board[2][2]) || (board[0][2] == board[1][1] && board[0][2] == board[2][0]))
			{
				if(board[1][1] == '+') cout << "Player 1 has won.";
				else cout << "Player 2 has won.";
				return true;
			}
			else return false;
		}

	public:
		Game()
		{
			for(int i = 0; i < 3; i++)
				for(int j = 0; j < 3; j++)
					board[i][j] = '*';
		}
		
		void setSpot(int player, int i, int j)
		{
			if(player == 1) board[i][j] = '+';
			else board[i][j] = '0';
		}

		void printBoard()
		{
			cout << "-------";
			for(int i = 0; i < 3; i++)
			{
				cout << "\n|";
				for(int j = 0; j < 3; j++)
				{
					cout << board[i][j] << "|";
				}
			}
			cout << "\n-------";
		}

		bool hasWon()
		{
			return rowWin() || diagWin();
		}

		void doTurn()
		{
			int m, n;
			for(int i = 0; i < 2; i++)
			{
				printBoard();
				do
				{
					cout << "\n\nPlayer " << i+1 << " enter row and column of your move. (1-3):   ";
					cin >> m >> n;
					setSpot(i+1, m-1, n-1);
				} while(m <= 0 || m > 3 || n <= 0 || n > 3);
			}
			
		}
};

int main()
{
	Game tic;

	cout << "Welcome to the Tic-Tac-Toe game!\n";
	system("PAUSE");
	do
	{
		system("CLS");
		tic.doTurn();
		cout << "\n";
		system("PAUSE");
	} while(!tic.hasWon());

	return 0;
}

In Topic: TicTacToe Game and Private Member Functions

Posted 27 Feb 2012

Hezekiah, on 27 February 2012 - 07:18 AM, said:
You haven't fixed the rowWin() function. Did you understand my explanation (I don't think I would)? Have another look at my code example, and ask if you don't understand.

I think I get what you were saying in general. Since only one variable really changes, I don't need the nested loops. I'll fix that and then post back and see if it changes anything...

OH also because the array will go out of bounds if I don't do it this way. I get it.

In Topic: TicTacToe Game and Private Member Functions

Posted 27 Feb 2012

Hezekiah, on 27 February 2012 - 06:27 AM, said:

Quote

I'm getting some errors with this code.

Copy them and paste them here. If you don't get the output you expect, post the input, output and expected output.

		bool rowWin()
		{
			for(int i = 0; i < 3; i++)
			{
				for(int j = 0; j < 3; j++)
				{
					if((board[i][j] == board[i+1][j] && board[i][j] == board[i+2][j]) || (board[i][j] == board[i][j+1] && board[i][j] == board[i][j+2])) //condition for 3 in a row
					{
						if(board[i][j] == '+') cout << "Player 1 has won.";
						else cout << "Player 2 has won.";
						return true;
					}
					else return false;
				}
			}
		}

This will access board with invalid indices. When i=1, i+2 will be an invalid index. When i=2, i+1 and i+2 will be invalid indices. You have the same problem with j. You only need one loop here. The counter can be used where you didn't do i+x or j+x (where x is 0, 1 or 2). The other index should use the values 0, 1 and 2:

for(int i = 0; i < 3; ++i) {
	if((board[0][i] == board[1][i] && board[0][i] == board[2][i])
		|| (board[i][0] == board[i][1] && board[i][0] == board[i][2])) {
		//game over
	}
}
//continue playing

The return should also be only after the loop.

		bool diagWin()
		{
			if((board[0][0] == board[1][1] && board[0][0] == board[2][2]) || (board[0][2] == board[1][1] && board[0][2] == board[2][0]))
			{
				if(board[0][0] == '+' || board[0][2] == '+') cout << "Player 1 has won.";
				else cout << "Player 2 has won.";
				return true;
			}
			else return false;
		}

Testing if either board[0][0] or board[0][2] is + could give the wrong result. Rather test board[1][1], which is part of both diagonal rows.

		bool hasWon()
		{
			return rowWin() && diagWin();
		}

That should be ||. You only need one row to win.

		void doTurn()
		{
			int m, n;
			for(int i = 0; i < 2; i++)
			{
				printBoard();
				do
				{
					cout << "\n\nPlayer " << i+1 << " enter row and column of your move. (1-3):   ";
					cin >> m >> n;
					setSpot(i, m-1, n-1);
				} while((m > 0 || n < 4) && (n > 0 || n < 4));
			}
			
		}

The first parameter to setSpot() should be i+1. The loop condition isn't correct. You want to continue while m<=0 or m>3 or n<=0 or n>3. The call to setSpot() should be after the loop.

int main()
{
	Game tic;

	cout << "Welcome to the Tic-Tac-Toe game!\n";
	system("PAUSE");
	do
	{
		system("CLS");
		tic.doTurn();
		system("PAUSE");
	} while(tic.hasWon());

	
}

You want to continue playing while nobody has won, so add a not (!).

That was beyond helpful. I've been up all night trying to get this to work properly and you just fixed so many of my errors in seconds.

It seems now that my program is stuck where I run the test for a winning board. After the second turn, the game decides that Player 2 has won, when he obviously has not, and then quits.

#include <cstdlib>
#include <iostream>

using namespace std;

class Game
{
	private:
		char board[3][3];

		bool rowWin()
		{
			for(int i = 0; i < 3; i++)
			{
				for(int j = 0; j < 3; j++)
				{
					if((board[i][j] == board[i+1][j] && board[i][j] == board[i+2][j]) || (board[i][j] == board[i][j+1] && board[i][j] == board[i][j+2])) //condition for 3 in a row
					{
						if(board[i][j] == '+') cout << "Player 1 has won.";
						else cout << "Player 2 has won.";
						return true;
					}
					else return false;
				}
			}
		}

		bool diagWin()
		{
			if((board[0][0] == board[1][1] && board[0][0] == board[2][2]) || (board[0][2] == board[1][1] && board[0][2] == board[2][0]))
			{
				if(board[1][1] == '+') cout << "Player 1 has won.";
				else cout << "Player 2 has won.";
				return true;
			}
			else return false;
		}

	public:
		Game()
		{
			for(int i = 0; i < 3; i++)
				for(int j = 0; j < 3; j++)
					board[i][j] = '*';
		}
		
		void setSpot(int player, int i, int j)
		{
			if(player == 1) board[i][j] = '+';
			else board[i][j] = '0';
		}

		void printBoard()
		{
			cout << "-------";
			for(int i = 0; i < 3; i++)
			{
				cout << "\n|";
				for(int j = 0; j < 3; j++)
				{
					cout << board[i][j] << "|";
				}
			}
			cout << "\n-------";
		}

		bool hasWon()
		{
			return rowWin() || diagWin();
		}

		void doTurn()
		{
			int m, n;
			for(int i = 0; i < 2; i++)
			{
				printBoard();
				do
				{
					cout << "\n\nPlayer " << i+1 << " enter row and column of your move. (1-3):   ";
					cin >> m >> n;
					setSpot(i+1, m-1, n-1);
				} while(m <= 0 || m > 3 || n <= 0 || n > 3);
			}
			
		}
};

int main()
{
	Game tic;

	cout << "Welcome to the Tic-Tac-Toe game!\n";
	system("PAUSE");
	do
	{
		system("CLS");
		tic.doTurn();
		system("PAUSE");
	} while(!tic.hasWon());

	return 0;
}

I'll continue tweaking and trying to find the error, which I suspect lies in the diagWin function, but I'll leave this here in case anyone else has some better ideas.

Also, sorry. I'm not new to coding, but I am relatively new to C++.

In Topic: Data Not Reaching the Switch Statement?

Posted 22 Feb 2012
simeesta, on 22 February 2012 - 08:09 AM, said:
The error is having four Poly instances when you should only have two.
line 111
```
polynomials[i] = a; 
```
copies the values of a into polynomials[i] and does not pass a reference. You then call
```
polynomials[i].set( coef, exp)
```
in your loop. You then call
```
a.add(b )
```
in switch. You should be calling
```
polynomials[0].add(polynomials[1] )
```
instead.

The value of a.reg is garbage so when you use it to control array access you get a segfault as you're accessing data not in the array.

EDIT: should be polynomials[1] and not b.
This worked perfectly. I should have caught that. Thank you. I'll be sure to be keeping this in mind for future reference.