im getting the error Top-level statements must precede namespace and type declarations. hers my code

using UnityEngine;

public class WeaponSwap : MonoBehaviour{}

public int selectedWeapon =0;
{
// Start is called before the first frame update
void Start(){
SelectWeapon();
}

// Update is called once per frame
void Update(){

}
void SelectWeapon ()
{
int i = 0;
foreach (Transform weapon in transform)
{
if (i == selectedWeapon)
weapon.gameObject.SetActive(true);
else
weapon.gameObject.SetActive(false);
i++;
}
}
}

You can fix your own typing mistakes. Here’s how:

Remember: NOBODY here memorizes error codes. That’s not a thing. The error code is absolutely the least useful part of the error. It serves no purpose at all. Forget the error code. Put it out of your mind.

The complete error message contains everything you need to know to fix the error yourself.

The important parts of the error message are:

  • the description of the error itself (google this; you are NEVER the first one!)
  • the file it occurred in (critical!)
  • the line number and character position (the two numbers in parentheses)
  • also possibly useful is the stack trace (all the lines of text in the lower console window)

Always start with the FIRST error in the console window, as sometimes that error causes or compounds some or all of the subsequent errors. Often the error will be immediately prior to the indicated line, so make sure to check there as well.

Look in the documentation. Every API you attempt to use is probably documented somewhere. Are you using it correctly? Are you spelling it correctly?

All of that information is in the actual error message and you must pay attention to it. Learn how to identify it instantly so you don’t have to stop your progress and fiddle around with the forum.

In this case you have open bracket / close bracket at the end of your class declaration line, making ALL your code actually be outside of a class.

1 Like

Your first error is that you failed to use code tags here on the forum. The top sticky post in the scripting forum explains how to use code tags. Your second error is that you seem to don’t understand one of the most fundamental C# language elements: curly braces. Do you understand why they exist and what they are doing? What do you think this line does?

public class WeaponSwap : MonoBehaviour{}

Do you note the opening and closing braces at the end? Does that make any sense to you? You define a class that is derived from MonoBehaviour, but it’s body is empty. You know the body of a class is supposed to be inside those curly braces :slight_smile:

@Thecatking12 , you would do yourself a great service by taking a few small C# tutorials on the web. It’s an awesome language, and a few hours of instruction will pay you absolutely massive dividends in time saved in the future.

3 Likes