Код: Выделить всё
int fnSelect(int *arr)
{
if(arr[0] == arr[1])
{
arr[2]=arr[0];
return arr[0];
}
else
{
if(arr[0] == arr[2])
{
arr[1]=arr[0];
return arr[0];
}
else
{
if(arr[1] == arr[2])
{
arr[0]=arr[1];
return arr[1];
}
}
}
return -1;// ошибка
}
ut1wpr писал(а):Так просто?
Вы пробовали? У вас работает? В IAR?
Позвольте усомниться в вашем совете.
urry писал(а):Код: Выделить всё
int fnSelect(int *arr)
{
if(arr[0] == arr[1])
{
arr[2]=arr[0];
return arr[0];
}
else
{
if(arr[0] == arr[2])
{
arr[1]=arr[0];
return arr[0];
}
else
{
if(arr[1] == arr[2])
{
arr[0]=arr[1];
return arr[1];
}
}
}
return -1;// ошибка
}
Код: Выделить всё
#define FALSE 0
#define TRUE 1
unsigned char fnSelect(int *arr)
{
if(arr[0] == arr[1])
{
arr[2]=arr[0];
return TRUE;
}
else
{
if(arr[0] == arr[2])
{
arr[1]=arr[0];
return TRUE;
}
else
{
if(arr[1] == arr[2])
{
arr[0]=arr[1];
return TRUE;
}
}
}
return FALSE;// ошибка
}
Аlex писал(а):Что значит "исправить" ? И как понять "ошибка" ?
juzik писал(а):urry писал(а):Думаю, это немного сократит используемую память. Значение можно будет получить из любой ячейки массива arr.
Имхо этот код не так уж много места будет отъедать. Я к тому что, если оптимизировать, то не отдельный участок кода, а просмотреть весь код на предмет уменьшения памяти.
) Просто реально счет идет на байты.Спасибо.Так то уже все нормально, но дело уже принципа 
А если во все 3 запишется некорректно ?Next13 писал(а):Т.е. если произошел сбой и в одну ячейку информация была записана не корректно ее нужно исправить.
Аlex писал(а):А если во все 3 запишется некорректно ?Next13 писал(а):Т.е. если произошел сбой и в одну ячейку информация была записана не корректно ее нужно исправить.
)
Проверочный вопрос:Что делают?Next13 писал(а):Эти переменные храняТься в EEPROM.
juzik писал(а):Думаю, это немного сократит используемую память.
vitalik_1984 писал(а):Каким образом? В обоих случаях возвращается учар.
AI_Disable писал(а):juzik, оптимизатор уже довольно хорошо умеет чистить всё лишнее и видеть код наперёд, чтоб 10 раз не загружать одни и те же переменные из памяти в регистры и т.п.. Так что тут сложно сказать, что будет лучше.
AI_Disable писал(а):Да и вообще, неужели экономия в 1-2 байта это сегодня так существенно?
juzik писал(а):Имхо этот код не так уж много места будет отъедать. Я к тому что, если оптимизировать, то не отдельный участок кода, а просмотреть весь код на предмет уменьшения памяти.
Device: atmega8
Program: 290 bytes (3.5% Full)
(.text + .data + .bootloader)
Data: 6 bytes (0.6% Full)
(.data + .bss + .noinit)
Device: atmega8
Program: 318 bytes (3.9% Full)
(.text + .data + .bootloader)
Data: 6 bytes (0.6% Full)
(.data + .bss + .noinit)
Ну взвратился "0", а дальше что ? Опять анализ и гадание на коф. гуще : в каких же переменных правильные значения...juzik писал(а):Во втором варианте вернуть или 0 или 1.
По условиям было не только экономия местапо условиям места мало вот я и поставил экономию места
Аlex писал(а):Ну взвратился "0", а дальше что ? Опять анализ и гадание на коф. гуще : в каких же переменных правильные значения...
ТС'у нужно проанализировать переменные и отсеять/изменить ошибочные, ну или возвратить правильное значение, как я понял...
Тогда да, пойдёт. Не правильно понял, ссори. Думал что возвращает только 2 значения - есть/ нет ошибок.vitalik_1984 писал(а):Дак сама функция уже все выравнивает.
arr[1]=arr[0];
return arr[0];
Аlex писал(а):Можно попробовать не передавать указатель, а сделать массив глобальным. Заполнили массив и вызвал ф-ию. Возможно меньше места займёт и побыстрее будет.