Оптимизация кода?

Question

[piffo]

Есть 2 довольно больших скрипта.

Вот первый

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

public class PlayerControler : MonoBehaviour
{
    Rigidbody2D rb;
    bool isGround = false;
    public float maxSpeed = 10f;
    private bool isFacingRight = true;
    private Animator anim;
    public BoxCollider2D[] col9;
    public CapsuleCollider2D col3;

    void Start()
    {
        rb = GetComponent<Rigidbody2D>();
        anim = GetComponent<Animator>();
        col9 = GetComponents<BoxCollider2D>();
        col3 = GetComponent<CapsuleCollider2D>();

        kuk();
    }

    void OnTriggerStay2D(Collider2D col)
    {
        if (col.tag == "ground") isGround = true;
    }
    void OnTriggerExit2D(Collider2D col)
    {
        if (col.tag == "ground") isGround = false;
    }

    void Update()
    {
        if (Input.GetKeyDown(KeyCode.Space) && isGround)
        {
            puk();
            anim.SetBool("jump", true);
        }
    }

    void FixedUpdate()
    {
        rb.velocity = new Vector2(Input.GetAxis("Horizontal") * 6f, rb.velocity.y);
        float move = Input.GetAxis("Horizontal");
        anim.SetFloat("Speed", Mathf.Abs(move));
        if (move > 0 && !isFacingRight)
            Flip();
        else if (move < 0 && isFacingRight)
            Flip();
        rb.velocity = new Vector2(move * maxSpeed, rb.velocity.y);
        anim.SetBool("jump", isGround);
        if (!isGround)
            return;
    }

    void puk()
    {
        rb.AddForce(transform.up * 10f, ForceMode2D.Impulse);
    }

    void kuk()
    {
        col9[0].offset = new Vector2(0f, -0.41f);
        col9[1].offset = new Vector2(0f, -0.08f);
        col3.offset = new Vector2(0f, 0.29f);
    }

    private void Flip()
    {
        isFacingRight = !isFacingRight;
        transform.Rotate(0, 180, 0);
    }
}

Это основной скрипт персонажа, он висит на префабе персонажа, который есть на каждой сцене. Это те методы, которые должны быть на персонаже, на каждой сцене. Если нужно что-то добавить для определённой сцены, то я создаю отдельный скрипт и уже добавляю его на соответствующую сцену. Вот о каждом методе:
Start, здесь мы всё кэшируем.
OnTriggerStay2D и OnTriggerExit2D для того, чтобы проверять стоит персонаж на земле или нет.
Update для прыжка.
Fixed Update для перемещения вправо, влево и его переворота; также для того чтобы персонаж не прыгал бесконечно.
puk это я думаю ясно.
kuk чтобы при старте игры коллайдеры персонажа перемещались.
Flip тоже ясно.

Вот второй скрипт. Он висит на персонаже на одном из уровней.

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

public class transition : MonoBehaviour
{
    public GameObject came;
    public GameObject charter;
    public GameObject txt;
    public GameObject move1;
    public GameObject[] cames;
    int wer = 5;
    bool bol;
    Vector2 vec;
    Vector3 came1;
    Vector3 came2;

    private void Start()
    {
        StartCoroutine(fuk());
        bol = cames[0].GetComponent<camemove31>().bol;
        came1 = new Vector3(0f, 0f, 0f);
        came2 = new Vector3(3.93f, 0f, 0f);
    }

    private void Update()
    {
        Debug.Log(wer);
        if (bol == true)
        {
            cames[2].SetActive(false);
            cames[3].SetActive(true);
        }
        if (bol == false)
        {
            cames[2].SetActive(true);
            cames[3].SetActive(false);
        }
        if (came.transform.position == came1)
            bol = true;
        if (came.transform.position == came2)
            bol = false;
    }

    IEnumerator fuk()
    {
        txt.SetActive(true);
        yield return new WaitForSeconds(5f);
        txt.SetActive(false);
    }

    IEnumerator dick()
    {
        this.gameObject.GetComponent<Animator>().enabled = false;
        yield return new WaitForSeconds(0.1f);
        this.gameObject.GetComponent<Animator>().enabled = true;
    }

    void OnTriggerEnter2D(Collider2D col)
    {
        if (col.gameObject.name == "set5")
        {
            kok();
            came.transform.position = vec;

            if (wer == 4)
            {
                wer--;
                return;
            }
            if (wer == 3)
            {
                wer--;
                return;
            }
            if (wer != 4 || wer != 3)
                wer = 5;
        }
        if (col.gameObject.name == "wall3")
        {
            StartCoroutine(dick());
            charter.transform.position = new Vector3(12.456f, -2.817f, 90f);
            came.transform.position = came2;

            if (wer == 5)
            {
                wer--;
                return;
            }
            if (wer == 1)
                SceneManager.LoadScene("Level4");
            if (wer != 5 || wer != 1)
                wer = 5;
        }
        if (col.gameObject.name == "wall4")
        {
            StartCoroutine(dick());
            charter.transform.position = new Vector3(-8.527035f, -2.821f, 90f);
            came.transform.position = came1;

            if (wer == 2)
            {
                wer--;
                return;
            }
            if (wer != 2)
                wer = 5;
        }
        if (col.gameObject.name == "cametrig")
        {
            cames[0].SetActive(true);
        }
        if (col.gameObject.name == "cametrig2")
        {
            cames[1].SetActive(true);
        }
    }

    void OnTriggerStay2D(Collider2D col)
    {
        if (col.gameObject.name == "set5")
        {
            kok();
            came.transform.position = vec;
        }
    }

