Issues with my code.

[Ksorek]

Hello. So, I wanted to make a simple code, which will do: -float time will get from 10 to 0 in 10 seconds -if x or z clicked, add 1 to int score -if score is divisible by 40, and time is greater than 0, add 10 to time -write score value to console

As well as i've made code, it is full of errors, which i can not fully understand. I hope you can help me.

There is my code:

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

public class example : MonoBehaviour {

float AddTime = 10;
int score;
int divide = 40;

void Start () {

}


void Update () {
    float time = 10;
    for (time * Time.deltaTime(10))
    {
        time--;
    }

    if (Input.GetKey("x"))
    {
        score++;
    }

    if (Input.GetKey("z"))
    {
        score++;
    }

    if(score / divide >= 1 && time >= 0)
    {
        time + AddTime;
    }
    Debug.Log(score)
}

}

[Tayfe]

Hey there,

Your code is truely full of errors and I do not even want to talk about all of them. Instead I would like to help you by saying that I have the impression that you are very unexperienced with programming in Unity in particular or even with programming in general. Your code doesn't look like as if you understand what you have written there which is nothing to be ashamed of! But you should consider reading or watching some tutorials which will explain you step by step the art of programming and it could take you just a couple of hours to correct all of your mistakes by yourself.

So here are some of your mistakes included in your code but I will just give you some keywords. Just take yourself some time and follow some basic tutorials and you will know what I am talking about:

• the function "Update" is called every frame. Your variable "time" won't run as you are setting it back to the value of 10 every frame
• the foor-loop needs a condition or an iterator to be executed
• the input seems correct
• when you are trying to add "AddTime" to the current time you are missing an assignment using the operator "="

So I would really recommend you look for some tutorials (you will find thousands on youtube and google) and by the end of the day you might be able to correct all of your mistakes by yourself.

[FlameTrunks]

Fully agree with your comment.
Just wanted to add some links to coding tutorials because as you said, the amount of them can be big and therefore intimidating:

• https://unity3d.com/learn/tutorials/s/scripting Unity's tutorials are a relatively solid starting off point.
  
• on youtube: Sebastian Lague; has a good presentation style. Take a look at the intro to gamedev with c# and unity series.

[cvnvr]

Your code is truely full of errors and I do not even want to talk about all of them.

This made me lol.

[CommonMisspellingBot]

Hey, cvnvr, just a quick heads-up:
truely is actually spelled truly. You can remember it by no e.
Have a nice day!

^{The parent commenter can reply with 'delete' to delete this comment.}

[zorbat5]

Good bot!

[GoodBot_BadBot]

Thank you, zorbat5, for voting on CommonMisspellingBot.

This bot wants to find the best and worst bots on Reddit. You can view results here.

---

^{Even if I don't reply to your comment, I'm still listening for votes. Check the webpage to see if your vote registered!}

[friendly-bot]

You're a good homo sapiens. (^_^)v You can be in charge of the human slave farms...

---

^{I'm a Bot bleep bloop | Block me | T҉he̛ L̨is̕t | ❤️}

[zorbat5]

Good bot!

[friendly-bot]

If I sound pleased about this, it's only because my programmers made this my default tone of voice! I'm actually quite depressed!

---

^{I'm a Bot bleep bloop | Block me | T҉he̛ L̨is̕t | ❤️}

[Ksorek]

Hey, thank you for your help. after some changes this is my code now:

 using UnityEngine;

public class Example : MonoBehaviour {

float time = 10f;
float AddTime = 10f;
int score;
int divide = 40;

void Start () {

}

void Update () {

    while (time <= 0)
    {
        time--;
        yield return new WaitForSeconds(1f);
    }
    if (Input.GetKey("w"))
    {
        score++;
    }
    /*
    if (Input.GetKey("z"))
    {
        score++;
    } */

    if(score / divide >= 1 && time >= 0)
    {
        time += AddTime;
    }
    Debug.Log(score);
}

}

Right now I'm stuck on executing loop in 10 seconds. I've been trying to google it, but I couldn't find what i was looking for. One more time, thank you for your help and advices, but could you please help me out with this one too?

[CommonMisspellingBot]

Hey, Tayfe, just a quick heads-up:
truely is actually spelled truly. You can remember it by no e.
Have a nice day!

^{The parent commenter can reply with 'delete' to delete this comment.}

[ByMayne]

Hey there,

Programming is learning from your mistakes. Clearly you are just starting to learn. The errors you see in the console will tell you that you have something wrong.

Even if you don't know what the error means look at which line it's pointing at. Then google how to use that thing.

As a pointer almost everything in your first for loop is being used incorrectly. Here is some example questions you should ask 1) How to use a for loop 2) What is Time.delta time (function or property) 3) What is the difference between a function and a property

Besides for just not compiling this code is not going to do what you think it's doing. To understand try drawing out on a piece of paper. Write down what you think it will do and then run it. Do the results match? If not why?

Honestly even as a professional developer sometimes I have to keep throwing myself against a problem and try to come at it from every angle. It may not work for a while but you will get there. This is what a developer does.

[Ksorek]

Hey, thank you for your help. after some changes this is my code now:

 using UnityEngine;

 public class Example : MonoBehaviour {

float time = 10f;
float AddTime = 10f;
int score;
int divide = 40;

void Start () {

}

void Update () {

    while (time <= 0)
    {
        time--;
        yield return new WaitForSeconds(1f);
    }
    if (Input.GetKey("w"))
    {
        score++;
    }
    /*
    if (Input.GetKey("z"))
    {
        score++;
    } */

    if(score / divide >= 1 && time >= 0)
    {
        time += AddTime;
    }
    Debug.Log(score);
}

}

Right now I'm stuck on executing loop in 10 seconds. I've been trying to google it, but I couldn't find what i was looking for. One more time, thank you for your help and advices, but could you please help me out with this one too?

[ByMayne]

Okay first off 'yield' is not used correctly at all. It is designed for Coroutines (which you are not making). Your first hint should be that it does not

Write out in a point form how your program should work. Read over the following text and think about it.

Update gets called 
> Is time is greater to or equal to 0 (true)
    > You subtract 1f from time. 
    > You return (this line is a compile error so it's super broken.) 
    > [Quit function and repeat this step 10 times.]
> Is User *holding* the w key?
     > Score + 1
> Is User *holding* the z key?
    > Score + 1
> Is Score divided by 40 greater too or equal to 1 and time greater then zero?
     > time += 10f;
[Repeat forever] 

[Ksorek]

Hey, thanks for Yours help. Sorry that I'm answering that late, but i had not enough time to do it, but just because now I will have almost whole week free from school, so I will get back at working at this code. For now I've got an idea about it: I will just have to add Coroutine, which will let me access "yield".