Проблема сортировки вставки Arduino со структурным массивом - PullRequest
0 голосов
/ 31 марта 2020

Вот структура:

struct RGB {
  byte r;
  byte g;
  byte b;
  byte v;
};

А вот и функция сортировки:

void insertion_sort(RGB arr[], size_t capacity) {
  RGB temp;
  size_t j;
  for(size_t i = 1; i < capacity; i++) {
    temp = arr[i];
    j = i - 1;
    while(j >= 0 && arr[j].v > temp.v) {
      arr[j+1] = arr[j];
      j--;
    }
    arr[j+1] = temp;
  }
}

элемент v структуры RGB - это позиция объекта в массиве до скремблирования ( и, надеюсь, где это заканчивается после сортировки). Вот вывод при печати элемента v каждого элемента массива:

165 171 53 164 171 13 13 167 156 168 163 14 15 16 17 18 19 20 20 156 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 142 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 

Я получаю один и тот же вывод каждый раз.

В массиве 150 элементов, и все кажется штраф после 21. Проблема в том, что элементы до этого являются неправильными и также состоят из значений, превышающих 149. В настоящее время члены RGB b и g имеют все 0 во всем массиве, а r - это значения, расположенные на равном расстоянии от 0 -254.

Следует отметить, что точно такая же функция сортировки работает, когда я воссоздаю все в C ++, и даже работает для сортировки массива целых в Arduino. Единственная проблема - сортировка массива структур RGB, что, похоже, вызывает проблемы с памятью.

Ответы [ 2 ]

0 голосов
/ 01 апреля 2020

Проблема была из-за типа данных size_t, который не имеет знака и не может стать значением <0, когда предполагалось. Спасибо @ JohnFilleau </p>

0 голосов
/ 31 марта 2020

Ваша проблема в определении j.
Ваши варианты:

  • Последовательный вывод в критических строках и анализ последовательного монитора -> Arduinos способ отладки
  • с помощью одна из ста протестированных функций сортировки (например, здесь для обмена стека), например, быстрая сортировка и т. д. c.
  • с использованием std :: sort

Для последнего варианта вам необходимо: :
Лучшее решение - использовать стандартную библиотечную функцию C ++ std :: sort. Эта функция является типобезопасной и не требует, чтобы вы приводили что-либо к void * и обратно, и вам не нужно вручную вводить размеры массива и его элементов.

 #include < Arduino_Helpers.h\>
 #include < AH/STL/algorithm\>
 #include < AH/STL/iterator\>

 auto cmpfunc = [](RGB A, RGB B) { return A.value < B.value; }; 

// or if you do not like this notation use
 bool cmpfunc(RGB A, RGB B) {
    return A.value < B.value;
 }

std::sort(std::begin(RGB arr[]), std::end(RGB arr[]), cmpfunc); // Dummy for a RGB arr[] containing all your values
  - 
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...