Помогите исправить код!

[zoss]

Вобщем делаю выделение юнитов. Юниты то выделются то нет(в основном нет).

делаю так:

с помощью selectRect.Contains(camera.WorldToScreenPoint(UM.Unitses[i].transform.position)) определяю есть ли в выбраном прямоугольнике обьекты из массива UM.Unitses и заношу их в другой массив.

вот полный код:

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

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

public class UnitControl : MonoBehaviour
{
        Vector3 poin1, point2;
        //int layerMaskDefault=1<<0;
        float height, width;
        public Rect selectRect,allRect;
        public Texture2D selectTextur;
        public bool DragMouse;
        public GameObject Selectr;
        UnitManager UM;

        void Start(){
                UM=gameObject.GetComponent<UnitManager>();
        }
        void Update(){
        //      allRect=new Rect(0,0,Screen.width,Screen.height);
               
                for(int t=0; 1000>t;t++)
                {
                //Debug.Log("PAROVOZIK TIR TIR TIR");
                GameObject mad=GameObject.FindGameObjectWithTag("Unit");
                if(mad){
                mad.gameObject.tag="UnitMaked";
                UM.Unitses.Add(mad);                   
                }
                else{
                                break;
                        }
                }
               
                if (Input.GetMouseButtonDown(0)){
                        poin1=Input.mousePosition;
                        Clear();
                }
                if (Input.GetMouseButton(0)){
                        point2=Input.mousePosition;
                        DragMouse=true;
                }
                if (Input.GetMouseButtonUp(0)){
                        DragMouse=false;
                        Select();
                }
        }

        void OnGUI(){
                if (DragMouse){
                        width=-poin1.x+point2.x;
                        height=(Screen.height-point2.y)-(Screen.height-poin1.y);
                        selectRect=new Rect(poin1.x,Screen.height-poin1.y,width,height);
                        GUI.DrawTexture(selectRect,selectTextur,ScaleMode.StretchToFill,true);

                }

        }

        void Select(){
               
               
               
               
                width=-poin1.x+point2.x;
                height=(Screen.height-point2.y)-(Screen.height-poin1.y);
                if (width<0){
                        width-=width+width;
                }
                if (height<0){
                        height-=height+height;
                }
                if(point2.x<poin1.x){
                        var x1=poin1.x;
                        var x2=point2.x;
                        poin1.x=x2;
                        point2.x=x1;
                }
                if(point2.y<poin1.y){
                        var y1=poin1.y;
                        var y2=point2.y;
                        poin1.y=y2;
                        point2.y=y1;
                }
                selectRect=new Rect(poin1.x,Screen.height-poin1.y,width,height);
                for(int i=0; UM.Unitses.Count>i;i++){
                       
                        if(selectRect.Contains(camera.WorldToScreenPoint(UM.Unitses[i].transform.position))){
                                Debug.Log("PAROVOZIK TIR TIR TIR");
                        UM.Unitses[i].gameObject.tag="UnitSelected";
                        //Instantiate(Selectr, UM.Unitses[i].transform.position, Quaternion.identity);
                        UM.UnitsesSelected.Add(UM.Unitses[i]);
                        }else{
                                Clear();
                        }
                }
               
                }
               
                void Clear(){
                for(int i=0; UM.UnitsesSelected.Count>i;i++){
                       
                        UM.UnitsesSelected[i].gameObject.tag="UnitMaked";
                        //Instantiate(Selectr, UM.Unitses[i].transform.position, Quaternion.identity);
                        //UM.UnitsesSelected.Add(UM.Unitses[i]);
                        }
               
                        UM.UnitsesSelected.Clear();
                }
}
 

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

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

public class UnitManager : MonoBehaviour {
       
        public List<GameObject> Unitses=new List<GameObject>();//Список для хранения всех юнитов
        public List<GameObject> UnitsesSelected=new List<GameObject>();//Список для хранения выделеных юнитов
}

 

И вообще правильно ли управлять юнитами с помощью тегов?

[extravert]

попробуйте дописать ниже

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

void OnDrawGizmos() {
Gizmos.DrawWireCube(new Vector3(selectRect.x, selectRect.y), new Vector3(selectRect.x+selectRect.width,selectRect.y+selectRect.height, 2));
}