    void OnTriggerExit2D(Collider2D col)
    {
        if (col.gameObject.name == "set5")
        {
            charter.transform.position = new Vector3(-7.62f, 2.79f, 90f);
            move1.SetActive(true);
        }
    }

    void kok()
    {
        vec = charter.transform.position;
    }
}

Да, я знаю жуткий говнокод, здесь я думаю ничего не нужно объяснять, достаточно почитать. Общая концепция уровня такая: выхода с уровня нету, если персонаж пойдёт налево, то его перенесёт направо, если налево, то соответственно направо. Также есть яма, если туда прыгнуть, то персонажа перенесёт обратно на землю. Чтобы пройти уровень нужно пройти налево, два раза вниз, направо и опять налево.
Так вот к сути. Как эти скрипты можно оптимизировать?

Comments

[2CHEVSKII]

Больших? Хех. Вот это - большой скрипт.

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

З.Ы. не надо называть методы таким образом. Эрекция мешает отвечать.

[piffo]

2CHEVSKII, да, я понимаю что это не большие скрипты, но они большие в масштабе моего проекта. И я прекрасно понимаю, что размер моего dick не влияет на его производительность. Но я только учу, как c# так и unity, и поэтому маленький говнокод в моём первом проекте, может вылиться в большую проблему в будущем. Это мой первый проект, который от силы весит пару десятков мб, как бы в нём не были написаны скрипты, это практически никак не повлияет на производительность, но как раз на этом проекте я бы хотел отработать все свои недочёты и натянуть сову на грёбанный глобус. Поправьте меня, если я не прав.
ps "не надо называть методы таким образом. Эрекция мешает отвечать." Учту

[2CHEVSKII]

puk999, ладно, тогда проанализирую ваш код:
С точки зрения читабельности - отвратительно. Названия методов, переменных и т.п. должны быть самодескриптивными, то есть - название должно давать примерное представление о функциональности.
Эти вектора безумные - лучше вынести в отдельные переменные с понятными названиями, и их переиспользовать. Это и на производительности скажется положительно.
Также - я обычно выношу весь функционал, вызываемый хуками, в отдельные методы, тоже полезно для читабельности кода. Когда начнете ваять компоненты на 2к строк - поймете, что это важно.
FixedUpdate 1-го скрипта:
Ваш if-elseif можно превратить в один лишь if, дабы не повторять код:

if (move > 0 && !isFacingRight || move < 0 && isFacingRight)
{
    Flip();
}

На кой хер это нужно, вообще не шарю:

if (!isGround)
                return;

Этот вот метод больше нигде не вызывается, нахрен он существует? Тогда уж все из Update перенесите в отдельный метод, и туда же всуньте логику puk()

Такая же претензия к методу kok() во втором скрипте.

Ну и в целом код выглядит очень разнообразным, такого быть не должно, придерживайтесь одного стиля.

Также - не злоупотребляйте GetComponent'ами, они не бесплатные, в плане производительности, лучше все, что будете использовать, заранее в Start/Awake вытащить в поля, и их уже использовать.

Как то так.

[piffo]

2CHEVSKII, большое спасибо, это как раз то что мне нужно было). Разнообразность кода объясню тем, что половину кода я писал сам, а половину напрямую списывал с видео уроков в Ютубе и тд. И кстати весь код который вы выделили у себя в комментарии как раз таки оттуда(только метод kok моя ошибка). Эти строчки кода ещё с того времени, когда я сам не мог написать не строчки. Не знаю почему я их уже давно сам не рассмотрел. Наверно думал то, что если они с YouTube, то они идеальны.) С векторами и сам иногда путаюсь(организую это дело). Только не совсем понял что вы имеете ввиду под "функционал вызываемые хуками"?
PS насчёт названий понял принял.

[2CHEVSKII]

puk999, хуки - это я неправильно назвал. Правильно будет Unity Messages, то есть Update, Start, OnTriggerEnter, etc.

[piffo]

Понял, спасибо.
Жаль, что нельзя ваш комментарий как решение выделить.

[2CHEVSKII]

puk999, ну я могу в ответы кинуть, тогда отметьте.

upd. кинул.

[piffo]

2CHEVSKII, отметил

[DanielMcRon]

2CHEVSKII, ой ору, логика puk kok fuk dick, фантазии у автора, конечно, классные

[piffo]

DanielMcRon, это вы ещё другие скрипты не видели)

Answer 1

[2CHEVSKII]

puk999, ладно, тогда проанализирую ваш код:
С точки зрения читабельности - отвратительно. Названия методов, переменных и т.п. должны быть самодескриптивными, то есть - название должно давать примерное представление о функциональности.
Эти вектора безумные - лучше вынести в отдельные переменные с понятными названиями, и их переиспользовать. Это и на производительности скажется положительно.
Также - я обычно выношу весь функционал, вызываемый хуками, в отдельные методы, тоже полезно для читабельности кода. Когда начнете ваять компоненты на 2к строк - поймете, что это важно.
FixedUpdate 1-го скрипта:
Ваш if-elseif можно превратить в один лишь if, дабы не повторять код:

if (move > 0 && !isFacingRight || move < 0 && isFacingRight)
{
    Flip();
}

На кой хер это нужно, вообще не шарю:

if (!isGround)
                return;

Этот вот метод больше нигде не вызывается, нахрен он существует? Тогда уж все из Update перенесите в отдельный метод, и туда же всуньте логику puk()

Такая же претензия к методу kok() во втором скрипте.

Ну и в целом код выглядит очень разнообразным, такого быть не должно, придерживайтесь одного стиля.

Также - не злоупотребляйте GetComponent'ами, они не бесплатные, в плане производительности, лучше все, что будете использовать, заранее в Start/Awake вытащить в поля, и их уже использовать.

Как то так.