Trouble doing a randomly generated backgrounds.

[nicholashallwork]

Hello everyone, i would like to preface that i am new to unity. I am starting to develop a game. The first step i wanted to take was to have backgrounds randomly put in. Already made and in the resources folder, directory being /resources/Backgrounds. I wrote a script that i will put on the bottom. I keep getting an error code. I have also attached the script to the Main Camera in the scene. The error code i keep getting is ‘The referenced script on this Behaviour (Game Object ‘BackgroundCamera’) is missing!’. What do i need to do to fix this. Also if my script looks bad or can be improved please let me know i would love all the constructive criticism i can get.

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

public class RandomFromList : MonoBehaviour
{

public List roomList;
public GameObject[ ] roomListArray;

void Start()
{
GenerateRoom();
}

void GenerateRoom()
{
roomListArray = Resources.LoadAll(“Background”);

roomList = roomListArray.ToList();

GameObject roomToBuild = roomList[Random.Range(0, roomList.Count)];
GameObject newRoom = Instantiate(roomToBuild, transform.position, Quaternion.identity) as GameObject;
}
}

[nicholashallwork]

I would also like to add that the backgrounds are .png files. The scene also only has the camera so far. I wanted to get this working as my first step.

[Waffleslug]

That error can show up if you move the script files to a different folder outside of Unity. When you do that I think Unity will pick up the script as something new and assign it a new hash which breaks any existing references to it.

Does the error go away if you remove the component and add it again?

[Brathnann]

You could use the array directly or, you should be able to just do

roomList = Resources.LoadAll(“Background”).ToList();

instead of creating an array and then a list from the array. But that isn’t your error. As @Waffleslug suggested. First try removing and adding the script again. If you still have an error, make sure you properly saved your script and you also don’t have console errors.

[nicholashallwork]

@Waffleslug

That error can show up if you move the script files to a different folder outside of Unity. When you do that I think Unity will pick up the script as something new and assign it a new hash which breaks any existing references to it.

Does the error go away if you remove the component and add it again?

That had fix the error, thank you a ton. Though i do now have a compiling error output being ‘UnityEditor.SceneView:ShowCompileErrorNotification()’.

[nicholashallwork]

@Brathnann

You could use the array directly or, you should be able to just do

roomList = Resources.LoadAll(“Background”).ToList();

instead of creating an array and then a list from the array. But that isn’t your error. As @Waffleslug suggested. First try removing and adding the script again. If you still have an error, make sure you properly saved your script and you also don’t have console errors.

You would use that in place of these lines correct? How does this optimize more also?

roomListArray = Resources.LoadAll(“Background”);

roomList = roomListArray.ToList();

[nicholashallwork]

Update: I have fixed the error and it ‘runs’ now. Though it doesn’t put the background on the screen. What am i missing or doing wrong in my code? It still has a warning symbol beneath the script saying there are compile errors, though it does not show up in console when running.

[Brathnann]

Right now you are creating an array and then converting it to a list. You now have an array and a list. Are you using both? If not, you should pick one and go with it. Either the array or the list then you don’t have one of them just sitting there serving no purpose. Think of it as creating two shopping list and then going to the store with both, but only using one of them.

What are you loading? What is a “background” gameobject that you’re populating your list with?

[nicholashallwork]

@Brathnann

Right now you are creating an array and then converting it to a list. You now have an array and a list. Are you using both? If not, you should pick one and go with it. Either the array or the list then you don’t have one of them just sitting there serving no purpose. Think of it as creating two shopping list and then going to the store with both, but only using one of them.

What are you loading? What is a “background” gameobject that you’re populating your list with?

Alright i understand. Background is a folder i have with three .png files to use as a background display.

[Brathnann]

Hmm…if they are images, I don’t think your setup will work. You’ll have to load them as textures or sprites. I’ve never done a resource.loadall for images.

I don’t believe they will work as gameobjects. Do you even have any values when you debug.log your list count? Normally for something like this I’d probably just have a public away where I drop the images into it and then randomly pick from that array to apply a background.

[nicholashallwork]

@Brathnann

Hmm…if they are images, I don’t think your setup will work. You’ll have to load them as textures or sprites. I’ve never done a resource.loadall for images.

I don’t believe they will work as gameobjects. Do you even have any values when you debug.log your list count? Normally for something like this I’d probably just have a public away where I drop the images into it and then randomly pick from that array to apply a background.

I apologize looking at it today i realize that the Texture Type is Sprite (2D and UI). I also keep getting the error script is missing after every time i run playmode. Deleting and copying and pasting works though it is getting a bit annoying doing it every time i hit play. I also do not get anything from the debug.log and still getting that compiling error. I am not sure what all i am doing wrong.

[nicholashallwork]

Here are some images of my errors and backgrounds if that gives anymore information.