• Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
  • Asset Store
  • Get Unity

UNITY ACCOUNT

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account
  • Blog
  • Forums
  • Answers
  • Evangelists
  • User Groups
  • Beta Program
  • Advisory Panel

Navigation

  • Home
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
    • Blog
    • Forums
    • Answers
    • Evangelists
    • User Groups
    • Beta Program
    • Advisory Panel

Unity account

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account

Language

  • Chinese
  • Spanish
  • Japanese
  • Korean
  • Portuguese
  • Ask a question
  • Spaces
    • Default
    • Help Room
    • META
    • Moderators
    • Topics
    • Questions
    • Users
    • Badges
  • Home /
avatar image
0
Question by Blenovator · Feb 27, 2014 at 07:53 PM · c#script errorsimple

Simple Coding Error Help - C# - Following Packt Book

Got this error following a simple tutorial in the packt book - Unity Android Game Development By Example.

Its a simple game of TicTacToe it seems my boardIndex is not right...

I can't work it out - any help would be greatly appreciated.

Error Code: IndexOutOfRangeException: Array index is out of range. TicTacToeControl.OnGUI () (at Assets/TicTacToeControl.cs:18)

Script:

 using UnityEngine;
 using System.Collections;
 
 public class TicTacToeControl : MonoBehaviour {
 
 public SquareState[] board = new SquareState[9];
 public bool xTurn = true;
 
 public void OnGUI() {
     float width = 75;
     float height = 75;
     
     for(int y=0;y<3;y++) {
         for(int x=0;x<3;y++) {
         
         int boardIndex = (y * 3) + x;
     Rect square = new Rect(x * width, y * height, width, height);
     string owner = board[boardIndex] == SquareState.XControl ? "X" : board[boardIndex] == SquareState.OControl ? "O" : "";
     
     if(GUI.Button(square, owner))
         SetControl(boardIndex);
         }
     }
 }
 public void SetControl(int boardIndex) {
     if(boardIndex < 0 || boardIndex >= board.Length) return;
     
     board[boardIndex] = xTurn ? SquareState.XControl : SquareState.OControl;
     xTurn = !xTurn;
     }
 }
Comment
Add comment · Show 1
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users
avatar image Blenovator · Feb 27, 2014 at 07:45 PM 0
Share

sorry for the rubbish formating.

Line 18 referenced in the error code is the string owner line (i think)

1 Reply

· Add your reply
  • Sort: 
avatar image
0

Answer by zharik86 · Feb 27, 2014 at 08:04 PM

You mistake in cycle for. See, you write:

 for(int y=0;y<3;y++) {
    for(int x=0;x<3;y++) { // mistake y++
 ...

Change to:

 for(int y=0;y<3;y++) {
    for(int x=0;x<3;x++) { // change to x++
Comment
Add comment · Show 2 · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users
avatar image Blenovator · Feb 27, 2014 at 08:10 PM 0
Share

Hero!!

Where are my glasses...

avatar image zharik86 · Feb 28, 2014 at 07:47 AM 0
Share

@Blenovator Check this answer if I helped you.:)

Your answer

Hint: You can notify a user about this post by typing @username

Up to 2 attachments (including images) can be used with a maximum of 524.3 kB each and 1.0 MB total.

Welcome to Unity Answers

If you’re new to Unity Answers, please check our User Guide to help you navigate through our website and refer to our FAQ for more information.

Before posting, make sure to check out our Knowledge Base for commonly asked Unity questions.

Check our Moderator Guidelines if you’re a new moderator and want to work together in an effort to improve Unity Answers and support our users.

Follow this Question

Answers Answers and Comments

21 People are following this question.

avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image

Related Questions

Multiple Cars not working 1 Answer

Distribute terrain in zones 3 Answers

How to call OnTriggerEnter once 5 Answers

Line Renderer Positioning Problem 1 Answer

Prefab script on clones only reference the first clone instantiated 1 Answer


Enterprise
Social Q&A

Social
Subscribe on YouTube social-youtube Follow on LinkedIn social-linkedin Follow on Twitter social-twitter Follow on Facebook social-facebook Follow on Instagram social-instagram

Footer

  • Purchase
    • Products
    • Subscription
    • Asset Store
    • Unity Gear
    • Resellers
  • Education
    • Students
    • Educators
    • Certification
    • Learn
    • Center of Excellence
  • Download
    • Unity
    • Beta Program
  • Unity Labs
    • Labs
    • Publications
  • Resources
    • Learn platform
    • Community
    • Documentation
    • Unity QA
    • FAQ
    • Services Status
    • Connect
  • About Unity
    • About Us
    • Blog
    • Events
    • Careers
    • Contact
    • Press
    • Partners
    • Affiliates
    • Security
Copyright © 2020 Unity Technologies
  • Legal
  • Privacy Policy
  • Cookies
  • Do Not Sell My Personal Information
  • Cookies Settings
"Unity", Unity logos, and other Unity trademarks are trademarks or registered trademarks of Unity Technologies or its affiliates in the U.S. and elsewhere (more info here). Other names or brands are trademarks of their respective owners.
  • Anonymous
  • Sign in
  • Create
  • Ask a question
  • Spaces
  • Default
  • Help Room
  • META
  • Moderators
  • Explore
  • Topics
  • Questions
  • Users
  • Badges