Arduino.ru

Инвертировать условия, по срабатыванию делать return. Будет покрасивее и уменьшится шанс запутаться в дереве.

За парсер зачитай чонить

А в чём проблема-то? Ифы - как ифы. В строке №14 else перед иф не хватает, а так, код как код. 

Хочется в одну строчку? Тогда сюда, только потом не жалуйтесь :-)

Для одной команды regexp это наверное слишком, а так надо попробовать :)
Да как бы проблем нет, даже работает все :)

Мне почему-то бросилась в глаза такая вещь. С одной стороны проверок накручено много - например проверяется наличие слэша, осбого смысла не несущего. А с другой стороны - читаются символы по номеру в строке без проверки длины этой самой строки. Чего там этот charAt(x) вернёт?

Ноль.

Если r-value, то просто ноль, а если l-value, то ссылку на место, в котором уже есть ноль и куда можно что-то записать.

*чтобы не плодить темы уже спрошу тут*

Здравствуйте.
такая ситуация - тоже много if else if (но не вложенных)

if (flag == 2) {//заполняем массив}
else if (flag == 3) {//по другому заполняем массив}
else if (flag == 4) {//по другому заполняем массив}
и тд
в конце 
Serial.print(array);

но самый частый случай это когда flag==1 и для этого случая мы ничего не делаем с массивом а просто его печатаем. в данной реализации мы должны проверить все if и только потом распечатать самый частый случай.

если сделать так

if (flag == 1) {}
else if (flag == 2) {//заполняем массив}
else if (flag == 3) {//по другому заполняем массив}
else if (flag == 4) {//по другому заполняем массив}
и тд
в конце 
Serial.print(array);

то самый частый случай быстрее отрабатывает но смущает такой пустой if. как делают профи в такой ситуации ? или это нормально такая пустая заглушка если хотим побыстрее ?

Можно и так, но в вашем случае, когда в if-ах нет условий посложнее, удобнее пользоваться switch..case - проще читается.

switch(flag) {
  case 2:
    // fill array for case 2
    break;
  case 3:
    // fill array for case 3
    break;
  default:
    // here case 1
    break;
}

Serial.println(array);

if (flag != 1) {
 if (flag == 2) {//заполняем массив}
 else if (flag == 3) {//по другому заполняем массив}
 else if (flag == 4) {//по другому заполняем массив}
}
Serial.print(array);

negavoid2 ну так тоже все пробегут пока до дефолта дойдём, хотя надо на асме посмотреть что получается...

вот как b707 получается то что надо

  switch (key) {
    case KEY_CODE1:      on(PIN_R1); break;
    case KEY_CODE1_LONG: on(PIN_R0); break;
    case KEY_CODE2:      on(PIN_R2); break;
    case KEY_CODE3:      on(PIN_R3); break;
    case KEY_CODE4:      on(PIN_R4); break;
    case KEY_CODE4_LONG: on(PIN_R5); break;
  }

еще можно так :

typedef void (*Func_t)(char *arr , uint16_t len);

void printArray(char* arr, uint16_t len)
{
  serial.print(arr , len);
}

void modifyArray_1(char* arr, uint16_t len)
{
   for(uint16_t i = 0 ;i < len ;i++)
   {
       arr[i] = i ;
   }
}

void modifyArray_2(char* arr, uint16_t len)
{
   for(uint16_t i = 0 ;i < len ;i++)
   {
       arr[i] = i * i ;
   }
}

char dataArray[5] = {1 ,2, 3, 4, 5};


Func_t arraysFunc[3] = {printArray , modifyArray_1 , modifyArray_2} ; 


if(arraysFunc[flag])
     arraysFunc[flag](dataArray , sizeof(dataArray));

трудно выстрелить себе в ногу стотонным танком... но у некоторых получается

Что вы тут за задачи решаете, что свич нужно в asm-listingе смотреть? Прямо страшно становится...

Очевидно, параноидальную экономию микросекунд :)

negavoid2 еще раз посмотрел неправильно у вас. нужнобыло это

switch(flag) {
  case 2:
    // fill array for case 2
    break;
  case 3:
    // fill array for case 3
    break;}
// here case 1
Serial.println(array);

в асме не микрос хотел увидеть :) а разницу в коде. посмотрел разницы нет. он также пробегает все условия.  а хотелось проскочить все лишние условия. как мой второй пример или пример b707

Тяжелый случай.(

Боджечки )) ну поставьте первым case 1:break; хотя раз неправильно, значит неправильно :)))))

это ж алекс...

https://www.youtube.com/watch?v=aw10753OJyg

Надо, alexbmd, надо! (с)