why is the error Assets\PlayerController.cs(26,1): error CS0106: The modifier 'private' i-CodePudding

i was making a game in unity and i got this error: Assets\PlayerController.cs(26,1): error CS0106: The modifier 'private' is not valid for this item

and this is the code:

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

public float speed;
public float jumpHeight;
public Rigidbody rigid;

private Vector3 direction;




void Start()
{

}



void Update()
{
 direction = new Vector3(Input.GetAxis("Horizontal"), 0, Input.GetAxis("Vertical"));
 if(Input.GetButtonDown("Jump"))
{
 rigid.AddForce(Vector3.up * JumpHeight,ForceMode.VelocityChange);
}
}

 private void FixedUpdate()
 {
 rigid.MovePosition(transform.position   direction * speed * Time.deltaTime);
 }

i changed private to public and the error still shown pls help

CodePudding user response：

You need to wrap your code in a namespace and a class.

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

namespace Playground
{
    public class PlayerController
    {
        public float speed;
        public float jumpHeight;
        public Rigidbody rigid;

        private Vector3 direction;

        void Start()
        {

        }

        void Update()
        {
            direction = new Vector3(Input.GetAxis("Horizontal"), 0, Input.GetAxis("Vertical"));
            if (Input.GetButtonDown("Jump"))
            {
                rigid.AddForce(Vector3.up * JumpHeight, ForceMode.VelocityChange);
            }
        }

        private void FixedUpdate()
        {
            rigid.MovePosition(transform.position   direction * speed * Time.deltaTime);
        }
    }
}

CodePudding user response：

As already answered by Matthew M., I would like to add a few more information and points to it.

First, all the code has to be wrapped in a public class with the name of the script in which the current code is being written. And for the methods Start(), Update(), and FixedUpdate(), these are inherited from Monobehaviour.

Wrapping in a namespace is optional, its generally a user preference

public class PlayerController : Monobehaviour
{   
    public float speed;
    public float jumpHeight;
    public Rigidbody rigid; 

    private Vector3 direction;

    void Start()
    {

    }

    void Update()
    {
        direction = new Vector3(Input.GetAxis("Horizontal"), 0, 
        Input.GetAxis("Vertical"));
        if(Input.GetButtonDown("Jump"))
        {
             rigid.AddForce(Vector3.up * JumpHeight,ForceMode.VelocityChange);
        }
    }

    private void FixedUpdate()
    {
        rigid.MovePosition(transform.position   direction * speed * Time.deltaTime);
    }
}