Error with the term else

Question

0.00/5 (No votes)

See more:

Hello, My language is Spanish but I will try to speak in English so that they understand me.

I am learning to use unity through youtube videos and in the video that I see he writes a code that uses if, else if and else but, I put exactly the same as in the video and it does not let me, I get this "Assets \ Scripts \ PlayerMove.cs (34,10): error CS1525: Invalid expression term 'else' "

I have tried watching other videos but they all do the same and it does not work for me, can someone help me?


I leave the code for you to see the problem.

Unity 2020
Visual studio 2017
Win10

according to unity the error is in line 34,10

If someone helps me I would really appreciate it

What I have tried:

<pre lang="C#">

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

public class PlayerMove : MonoBehaviour
{

    //Variables
    public float runSpeed = 2;
    public float JupmSpeed = 3;

    Rigidbody2D rb2d;




    // Start is called before the first frame update
    void Start()
    {
        rb2d = GetComponent<Rigidbody2D>();
    }

    // Update is called once per frame
    private void FixedUpdate()
    {
        if (Input.GetKey("d")) || Input.GetKey("right"))
        {
            rb2d.velocity = new Vector2(runSpeed, rb2d.velocity.y);
        }

        else if (Input.GetKey("a")) || Input.GetKey("left"))
        {
            rb2d.velocity = new Vector2(-runSpeed, rb2d.velocity.y);
        }
        else
        {
            rb2d.velocity = new Vector2(0, rb2d.velocity.y);
        }

    }

Posted 22-May-21 14:14pm

Member 15211818

Updated 22-May-21 22:16pm

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

The Other John Ingram · Answer 1 · 2021-05-22T16:42:00

Solution 1

if (Input.GetKey("d")) || Input.GetKey("right"))

should be

if (Input.GetKey("d") || Input.GetKey("right"))

Posted 22-May-21 16:42pm

The Other John Ingram

Comments

BillWoodruff 23-May-21 0:49am

+5

George Swan · Answer 2 · 2021-05-22T22:16:00

Solution 2

As has already been stated, you have a syntax error in your nested if then else statements. My recommendation is to avoid such nested statements as they are error-prone and convoluted. Simply call Input.GetKey once and then call a method that uses a switch statement to return the vector. Something along these lines.

C#

private Vector2 ChooseVector(char key)
     {

         return key switch
         {
            ('d') or ('r') => new Vector2(runSpeed, rb2d.velocity.y),
            ('a') or ('l')  => new Vector2(-runSpeed, rb2d.velocity.y),
             _ =>  new Vector2(0, rb2d.velocity.y)
         };
     }

Posted 22-May-21 22:16pm

George Swan

Comments

Richard Deeming 24-May-21 6:08am

NB: The "switch statement" syntax is only supported in C# 8 or later. If you're targeting .NET Framework, or an earlier version of .NET Core, you will need to manually set the <LangVersion> in your project file to enable support. It's not officially supported in anything other than .NET Core 3.x or later.

Does C# 8 support the .NET Framework? - Stack Overflow[^]

I also doubt it would work in VS2017, which is what the OP is using.

And I believe the or pattern is part of C# 9. Again, you can enable it in VS2019 by manually editing your project file, but it's not officially supported, so many C# 9 features won't work.

George Swan 24-May-21 8:31am

You are quite correct, Richard. The solution will compile with Net 5 and the default language version C#9.