проблему это конечно не решит, но покажет вам линиями положение selectRect. Соответственно будет видно правильно ли производятся вычисления по ректанглу.
Если захотите рисовать линии вне OnDrawGizmos - используйте Debug.DrawLine()
Если хотите чтобы Gizmos показывался в редакторе - используйте [ExecuteInEditMode] прямо перед объявлением класса

[BornFoRdeatH]

А что это у вас такое?

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

for(int t=0; 1000>t;t++)
                {
                //Debug.Log("PAROVOZIK TIR TIR TIR");
                GameObject mad=GameObject.FindGameObjectWithTag("Unit");
                if(mad){
                mad.gameObject.tag="UnitMaked";
                UM.Unitses.Add(mad);                    
                }
 

Вы каждый апдейт тысячу раз выполняете поиск объектов с тегом юнит, и что вы делаете с массивом обьектов, непонятно?

Загляните хоть в справку
http://docs.unity3d.com/Documentation/S ... thTag.html

[zoss]

А что это у вас такое?

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

for(int t=0; 1000>t;t++)
                {
                //Debug.Log("PAROVOZIK TIR TIR TIR");
                GameObject mad=GameObject.FindGameObjectWithTag("Unit");
                if(mad){
                mad.gameObject.tag="UnitMaked";
                UM.Unitses.Add(mad);                    
                }
 

Вы каждый апдейт тысячу раз выполняете поиск объектов с тегом юнит, и что вы делаете с массивом обьектов, непонятно?

Я сначала заполняю массив юниты, потом проверяю есть ли они в прямоугольнике выделения и меняю им тег. Вы не до конца смотрели, там если условие не выполняется то происходит break, у меня стоит три юнита и он дальше трёх не заполняет.

[extravert]

А почему бы просто не сделать так?
1. Объявляем в классе статик поле коллекции. Например

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

public class Unit : MonoBehaviour {
public static List<Unit> AllUnits = new List<Unit>();
}

2. В методах Awake() и Destroy() класса Unit пишем:

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

private void Awake() {
AllUnits.Add(this);
}
private void Destroy() {
AllUnits.Remove(this); //насчет команды удаляющей содержимое из коллекции не скажу точно но вроде так
}
 

3. Где нам нужно обращаемся к элементам например так:

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

var a = Units.AllUnits.First(x => x.MyValue == 0); //Выберет первого юнита который имеет значение MyValue равное 0
 

И не нужно никаких сумасшедших лишних выборок, каждый круг проверяя весь список объектов на принадлежность

[zoss]

попробуйте дописать ниже

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

void OnDrawGizmos() {
Gizmos.DrawWireCube(new Vector3(selectRect.x, selectRect.y), new Vector3(selectRect.x+selectRect.width,selectRect.y+selectRect.height, 2));
}

проблему это конечно не решит, но покажет вам линиями положение selectRect. Соответственно будет видно правильно ли производятся вычисления по ректанглу.
Если захотите рисовать линии вне OnDrawGizmos - используйте Debug.DrawLine()
Если хотите чтобы Gizmos показывался в редакторе - используйте [ExecuteInEditMode] прямо перед объявлением класса

OnDrawGizmos рисует в совсем другом месте! хотя в OnGUI я использую теже координаты и он рисует нормально!

Вроде разобрался, там в коде была ошибка:

вместо

if(point2.y<poin1.y){
var y1=poin1.y;
var y2=point2.y;
poin1.y=y2;
point2.y=y1;
}

должно быть

if(point2.y>poin1.y){
var y1=poin1.y;
var y2=point2.y;
poin1.y=y2;
point2.y=y1;
}

Выделение стало лучше, но всёравно половину раз не выделят! Может есть другой способ?


extravert , спасибо! сейчас попробую, но проблема не в этом, а в том что прямоугольник косячно выделяет юнитов!

[extravert]

Почитай внимательно про Gizmos, я привел пример использования координат по x/y, а тебе возможно нужно по x/z. Он точно покажет то что ты ищешь

[gturk]

а может быть дело в том что не все юниты пересекают плоскость selectRect, если так, то можно заменить Rect на Bounds