Assets\cameraManager.cs(16,27): error CS1003: Syntax error, ',' expected plz help

Unity Engine
1

i’m a beginner to unity and i’ve made my own script for a remake of FNAF 1 (Security cams) but I keep getting a “Assets\cameraManager.cs(16,27): error CS1003: Syntax error, ‘,’ expected” I really need help, here’s the script:

public class cameraManager : MonoBehaviour
{
    public GameObject Camera_SecurityOffice;
    public GameObject Camera1A;
    public GameObject Camera1B;
    public GameObject Camera1C;
    public GameObject Camera2A;
    public GameObject Camera2B;
    public GameObject Camera3;
    public GameObject Camera4A;
    public GameObject Camera4B;
    public GameObject Camera5;
    public GameObject Camera6;
    public GameObject Camera7;

    public void Cameras_ON/OFF()
    {
        if (Cameras_ON)
        {
            Camera_SecurityOffice.SetActive(true);
            Camera1A.SetActive(false);
            Camera1B.SetActive(false);
            Camera1C.SetActive(false);
            Camera2A.SetActive(false);
            Camera2B.SetActive(false);
            Camera3.SetActive(false);
            Camera4A.SetActive(false);
            Camera4B.SetActive(false);
            Camera5.SetActive(false);
            Camera6.SetActive(false);
            Camera7.SetActive(false);
        }
        else
        {
            Camera_SecurityOffice.SetActive(false);
            Camera1A.SetActive(true);
            Camera1B.SetActive(false);
            Camera1C.SetActive(false);
            Camera2A.SetActive(false);
            Camera2B.SetActive(false);
            Camera3.SetActive(false);
            Camera4A.SetActive(false);
            Camera4B.SetActive(false);
            Camera5.SetActive(false);
            Camera6.SetActive(false);
            Camera7.SetActive(false);
        }

    }

    public void CameraActivate1A()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(true);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate1B()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(true);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate1C()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(true);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate2A()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(true);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate2B()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(true);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate3()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(true);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate4A()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(true);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate4B()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(true);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate5()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(true);
        Camera6.SetActive(false);
        Camera7.SetActive(false);
    }

    public void CameraActivate6()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(true);
        Camera7.SetActive(false);
    }

    public void CameraActivate7()
    {
        Camera_SecurityOffice.SetActive(false);
        Camera1A.SetActive(false);
        Camera1B.SetActive(false);
        Camera1C.SetActive(false);
        Camera2A.SetActive(false);
        Camera2B.SetActive(false);
        Camera3.SetActive(false);
        Camera4A.SetActive(false);
        Camera4B.SetActive(false);
        Camera5.SetActive(false);
        Camera6.SetActive(false);
        Camera7.SetActive(true);
    }
}
2

public void Cameras_ON/OFF()
You cannot have a / in a method name. Get rid of the / and fix any calls to this method so they match the new method name.

3

And to make your code a little more readable make a method which deactivates all cams and call that and then activate the one you need.

    public void CameraActivate7()
    {
         DeactivateAllCams();
         Camera7.SetActive(true);
    }

The reason is, when you add, remove or rename a camera you have MANY places to change. This is prone to error and leads to bugs. So try to never repeat/copy paste your code.

4

It’s not that you “keep getting” but rather that you have typos. You cannot have typos when coding.

The complete error message contains everything you need to know to fix your typos by 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.

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.

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.

As an aside, you should consider learning about collections (arrays, lists, etc.) in C#. It will make your above wall of code into about 5 lines total with a far smaller chance of errors.

5

thank you so much your a legend!! :slight_smile: