Говнокод #27376 — C# — Говнокод.ру

// Update is called once per frame
    void Update () {
        if (!isWin && !isFail && !isPaused)
        {
            if (timeForUnhit > 0) //Для состояния восстановления игрока
            {
                timeForUnhit -= Time.deltaTime;
                //LevelGenerate.Instance.player.GetComponent<SpriteRenderer>().sprite = hitPlayer;
            }
            else if (timeForInvc > 0)
            //Для состояния непобедимости игрока
            {
                timeForInvc -= Time.deltaTime;
                //LevelGenerate.Instance.player.GetComponent<SpriteRenderer>().sprite = invcPlayer;
            }

            else
            {
                //LevelGenerate.Instance.player.GetComponent<SpriteRenderer>().sprite = player;
                LevelGenerate.Instance.player.GetComponent<Animator>().CrossFade(animNames[0], 0);
                if (invc)
                {
                    
                    MusicManager.Instance.gameObject.GetComponent<AudioSource>().clip = MusicManager.Instance.music[1];
                    MusicManager.Instance.gameObject.GetComponent<AudioSource>().Play();
                }
                invc = false;
                
            }
        }

        i = LevelGenerate.Instance.playerY;
        j = LevelGenerate.Instance.playerX;
        
        if (!isWin && !isFail) //Если уровень не завершен
        {
            collideEnemy(); //Обнаружение столкновения с врагом
            collectItem(); //Обнаружения столкновения с собираемым предметом
            genNthOrdColls(2); //Генерация предметов n-ого порядка после сбора предметов (n-1)-ого
            genNthOrdColls(3);
            genNthOrdColls(4);

            if (colls[0] == 0 && colls[1] == 0 && colls[2] == 0 && colls[3] == 0 && LevelGenerate.Instance.resLoaded) isWin = true; //Если все предметы собраны, то уровень завершен с прохождением
        }

        if (isWin && animationSet == 0) //Меняем спрайт игрока при завершении уровня
        {
            //LevelGenerate.Instance.player.GetComponent<SpriteRenderer>().sprite = winPlayer;
            LevelGenerate.Instance.player.GetComponent<Animator>().CrossFade(animNames[1], 0);
            animationSet++;
            delayTime = 1.5f;
            MusicManager.Instance.gameObject.GetComponent<AudioSource>().mute = true;
            SoundManager.Instance.gameObject.GetComponent<AudioSource>().clip = SoundManager.Instance.sounds[0];
            SoundManager.Instance.gameObject.GetComponent<AudioSource>().Play();
        }

        if (isWin && delayTime <= 0)
        {
            //path = Application.dataPath + "\\Levels\\SaveData1";

            /*if (Application.platform == RuntimePlatform.WindowsEditor)
            {
                path = Application.dataPath;
                path = Path.Combine(path, "Levels");
            }
            else if (Application.platform == RuntimePlatform.Android)
                path = Application.persistentDataPath;
            
            path = Path.Combine(path, "SaveData1");
            fs = new FileStream(path, FileMode.Open);
            bw = new BinaryWriter(fs);*/
            levelNum = (byte)(Convert.ToByte(LevelGenerate.Instance.levelFile.Substring(5)) - 1);
            levelNum++;
            if (PlayerPrefs.GetInt("maxLevel") == levelNum)
            {


                
                PlayerPrefs.SetInt("maxLevel", (int)levelNum);
                PlayerPrefs.Save();
            }
            PlayerPrefs.SetInt("level", (int)levelNum);

            /*bw.Write(levelNum);
            
            bw.Write("Level" + (levelNum+1).ToString());
            bw.Close();
            fs.Close();*/
            SceneManager.LoadScene("Win");
        } else if (delayTime > 0)
        {
            delayTime -= Time.deltaTime;
        }
        if (isFail && delayTime <= 0)
        {
            //path = Application.dataPath + "\\Levels\\SaveData1";
            /*if (Application.platform == RuntimePlatform.WindowsEditor)
            {
                path = Application.dataPath;
                path = Path.Combine(path, "Levels");

```
}
else if (Application.platform == RuntimePlatform.Android)
path = Application.persistentDataPath;

path = Path.Combine(path, "SaveData1");
fs = new FileStream(path, FileMode.Open);
bw = new BinaryWriter(fs);

fs.Seek(1, SeekOrigin.Begin);
bw.Write(LevelGenerate.Instance.levelFil e);
bw.Close();
fs.Close();*/
PlayerPrefs.SetString("levelFile", LevelGenerate.Instance.levelFile);
PlayerPrefs.Save();
SceneManager.LoadScene("Fail");
} else if (delayTime > 0)
{
delayTime -= Time.deltaTime;
}
}
```

Самый страшный метод из EventManager-а (модуль который отвечал за все события в игре - коллизию с врагом, таймаут непобедимости и пр.).

Комментарии (3) RSS

Soul_re@ver 25.04.2021 01:10 # +3

Классический God object, хоть в палату мер и весов ставь.
Ответить
Desktop 25.04.2021 01:12 # 0

Всем ягни, пацаны
Ответить
- JaneBurt 26.04.2021 00:34 # +1
  
  Тут проблема была не в ненужном усложнении реализации спроектированных фичей или написании кода который "потом как-нибудь пригодится", а в разделении ответственности. Причем работающей в конечном счете криво, т.к. в одном хуке, который вызывается после отрисовки кадра, выполняется и проверка на столкновения с объектами, и обновление карты для collectibles (на каждом подуровне они генерируются разные), и сохранение состояния прохождения уровня в файл (возможно, из-за этого управление игроком жутко тормозило). Как выше подметили, это эталонный божественный объект. А всё от того, что не удосужился в свое время разобраться с движком Unity досконально как с программируемой средой, с базой по архитектуре игр (которая в общих чертах чем-то напоминает знакомую фронтовую MVVM, с разделением объектов на presentational и behavioral), событийной, плагинной, хуковой моделью, классическими алгоритмами и структурами данных в конце концов.
  Ответить

Говнокод: по колено в коде.

C# / Говнокод #27376

Комментарии (3) RSS

Добавить комментарий