next topic »

[Reply #75 on:]

In place of:-

Code:

ledBits.portMode(0, 0b0000000000000000); // set all pins on chip 0 to output

Try

Code:

for(int i=0; i<16; i++)ledBits.portMode(i,OUTPUT);

[Reply #76 on:]

portMode is actually 16 bits wide, so the original usage is OK.

The part where you go if(PadNote == 60) pattern |= 0x00; yada yada is later followed by if(PadNote == 60) pattern &= ~0x00; in the same unbroken chain of execution so OF COURSE it will never activate a pin.

[Reply #77 on:]

"in the same unbroken chain of execution so" yada yada...

if(PadNote == 60) pattern |= 0x00; yada yada is later followed by if(PadNote == 60) pattern &= ~0x00; //OF COURSE it will never activate a pin

New approach.....

instead of a big bunch of if statements

Code:

if ((PadNote[i] >= 60) && (PadNote[i] <= 75)) {
    mcp.digitalWrite(PadNote[i]-60, HIGH);
}

Code:

if ((PadNote[i] >= 60) && (PadNote[i] <= 75)) {
    mcp.digitalWrite(PadNote[i]-60, LOW);
}

lights an LED number 0 to number 15 if the PadNote is between 60 and 75, so if PadNote is 65 it will light LED number 5.

[Reply #78 on:]

That new approach may actually blink an LED, but it'll be so fast you probably won't see it. You need to think about when you want to turn the LED on, when you want to turn the LED off, and why those events can't occur in the exact same function right after one another.

Overall...you need to just sit back, take a deep breath, absorb everything you've learned so far, and THINK. Slowly. Draw stuff on a piece of paper, with a pencil. Right now I see a lot of typing, rapid fire posting, instant changes to code made on poorly understood suggestions. It looks a lot like letting a balloon go in a small room. It bounces off all the walls and makes a lot of noise, but nothing much happens.

And remember...if someone comes in here and just solves your entire problem, top to bottom, that means they don't like you. In this forum we usually try to point people in the right direction, so they can pick up the skillset to solve other problems.