Как удалить объект из иерархии с помощью луча?

Question

[/ "BRO TIGER"]

Доброго времени суток обитатели Тостера! Долгое время не было вопросов на тему Программирования и Разработки игр, думаю пора задать вопрос :)
Решил доделать Систему Строительства при этом избавиться от некоторых ненужных скриптов (Тем самым чуть-чуть оптимизировать саму игру). Вот и зашёл в тупик на Уничтожении объектов внутри Родителя! Я хотел с помощью луча от курсора мыши удалить объект из иерархии, но метод удалял саму иерархию а после можно было спокойно удалять объекты так как они уже не находились в иерархии... Вот сам скрипт, а точнее его части:

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


public class BuildingSystem : MonoBehaviour
{

    public LayerMask detailLayer;
    public bool IsBuilding;

    void Update()
    {
        if (IsBuilding && Input.GetMouseButtonDown(1))
        {
            Destroy();
        }
    }


    public void Destroy()
    {
        Ray ray = Camera.main.ScreenPointToRay(Input.mousePosition);

        if (Physics.Raycast(ray, out hit, 20, detailLayer))
        {
            Destroy(hit.transform.gameObject);
        }
    }
}

Заранее Спасибо!

Answer 1

[Denis Gaydak]

толком и не понял в чем собственно вопрос. Как вместо всего родительского объекта удалить только пару детишек внутри??
Тут целая масса вариантов.
1) Отдельные слои/теги для нужных объектов (так же лучом смотрим куда попали, только не первый попавшийся объект. а все кого луч зацепил. RaycastAll метод)
2) по прежнему попав лучом в родительский объект - просто по именам искать (если однотипные или объекты с определенным компонентом. FindChild , GetComponent и тд.
3) Наверно мой любимая вариация варианта номер 2 - на родительские однотипные объекты навесить скрипт(или разные скрипты с наследованием) аля "Инфо о здании" какой нить BuildingComponentRefs.cs ,в котором пару публичных ссылок с подставленными нужными объектами. и возможно даже пару методов для работы с ними, для упрощения себе жизни. всякие аля "Выключить всё" "Включить все" "Пометить красным" и тд и тп. Получаться префабы с уже настроенными/подставленными ссылками. и ничего искать не надо, предполагая что на родительском объекте висит этот "описывающий" скрипт (и так как ссылки подставляются ручками, то не обязательно имена и места/иерархию в префабах прям так строго соблюдать)

Но все вообще сильно упирается в начальную архитектуру приложения и то как вы там что у себя реализовали и что планируете дальше.

Comments

[Denis Gaydak]

ну и так в добавок. а зачем вам прям уничтожать? не устроит просто отключить объекты? их там что сотнями скапливается таких вот "на удаление" за сессию?..
удалить тяжело. а выключить - просто) для сборщика муссора и тд.
ну и так то , на всякий. пулл объектов? вкурсе что такое? заготовки (сделанные на старте или в загрузочной сцене), которые просто включаются, вместо инстанцииации/создания.

[/ "BRO TIGER"]

Denis Gaydak: Мда, вариантов как всегда много, но дело в том что есть у меня в проекте сцена где можно отредактировать свой корабль (С нуля собрать его т.к. это некое "Ноу-хау"). Строительная система менялась в процессе разработки и я думал как сделать лучше ведь когда что-то добавляешь к примеру "Физику воды", то оно имеет последствие) Раньше было просто если Превью Объект входил в триггер "Детали", то он удалял только ту деталь в которой находится Превью Объект (Решение прошлой проблемы, если помните). Сейчас я сделал так чтобы Превью Объект отталкивался от коллайдера детали и теперь нужда в скрипте пропала т.к. Превью Объект не входит в триггер детали... Мне нужно сделать удаление с помощью курсора мыши т.е. когда наводишь курсор на деталь которую нужно удалить (Через Raycast)... Да, наверное всё объяснил... Щас работаю над вашими вариантами, может что-то подойдет или самому идея придет)

[Denis Gaydak]

Ну в общем то вариант с скриптом(со ссылками нужэными) одниковым на каждой "детали" - должен более чем сгодится. вешаете возле тригера/коллайдера. и лучом попадая в объет ищете на солладере компонент (скрипт) , в котором все нудные ссылки для отключения, выделения и прочих нужных деиствий.
Так же скрипт этот поидее может хранить текущее состояние - вкл, выкл, тип детали вставленной сейчас в ячейку и тд.

