There are certain classes that I only use in the editor and I would like that when I build the game they don’t get included in it.
I thought defining preprocessor directives with #if UNITY_EDITOR around the whole class would be enough but I get messages in the build that the script is missing from the gameObjects it is on in editor mode.
What can I do to not include certain scripts in the build despite the fact that they’re on gameObjects?
Well, no instructions you give to the C# compiler are going to alter what components are included in objects in your scene. Hypothetically, you’d need some sort of instructions to Unity’s build process that tells it to remove those components from the scene when exporting a build. I’m not sure if any feature like that exists.
You could use #if UNITY_EDITOR to remove the methods from the classes, so that they don’t have any executable code (other than what they inherit from MonoBehaviour).
Why exactly are you concerned about removing those classes from your build?
Because they give me warnings that the component on the gameObject is missing and it’s just not clean :). I could go for removing all the contents of the class instead of the whole class but it’s something I’d rather avoid.
I was wondering if there wasn’t a tag to put on the class itself to not include it in the build?
I do this sometimes:
public class MyClass : Monobehaviour
{
#if UNITY_EDITOR
// Do your normal editor stuff
#else
void OnEnable()
{
// This will remove this component and won't destroy the GameObject
Destory(this);
}
#endif
}
See Editor folder.
That removes the script references to the gameObjects, so it doesn’t work.
IMHO it’s very bad to mix editor and run-time functionality. Editor scripts (stuff what are running in the editor) should be in Editor folders.
I don’t understand what references will be removed? You asked the question, why you’re getting errors that some scripts are missing. You’re getting the error because you’re keeping garbage references to editor scripts but you are not including those scripts in the run-time.
You shouldn’t have any references to editor scripts EVER in run-rime scripts and/or game objects.
I do this as well… minus the destroy. I mean a dummy/empty component out there doesn’t really cost at all that much. So I don’t really waste my time destroying it anyhow.
A place I use this is something like this:
using UnityEngine;
using System.Collections;
using System.Linq;
using com.spacepuppy;
using com.spacepuppy.Collections;
using com.spacepuppy.Utils;
using com.mansion.Entities.Weapons;
using com.mansion.Messages;
namespace com.mansion.DebugScripts
{
#if (UNITY_EDITOR || DEVELOPMENT_BUILD)
public class DebugStartup : SPComponent, IGameStateChangedGlobalHandler
#else
public class DebugStartup : SPComponent
#endif
{
#if UNITY_EDITOR
[SerializeField]
private EpisodeSettings _episodeSettings;
[SerializeField]
[Tooltip("Set with scene name to load if death occurs before any checkpoint occurs. Otherwise no reload occurs.")]
private string _debugCheckpointReloadScene;
protected override void Awake()
{
base.Awake();
if (!Game.Initialized)
{
Debug.Log("DEBUG INITIALIZING GAME");
//Cursor.visible = false;
Game.Init((g) =>
{
if (_episodeSettings != null) g.SetEpisodeSettings(_episodeSettings);
});
_dispatchStartMessage = true;
if (_episodeSettings != null && _episodeSettings.ScenarioType != null)
Game.StartScenario(_episodeSettings.ScenarioType).StartGame(new SaveGameToken()
{
Data = null,
Checkpoint = new CheckpointState(string.IsNullOrEmpty(_debugCheckpointReloadScene) ? Constants.DEBUG_STARTUP_CHECKPOINT_SCENE : _debugCheckpointReloadScene)
});
}
}
#endif
#if (UNITY_EDITOR || DEVELOPMENT_BUILD)
private bool _dispatchStartMessage;
private bool _showDebugMenu;
private Texture2D _background;
protected override void Start()
{
base.Start();
_background = new Texture2D(1, 1);
_background.SetPixels(new Color[] { Color.gray });
Messaging.RegisterGlobal<IGameStateChangedGlobalHandler>(this);
if(_dispatchStartMessage)
{
Messaging.FindAndBroadcast<IDebugStartMessageHandler>((o) => o.OnDebugStart());
}
}
protected override void OnDestroy()
{
base.OnDestroy();
Messaging.UnregisterGlobal<IGameStateChangedGlobalHandler>(this);
}
private void Update()
{
if(Input.GetKeyDown(KeyCode.BackQuote))
{
const string TOKEN_DEEBUGPAUSE = "*DebugMenuPause*";
_showDebugMenu = !_showDebugMenu;
if (_showDebugMenu)
Game.Scenario.GameStateStack.Push(GameState.Paused, TOKEN_DEEBUGPAUSE);
else
Game.Scenario.GameStateStack.Pop(TOKEN_DEEBUGPAUSE);
}
}
private void OnGUI()
{
if (!_showDebugMenu) return;
var style = new GUIStyle(GUI.skin.box);
style.normal.background = _background;
GUI.Box(new Rect(10f, 10f, 500f, 300f), "DEBUG MENU", style);
GUILayout.BeginArea(new Rect(15f, 60f, 490f, 245f));
//validate player exists
var playerEntity = IEntity.Pool.Find<IEntity>((e) => e.Type == IEntity.EntityType.Player);
if (playerEntity == null)
{
GUILayout.BeginHorizontal();
GUILayout.FlexibleSpace();
GUILayout.Label("No player was located.");
GUILayout.FlexibleSpace();
GUILayout.EndHorizontal();
goto Finish;
}
//infinite ammo
var ammo = playerEntity.FindComponent<AmmoPouch>();
if(ammo != null)
{
foreach(var e in System.Enum.GetValues(typeof(AmmoType)).Cast<AmmoType>())
{
GUILayout.BeginHorizontal();
var oldcnt = ammo.GetAmmoCount(e);
var cnt = DiscreteFloatField(e.ToString() + " Ammo Count: ", oldcnt);
if(oldcnt != cnt)
ammo.SetAmmoCount(e, cnt);
bool oldInfAmmo = float.IsPositiveInfinity(cnt);
bool infAmmo = GUILayout.Toggle(oldInfAmmo, " Infinite Ammo");
if (oldInfAmmo != infAmmo)
{
ammo.SetAmmoCount(e, infAmmo ? float.PositiveInfinity : 10f);
}
GUILayout.EndHorizontal();
}
}
//Health - God Mode
if(playerEntity.HealthMeter != null)
{
playerEntity.HealthMeter.MaxHealth = FloatField("Max Health", playerEntity.HealthMeter.MaxHealth);
playerEntity.HealthMeter.Health = FloatField("Health", playerEntity.HealthMeter.Health);
bool oldGodMode = float.IsPositiveInfinity(playerEntity.HealthMeter.Health);
bool godMode = GUILayout.Toggle(oldGodMode, " God Mode");
if(godMode != oldGodMode)
{
if(godMode)
{
playerEntity.HealthMeter.MaxHealth = float.PositiveInfinity;
playerEntity.HealthMeter.Health = float.PositiveInfinity;
}
else
{
playerEntity.HealthMeter.MaxHealth = 100f;
playerEntity.HealthMeter.Health = 100f;
}
}
}
//1-btn grapple release
/*
var grapple = playerEntity.FindComponent<com.mansion.Entities.Actors.Player.PlayerGrappledState>();
if(grapple != null)
{
GUILayout.BeginHorizontal();
GUILayout.Label("Release Odds: " + grapple.GrappleReleaseOdds.ToString("P"));
grapple.GrappleReleaseOdds = Mathf.Clamp01(GUILayout.HorizontalSlider(grapple.GrappleReleaseOdds, 0f, 1f));
GUILayout.EndHorizontal();
}
*/
var actionMotor = playerEntity.FindComponent<com.mansion.Entities.Actors.Player.PlayerActionMotor>();
if(actionMotor != null && actionMotor.DefaultMovementSettings != null)
{
GUILayout.BeginHorizontal();
GUILayout.Label("Release Odds: " + actionMotor.DefaultMovementSettings.GrappleReleaseOdds.ToString("P"));
actionMotor.DefaultMovementSettings.GrappleReleaseOdds = Mathf.Clamp01(GUILayout.HorizontalSlider(actionMotor.DefaultMovementSettings.GrappleReleaseOdds, 0f, 1f));
GUILayout.EndHorizontal();
}
Finish:
GUILayout.EndArea();
}
private static float DiscreteFloatField(string label, float value)
{
GUILayout.BeginHorizontal();
GUILayout.Label(label);
string val = GUILayout.TextField(value.ToString());
//val = System.Text.RegularExpressions.Regex.Replace(val, @"^[0-9]", "");
if (!float.TryParse(val, out value))
value = 0;
GUILayout.EndHorizontal();
return value;
}
private static float FloatField(string label, float value)
{
GUILayout.BeginHorizontal();
GUILayout.Label(label);
string val = GUILayout.TextField(value.ToString());
//val = System.Text.RegularExpressions.Regex.Replace(val, @"^[0-9\.\+\-]", "");
if (!float.TryParse(val, out value))
value = 0;
GUILayout.EndHorizontal();
return value;
}
void IGameStateChangedGlobalHandler.OnGameStateChanged(GameState last, GameState current)
{
if ((current & GameState.Paused) == 0)
{
_showDebugMenu = false;
}
}
#endif
}
}
Basically in our current project we want to be able to start our game from any scene in the editor.
But if you do this… the game isn’t necessarily initialized yet. Certain actions that usually occur at the start of the game in our boot scene haven’t necessarily occurred yet if you just press play in the editor.
I also use this as a place to have an optional debug HUD display in the game if I press the tilde key.
But in game when I finally deploy… all that’s left is an empty ‘DebugStartup’ script that does nothing.
You can use [PostProcessScene] to delete all components of the type on build:
Here’s the relevant parts from our codebase:
public static class DestroyUnwantedScripts {
private static readonly Type[] typesToDeleteOnBuild = {
typeof(A),
typeof(B),
// etc
};
[PostProcessScene]
public static void DeleteObjects() {
if (BuildPipeline.isBuildingPlayer) {
foreach (var type in typesToDeleteOnBuild) {
Debug.Log($"Destroying all instances of {type.Name} on build!");
foreach (var obj in ObjectFinder.FindAll(type, true)) {
Object.DestroyImmediate(obj);
}
}
}
}
}
There’s more to it, but that’s the important bits. ObjectFinder is a utility we have that also looks for the components on inactive objects, you can use FindObjectsOfType instead, but that will skip over inactive objects.
We also have some types that we destroy on entering play mode or on builds (they’re simply used to attach OnSceneGUI to some objects or things like that), but that code-path is horrible and slow.
That’s really interesting! Do you mind showing me the FindAll method of your ObjectFinder class? I tried it on my builds and it has destroyed my scripts even after the builld (it deleted my scripts in the project).
But it might be because I’m not using your method.
I’m at home now until Monday, so that’ll take a while.
It essentially looks at the current scene, grabs the root objects, calls GetComponents on those, and does the same with all of the root objects’ children.
It shouldn’t be removing the scripts outside of builds, if you include the BuildPipeline.isBuildingPlayer check. Unless you’re using Resources.FindObjectsOfTypeAll, which you should not. I was referencing Object.FindObjectsOfType.
Here’s the script I used:
#if UNITY_EDITOR
using System;
using System.Collections.Generic;
using UnityEditor;
using UnityEditor.Callbacks;
using UnityEngine;
using UnityEngine.SceneManagement;
using Object = UnityEngine.Object;
public static class DestroyEditorMonobehaviourScripts
{
private static readonly Type[] typesToDeleteOnBuild = {
typeof(DT_Object),
typeof(DT_Environment),
typeof(DT_Sprite),
typeof(DT_Background),
typeof(EasyAnimate),
typeof(TestAnimate),
typeof(AnimatorPreviewer),
typeof(TerrainCreator),
typeof(ShapeMesh),
};
[PostProcessScene]
public static void DeleteObjects()
{
if (BuildPipeline.isBuildingPlayer)
{
foreach (var type in typesToDeleteOnBuild)
{
Debug.Log("Destroying all instances of " + type.Name + " on build!");
foreach (var obj in FindObjectsOfTypeAll(type, true))
{
Object.DestroyImmediate(obj);
}
}
}
}
/// Use this method to get all loaded objects of some type, including inactive objects.
/// This is an alternative to Resources.FindObjectsOfTypeAll (returns project assets, including prefabs), and GameObject.FindObjectsOfTypeAll (deprecated).
public static List<Component> FindObjectsOfTypeAll(Type _type, bool _findInactive = false)
{
var results = new List<Component>();
for (int i = 0; i < SceneManager.sceneCount; i++)
{
var s = SceneManager.GetSceneAt(i);
if (s.isLoaded)
{
var allGameObjects = s.GetRootGameObjects();
for (int j = 0; j < allGameObjects.Length; j++)
{
var go = allGameObjects[j];
results.AddRange(go.GetComponentsInChildren(_type, _findInactive));
}
}
}
return results;
}
}
#endif
This works quite well but I realized it’s still giving me some errors in the development build.
A script behaviour (probably EasyAnimate?) has a different serialization layout when loading. (Read 32 bytes but expected 328 bytes)
Did you #ifdef UNITY_EDITOR a section of your serialized properties in any of your scripts?
Specifically one related to a script called EasyAnimate, a Monobehaviour script attached on the effects I instantiate @ runtime.
EasyAnimate’s entire class is surrounded by a preprocessor directive #ifdef UNITY_EDITOR but then again so do all the other classes, yet those don’t give me any errors.
It seems that we’re only removing the scripts from all the scenes and not from potential prefabs that could also have those scripts on them? Am I understanding this correctly?
Yeah, that only removes those scripts from scenes.
There’s currently no way to post-process prefabs at builds, unless you do the hard work yourself to keep a backup of the non-processed prefab that you restore after the build. There was a thread on it just a week ago.
Note that for the script as it is now, you don’t have to iterate the scenes and remove objects. PostProcessScene is called once per scene, with that scene being the active scene when it’s running.
HideFlags maybe?
theComponent.hideFlags |= HideFlags.DontSaveInBuild;
Or the “EditorOnly” GameObject Tag?
An asset is marked with HideFlags.DontSave but is included in the build:
Asset: ‘Assets/Art/_FX/_FX_Skills/Player/Air/JumpUp_Start/FX_Smear.prefab’
Asset name: FX_Smear
(You are probably referencing internal Unity data in your build.)
UnityEditor.BuildPlayerWindow:BuildPlayerAndRun()
1st option doesn’t work
and the 2nd is for the entire gameObject, which I want to keep.