NXT PROBLEM

[naneczek]

You should have told me this from the start. I gave you the code, You didn't even look at it...

[HaWe]

no, that's why I told you to post your whole code- what it is all about - in code tags.
I guess you'd better did it

And now try to debug your code when you'll finally have your NXT again to work with!

[naneczek]

Code: Select all

task main() {
ClearScreen();
int y;
int i;
int MyArray[100];

            SetSensorLowspeed(IN_4);

                  for (i=0;i<=100;i++) {
                  y= SensorUS(IN_4);
                  MyArray[i]=y

                  NumOut(0, 56, y, DRAW_OPT_CLEAR);

                   }

                  for (i=0;i<=100;++i) {
                  y=MyArray[i];
                  if (i==99) ClearScreen();

                  PointOut(i, y/4, DRAW_OPT_NORMAL);

                  }

          while (true);

}

Ok, now can you help me with this code? How to modify it to achieve desirable effects?

[HaWe]

to make your code readable, please use:
for each { :
2 (or 3) extra spaces for indentation further to the right in the next line
for each } : 2 (or 3) spaces for indentation back to the left in the same line

Code: Select all

int main(int argc, char *argv[])
{
    ...
    while (x == y) {
        something();
        somethingelse();
 
        if (some_error) {
            /* the curly braces around this code block could be omitted */
            do_correct();
        } else
            continue_as_usual();
    }
 
    finalthing();
    ...
}

The K&R style, so named because it was used in Kernighan and Ritchie's book The C Programming Language, is commonly used in C. It is also used for C++, C#, and other curly brace programming languages.

have a look at this link: http://en.wikipedia.org/wiki/Indent_style#K.26R_style

[HaWe]

ok,
your code still has not got the correct intendations, so you can't actuylly check what each block does, where it begins and where it ends.

That's important for debugging.

so let's improve your intendations:
e.g., from the line below "task main()...
until the line for(i...) all intendations must have the same level,
because it's the main body of the task main.

then the body of the for- loop must have the same level all over,
and so on.

So try once more, maybe you'll find first indications why your code won't work.

[naneczek]

Code: Select all

task main()
{
    ClearScreen(); 
    int y;   
    int i;    
    SetSensorLowspeed(IN_4); 
   
    for (i=0;i<=100;++i)     
    {
        if (i>=100) {i=0; ClearScreen(); }    
        y = SensorUS(IN_4);    
        LineOut(i, y/4, DRAW_OPT_NORMAL);    
        NumOut(0,56, y, DRAW_OPT_CLEAR);    
    }
    while (true);    
}

Sth like this will do? I cannot check on any machine so I've got a request to check it. I want the measurements to be done in whole lines, not pixel by pixel. This whole line should be seen on the display, until the next measueremnt is done and when it is done it is depicted on the screen and so on.

[HaWe]

now think about what you are doing:

Code: Select all

lines 1-4 in the main body: initialize.
 
for loop: 
- you poll a sensor value and assign it to y.
- you make a line out every time for each i
- you make a num out for every i

when the for loop is done once, then the program terminates in a perpetual wait state (while(true) ).

is this the way you want it?

no.

what you want is:
first get 100 values in a for loop (0-99) and assign it to an array,
then print the whole array in a for loop (0-99) on the screen,

and then you repeat all that endlessly.

Try it!

[naneczek]

That's the case that I don't know how to do it. What should I put in the first array, how to do this loop, what kind of code should I implement? Could you help me out with this? or anyone? Or just show me the body of it with description.

[HaWe]

I already showed you the principle some posts before. Just approach the problem logically!

[naneczek]

Principle? I iwsh you would show me the code with description. So you won't help me?