Answer 2

[Griboks]

А почему бы просто не использовать void OnMouseDown() на самой постройке? Вы, по сути, это и делаете, только через костыль. Если, конечно, за экраном нет невидимой стенки.

Comments

[/ "BRO TIGER"]

Спасибо! Это то что мне нужно было!) Конечно пришлось поработать со скриптами и добавить ещё один, но этот метод работает как надо!)

Answer 3

[/ "BRO TIGER"]

Всем снова привет! Мне понравились несколько вариантов ответов... Это ответ Denis Gaydak и ответ Griboks! И вот как я это сделал - Я переписал старый скрипт и упростил до невозможного! (Спасибо Griboks!)

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

public class DestructibleObject : MonoBehaviour
{
    void OnMouseDown()
    {
        Destroy(gameObject);
    }
}

Дальше я добавил булевскую переменную в Скрипт Строительства:

public bool IsRemove;

А потом сделал скрипт с помощью которого я в детали добавлял Твердые Тела (Rigidbody) для работы с методом [OnMouseDown();]:

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

public class AddComponent : MonoBehaviour {

    public GameObject buildingCamera;
    Rigidbody rig;


    void Start()
    {
        buildingCamera = GameObject.FindGameObjectWithTag("MainCamera");
    }


    void Update ()
    {
        BuildingSystem BS = buildingCamera.GetComponent<BuildingSystem>();

        if (BS.IsRemove)
        {
            if (rig == null)
            {
                rig = gameObject.AddComponent<Rigidbody>();
                rig.constraints = RigidbodyConstraints.FreezeAll;
            }
        }

        if (!BS.IsRemove)
        {
            if (rig != null)
                Destroy(rig);
        }               
	}
}

А позже всё соединил с помощью Менеджера Строительства:

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

public class BuildingHUB : MonoBehaviour {

    public GameObject buildingCamera;

    public GameObject currentPreview;

    BuildingSystem buildingSystem;


    void Start()
    {
        buildingSystem = buildingCamera.GetComponent<BuildingSystem>();
    }


    void Update ()
    {
        currentPreview = buildingSystem.currentpreview.gameObject;

        if (buildingSystem.IsBuilding && Input.GetKeyDown(KeyCode.Delete))
        {
            ToRemove();
        }

        if (buildingSystem.IsRemove && Input.GetKeyDown(KeyCode.Escape))
        {
            RemoveToEscape();
        }
    }


    public void ToRemove()
    {
        buildingSystem.IsRemove = true;
        buildingSystem.IsBuilding = false;
        currentPreview.SetActive(false);
    }


    public void RemoveToEscape()
    {
        buildingSystem.IsRemove = false;
        buildingSystem.IsBuilding = true;
        currentPreview.SetActive(true);
    }
}

Всем Спасибо за ответы!)

Comments

[Griboks]

Есть пара моментов:
1) Camera.main - получить главную камеру.
2) Когда-то давно в документации читал, что событие клика работает с коллайдерами или рендерами. Не знаю, зачем вам rigidbody.

[/ "BRO TIGER"]

1) У меня пока что две Камеры (Позже, возможно сделаю одну камеру, а так работает).
2) Без Rigidbody не работает, не знаю что за странность... (Может всё изменилось если говорите что это было давно).

Answer 4

[Alex Maximovich]

Если коллайдер в который попадает рейкас висит на родителе то все верно, он и удалит родителя. Ну и по этому коду достаточно сложно строить какие-то предположения.

Answer 5

[GreatRash]

А почему вы не объявили переменную hit?

public void Destroy() {
  Ray ray = Camera.main.ScreenPointToRay(Input.mousePosition);
  RaycastHit hit;

  if (Physics.Raycast(ray, out hit, 20, detailLayer)) {
    // теперь в hit что-то будет
  }
}

Comments

[/ "BRO TIGER"]

Объявлял, просто забыл добавить в деталях вопроса)

[GreatRash]

Попробуйте вот так:

public void Destroy() {
  Ray ray = Camera.main.ScreenPointToRay(Input.mousePosition);
  RaycastHit hit;

  if (Physics.Raycast(ray, out hit, 20, detailLayer)) {
    Destroy(hit.collider.gameObject);
  }
}