Как сравнить биты памяти в C ++? - PullRequest
Новая
       12

Как сравнить биты памяти в C ++?

1 голос
/ 24 декабря 2010

Мне нужна помощь с функцией сравнения битов памяти.

Я купил здесь светодиодную матрицу с четырьмя чипами HT1632C и использую ее на моем Arduino Mega2560 .

Нет кода, доступного для этого чипсета (он отличается от HT1632), и я пишу самостоятельно. У меня есть функция графика, которая получает координаты X, Y и цвет, и этот пиксель включается. Только это работает отлично.

Но мне нужно больше производительности на моем дисплее, поэтому я попытался создать переменную shadowRam, которая будет «копией» памяти моего устройства. Перед тем, как я что-то показываю на экране, он проверяет shadowRam, чтобы увидеть, действительно ли необходимо изменить этот пиксель. Когда я включил это (getShadowRam) в функции графика, на моем дисплее было несколько, просто НЕКОТОРЫЕ (например, 3 или 4 на всем дисплее) призрачных пикселей (пикселей, которые не должны быть включены).

Если я просто прокомментирую prev_color if в моей функции сюжета, то она отлично работает.

Кроме того, я очищаю свой массив shadowRam, устанавливая всю матрицу в ноль.

Переменные: 

#define BLACK  0
#define GREEN  1
#define RED    2
#define ORANGE 3
#define CHIP_MAX 8
byte shadowRam[63][CHIP_MAX-1] = {0};

getShadowRam функция: 

byte HT1632C::getShadowRam(byte x, byte y) {
    byte addr, bitval, nChip;

    if (x>=32) {
        nChip = 3 + x/16 + (y>7?2:0);
    } else {
        nChip = 1 + x/16 + (y>7?2:0);
    }

    bitval = 8>>(y&3);

    x = x % 16;
    y = y % 8;
    addr = (x<<1) + (y>>2);

    if ((shadowRam[addr][nChip-1] & bitval) && (shadowRam[addr+32][nChip-1] & bitval)) {
      return ORANGE;
    } else if (shadowRam[addr][nChip-1] & bitval) {
        return GREEN;
    } else if (shadowRam[addr+32][nChip-1] & bitval) {
        return RED;
    } else {
        return BLACK;
    }
}

Функция графика: 

void HT1632C::plot (int x, int y, int color)
{
    if (x<0 || x>X_MAX || y<0 || y>Y_MAX)
        return;

    if (color != BLACK && color != GREEN && color != RED && color != ORANGE)
        return;

    char addr, bitval;
    byte nChip;

    byte prev_color = HT1632C::getShadowRam(x,y);

    bitval = 8>>(y&3);

    if (x>=32) {
        nChip = 3 + x/16 + (y>7?2:0);
    } else {
        nChip = 1 + x/16 + (y>7?2:0);
    }

    x = x % 16;
    y = y % 8;
    addr = (x<<1) + (y>>2);

    switch(color) {
        case BLACK:
            if (prev_color != BLACK) { // compare with memory to only set if pixel is other color
                // clear the bit in both planes;
                shadowRam[addr][nChip-1] &= ~bitval;
                HT1632C::sendData(nChip, addr, shadowRam[addr][nChip-1]);
                shadowRam[addr+32][nChip-1] &= ~bitval;
                HT1632C::sendData(nChip, addr+32, shadowRam[addr+32][nChip-1]);
            }
            break;

        case GREEN:
            if (prev_color != GREEN) { // compare with memory to only set if pixel is other color
             // set the bit in the green plane and clear the bit in the red plane;
             shadowRam[addr][nChip-1] |= bitval;
             HT1632C::sendData(nChip, addr, shadowRam[addr][nChip-1]);
             shadowRam[addr+32][nChip-1] &= ~bitval;
             HT1632C::sendData(nChip, addr+32, shadowRam[addr+32][nChip-1]);
            }
            break;

        case RED:
            if (prev_color != RED) { // compare with memory to only set if pixel is other color
                // clear the bit in green plane and set the bit in the red plane;
                shadowRam[addr][nChip-1] &= ~bitval;
                HT1632C::sendData(nChip, addr, shadowRam[addr][nChip-1]);
                shadowRam[addr+32][nChip-1] |= bitval;
                HT1632C::sendData(nChip, addr+32, shadowRam[addr+32][nChip-1]);
            }
            break;

        case ORANGE:
            if (prev_color != ORANGE) { // compare with memory to only set if pixel is other color
                // set the bit in both the green and red planes;
                shadowRam[addr][nChip-1] |= bitval;
                HT1632C::sendData(nChip, addr, shadowRam[addr][nChip-1]);
                shadowRam[addr+32][nChip-1] |= bitval;
                HT1632C::sendData(nChip, addr+32, shadowRam[addr+32][nChip-1]);
            }
            break;
    }
}

Если это поможет: Таблица платы, которую я использую. На странице 7 отображается отображение памяти, которое я использую.

Кроме того, у меня видео дисплея работает.

Ответы [ 2 ]

2 голосов
/ 24 декабря 2010

Это не настоящий ответ, но я думаю, что это может быть шагом к выяснению этого. Поскольку существует так много дублирования кода и запутанного условного кода, вы должны начать с рефакторинга. Тогда будет намного легче понять алгоритм. Я попробовал его, но не обещаю, что он не будет содержать ошибок.

Избавьтесь от getShadowRam и измените график так, чтобы он выглядел так: 

void HT1632C::plot (int x, int y, byte color)
{
  if (x < 0 || x > X_MAX || y < 0 || y > Y_MAX)
    return;

  if (color != BLACK && color != GREEN && color != RED && color != ORANGE)
    return;

  // using local struct to allow local function definitions
  struct shadowRamAccessor {
    shadowRamAccessor(byte x, byte y) {
      nChip = (x >= 32 ? 3 : 1)
        + x / 16
        + (y > 7 ? 2 : 0);
      bitval = 8 >> (y & 3);
      addr = ((x % 16) << 1) + ((y % 8) >> 2);
      highAddr = addr + 32;
    }

    byte& getShadowRam(byte addr) {
      return shadowRam[addr][nChip-1];
    }

    byte getPreviousColor() {
      byte greenComponent = getShadowRam(addr) & bitval ? GREEN : BLACK;
      byte redComponent = getShadowRam(highAddr) & bitval ? RED : BLACK;
      return greenComponent | redComponent;
    }

    void setValue(byte newColor) {
      byte prev_color = getPreviousColor();
      if(newColor != prev_color)
        setValue(newColor & GREEN, newColor & RED);
    }

    void setValue(bool greenBit, bool redBit)
    {
      HT1632C::sendData(nChip, addr,
        greenBit
          ? getShadowRam(addr) |= bitval
          : getShadowRam(addr) &= !bitval
        );
      HT1632C::sendData(nChip, highAddr,
        redBit
          ? getShadowRam(highAddr) |= bitval
          : getShadowRam(highAddr) &= ~bitval
        );
    }

    byte nChip, bitval, addr, highAddr;
  };

  shadowRamAccessor(x, y).setValue(color);
}
1 голос
/ 24 декабря 2010

Хм .. Возможно, я что-то здесь упускаю, но почему бы вам не изменить этот большой переключатель на: 


if(color != prev_color)
{
    shadowRam[addr][nChip-1] |= bitval;
    HT1632C::sendData(nChip, addr, shadowRam[addr][nChip-1]);
    shadowRam[addr+32][nChip-1] &= ~bitval;
    HT1632C::sendData(nChip, addr+32, shadowRam[addr+32][nChip-1]);
}
...