Co tu jest źle

[pablos11]

using UnityEngine;
using System.Collections;
using System.Collections.Generic;

public class ekwipunek : MonoBehaviour {
	public int itemsInRow = 3;
	public int buttonWidth = 50;
	public int buttonHeight = 50;
	public int buttonMargin = 10;
	public Rect inventoryPosition = new Rect(20, 20, 0, 0);
	private List<ItemAbstract> itemsList = new List<ItemAbstract> ();
	private bool openInventory;
	private Staty ps;
	private int i = 0;
	private int j = 0;
	void Start() 
	{
		openInventory = false;
		ps = gameObject.GetComponent<Staty> ();
		inventoryPosition = new Rect (inventoryPosition.x, inventoryPosition.y, ((itemsInRow * buttonWidth) + ((itemsInRow + 1) * buttonMargin)), ((itemsInRow * buttonHeight) + ((itemsInRow + 1) * buttonMargin)));
	}
	void Update()
	{
		if (Input.GetKeyDown (KeyCode.I)) {
			openInventory = !openInventory;
		}
	}
	public void addItem(ItemAbstract item)
	{
		itemsList.Add (item);
	}
	void OnGUI()
	{
		if (openInventory) {
			int numerOfItems = itemsList.Count;
			int rows = Mathf.CeilToInt(numerOfItems / itemsInRow) + 1;
			
			GUILayout.BeginArea (inventoryPosition);
			for( i = 0 ; i < rows ; i++) {
				for( j = 0 ; j < itemsInRow ; j++) {
					if((i * itemsInRow) + j < numerOfItems) {
						if (GUI.Button(new Rect((buttonMargin + buttonMargin * j + buttonWidth * j),
						                        (buttonMargin + buttonMargin * i + buttonHeight * i),
						                        buttonWidth, 
						                        buttonHeight), itemsList[(i * itemsInRow) + j].getItemIcon())) {
							bool disposable = itemsList[(i * itemsInRow) + j].execute(ps);
							if(disposable) {
								itemsList.RemoveAt((i * itemsInRow) + j);
							}
							
						}
					} else {
						if (GUI.Button(new Rect((buttonMargin + buttonMargin * j + buttonWidth * j),
						                        (buttonMargin + buttonMargin * i + buttonHeight * i),
						                        buttonWidth, 
						                        buttonHeight), "")) {
							Debug.Log("Eq otwarte");
						}
					}
				}
			}
			GUILayout.EndArea ();
		}
	}
	if (GUI.Button(new Rect((buttonMargin + buttonMargin * j + buttonWidth * j), (buttonMargin + buttonMargin * i + buttonHeight * i), buttonWidth, buttonHeight), itemsList[(i * itemsInRow) + j].getItemIcon())) 
	{
		bool disposable = itemsList[(i * itemsInRow) + j].execute(ps);
		if(disposable)
		{
			itemsList.RemoveAt((i * itemsInRow) + j);
		}
	}
}

Unity rzuca się o ( i ) w bloku otwierania EQ, ale nie wiem o co może mu chodzić ilość nawiasów się zgadza, nie ma za dużo i za mało, chyba że przeoczyłem istotny szczegół

[endriu322]

 

 

"openInventory = false;"   zmień na "openInventory = true;"

 

chyba to ale nie jestem pewny

[Hans Kloss PL]

A co to za if luzem? W klasie możesz mieć tylko pola, właściwości i metody.

[pablos11]

@Hans Kloss PL2x to jest "blok" który na starcie zamyka eq, żeby nie było otwarte od startu gry

 

@Hans Kloss PL

Czyli zwykły void Update wystarczy jako blok i w ten blok tego if'a wrzucić?

[MistrzMaster]

@pablos11

Przecież samo z siebie się nie otworzy, możesz to dodać w konstruktorze, wtedy wykona się tylko raz przy tworzeniu obiektu.

[pablos11]

O ile pamiętam bool ma podstawową "formę" - true, więc na starcie muszę zmienić ją na false żeby nie otworzyło eq po włączeniu gry

[RexioManTM]

Przecież możesz od razu ustawić bool na false:

private bool openInventory = false;

[pablos11]

Bool nie ma znaczenia w tym kodzie, czy ustawię go na starcie, czy w deklaracji, czy też po X czasu.

 

Ważny był błąd braku bloku do if'a :p

[MistrzMaster]

@pablos11

No to było oczywiste, ale jeśli w konstruktorze tej klasy nie otworzysz eq, to ono się nie otworzy samo z siebie.