Wayback Machinekoobas.hobune.stream
May JUN Jul
Previous capture 13 Next capture
2021 2022 2023
1 capture
13 Jun 22 - 13 Jun 22
sparklines
Close Help
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
  • Asset Store
  • Get Unity

UNITY ACCOUNT

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account
  • Blog
  • Forums
  • Answers
  • Evangelists
  • User Groups
  • Beta Program
  • Advisory Panel

Navigation

  • Home
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
    • Blog
    • Forums
    • Answers
    • Evangelists
    • User Groups
    • Beta Program
    • Advisory Panel

Unity account

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account

Language

  • Chinese
  • Spanish
  • Japanese
  • Korean
  • Portuguese
  • Ask a question
  • Spaces
    • Default
    • Help Room
    • META
    • Moderators
    • Topics
    • Questions
    • Users
    • Badges
  • Home /
avatar image
0
Question by suicidalcrow · May 02, 2013 at 02:19 AM · c#errornamespace

A namespace can only contain types and namespace declarations

Hey, guys. So I'm fairly new to Unity and to the C# language and am getting "error CS0116: A namespace can only contain types and namespace declarations" on line 63,14 which is this piece of code:

     void Update () {
         if(Input.GetKeyDown(KeyCode.Tab))
         {
             TargetEnemy();
         }
     }

I'm sure it's a simple solution, I just can't seem to figure it out! Here's the full code, if it's needed?

 enter code hereusing UnityEngine;
 using System.Collections;
 using System.Collections.Generic;
 
 public class Targetting : MonoBehaviour {
 public List<Transform> targets;
 private Transform myTransform;
 public Transform selectedTarget;
 
 // Use this for initialization
     void Start () {
     targets = new List<Transform>();
     selectedTarget = null;
     AddAllEnemies();
     myTransform = transform;
     }
     public void AddAllEnemies()
     {
     GameObject[] go = GameObject.FindGameObjectsWithTag("Enemy");
         foreach(GameObject enemy in go)
             AddTarget(enemy.transform);
     }
     public void AddTarget(Transform enemy) {
         targets.Add(enemy);
     }
     private void SortTargetsByDistance()
     {
     targets.Sort(delegate(Transform t1, Transform t2){
         return(Vector3.Distance(t1.position, myTransform.position).CompareTo(Vector3.Distance(t2.position, myTransform.position)));
         });
     }
 
     private void TargetEnemy()
     {
     if(selectedTarget == null){
     SortTargetsByDistance();
     selectedTarget = targets[0];
     }
     else{
         int index = targets.IndexOf(selectedTarget);
             if(index < targets.Count - 1){
             index++;
         }
     else{
         index = 0;
         }
     DeselectTarget();
     selectedTarget = targets[index];
     SelectTarget();
     }
 }
     private void SelectTarget(){
     selectedTarget.renderer.material.color = Color.red;
 }
     
     private void DeselectTarget(){
         selectedTarget.renderer.material.color = Color.blue;
     selectedTarget = null;
     }
 }
 // Update is called once per frame
 
     void Update () {
         if(Input.GetKeyDown(KeyCode.Tab))
         {
             TargetEnemy();
         }
 }
 

Thanks in advance.

Comment
Add comment
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

2 Replies

· Add your reply
  • Sort: 
avatar image
1

Answer by Dave-Carlile · May 02, 2013 at 02:24 AM

  1. I assume your actual code doesn't have "enter code here" as the first few characters?

  2. Looks like on line 51 you have an extra } that closes out the class, and then you have more code after that. This is most likely the cause of the error, but I'm not 100% sure because the code is hard to follow because of how you're indenting things.

  3. Do some research on a good, standard way to do indentation in your code. That will make it much easier to spot these sorts of things.

Comment
Add comment · Show 1 · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users
avatar image suicidalcrow · May 02, 2013 at 02:35 AM 0
Share
  1. Aha, no of course not, didn't realise that stayed in there when adding the script!

  2. Hmm, yeah I thought that at first but when I got rid of it a new error appeared for line 52?

  3. Yeah, sorry about the way it's structured. I'll do some research on good ways to do indentation in my code!

avatar image
1

Answer by Eric5h5 · May 02, 2013 at 02:25 AM

Your Update function is outside the class declaration (and there are too many braces anyway). It's an extremely good idea to be consistent and not sloppy with spacing/layout, so this sort of thing can be seen at a glance.

Comment
Add comment · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

Your answer

Hint: You can notify a user about this post by typing @username

Up to 2 attachments (including images) can be used with a maximum of 524.3 kB each and 1.0 MB total.

Follow this Question

Answers Answers and Comments

14 People are following this question.

avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image

Related Questions

The type or namespace name 'StateMachine' could not be found (are you missing a using directive or an assembly reference?) 1 Answer

error CS0246: The type or namespace name `List' could not be found. 1 Answer

C# error - A namespace can only types and namespace declarations 2 Answers

Multiple Cars not working 1 Answer

UnityEngine.dll doesn't fix namespace error? 1 Answer


Enterprise
Social Q&A

Social
Subscribe on YouTube social-youtube Follow on LinkedIn social-linkedin Follow on Twitter social-twitter Follow on Facebook social-facebook Follow on Instagram social-instagram

Footer

  • Purchase
    • Products
    • Subscription
    • Asset Store
    • Unity Gear
    • Resellers
  • Education
    • Students
    • Educators
    • Certification
    • Learn
    • Center of Excellence
  • Download
    • Unity
    • Beta Program
  • Unity Labs
    • Labs
    • Publications
  • Resources
    • Learn platform
    • Community
    • Documentation
    • Unity QA
    • FAQ
    • Services Status
    • Connect
  • About Unity
    • About Us
    • Blog
    • Events
    • Careers
    • Contact
    • Press
    • Partners
    • Affiliates
    • Security
Copyright © 2020 Unity Technologies
  • Legal
  • Privacy Policy
  • Cookies
  • Do Not Sell My Personal Information
  • Cookies Settings
"Unity", Unity logos, and other Unity trademarks are trademarks or registered trademarks of Unity Technologies or its affiliates in the U.S. and elsewhere (more info here). Other names or brands are trademarks of their respective owners.
  • Anonymous
  • Sign in
  • Create
  • Ask a question
  • Spaces
  • Default
  • Help Room
  • META
  • Moderators
  • Explore
  • Topics
  • Questions
  • Users
  • Badges