Arduino.ru

Да что вы на меня напали то? Я написал чтобы критиковали код, а не меня

Петрович уже с языка снял :(

Критикуйте код, а не меня

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

согласен. Просто имел в виду, что это такая очевидная вещь, что может быть просто опечаткой... Но ответ ТС показывает. что для него - не очевидная.

Я тут в соседней теме не мог собрать long из байт, потому что забыл поставить преобразование. Тоже у гивера спросил как это делается?

Пока нет, потому что в МК иначе чем в прикладных программах

ну вы же там ответ получили? и никто вас не троллил.

А здесь вас троллят, потому что ведетесь. Реагируйте как взрослый. а не как 15-летний подросток

Кстати, тогда гиверу на это тоже Владимир указал (см. #3).

Я никогда не сидел на форумах, вживую все проиходит иначе

И за это спасибо. Надеюсь получить советы по поводу других идей

А какие, кстати, у Вас другие идеи? Наливайте :)

Нет, продолжайте. Вы даете очень дельные советы

А этой же теме или другой? У меня есть реализация класса настроект в EEPROM (надеюсь не как у гивера). Основной функционал выполняет, но хочется сделать лучше

думаю, лучше в этой.

если обсуждение пойдет не так - проще будет одну ветку чистить, а не две :)

:-)))

Ложи своими классами прямо здесь.

#define minutes(x) (x*60000ul)
#define seconds(x) (x*1000ul)

enum params
{
	is_empty = 0,
	pc_shutdown_timeout,
	batt_critical
};


class Settings
{
public:
	Settings();

	void setParam(uint par, ul val);
	ul getParam(params par);

	void resetSettings();

private:
	void initDefault();
};

Settings::Settings()
{
	bool isEmpty = getParam(is_empty);
 	if (isEmpty)
		initDefault();
}

void Settings::setParam(uint par, ul val)
{
#ifdef DEBUG
	Serial.println("EEPROM write val: " + String(val));
#endif

	const int addr = par * 4;

	byte value[4];
	value[0] = val & 0xFF;
	value[1] = (val >> 8) & 0xFF;
	value[2] = (val >> 16) & 0xFF;
	value[3] = (val >> 24) & 0xFF;

	for (int i = 0; i < 4; i++)
	{
		EEPROM[addr + i] = value[i];
#ifdef DEBUG
	Serial.println("Addr: " + String(addr + i) + " byte: " + String(value[i]));
#endif
	}
}

 ul Settings::getParam(params par)
{
	const int addr = par * 4;

	const ul val = (long)EEPROM.read(addr) | (long)EEPROM.read(addr + 1) << 8 | (long)EEPROM.read(addr + 2) << 16 | (long)EEPROM.read(addr + 3) << 24;

#ifdef DEBUG
	char str[20];
	Serial.println("EEPROM read addr: " + String(addr / 4) + " val: " + val);
#endif
	return val;
}

 void Settings::resetSettings()
 {
	 setParam(is_empty, 1);
	 initDefault();

	 Serial.println("Settings reseted");
 }

void Settings::initDefault()
{
	if (!getParam(is_empty))
		return;

	setParam(pc_shutdown_timeout, seconds(30));

	setParam(batt_critical, 112);
}

Собственно я храню значения от ms до min, конвертируя их в ms. У меня была идея использовать два байта, первое это значение, второй это порядок

я почему-то так и предполагал.

Собственная реализация записи long в ЕЕПРОМ? - уже много лет для этого есть методы EEPOM.put() и get()

Не зря выложил)

Пятница!

put - плохо - он аккуратно через update сделан. То ли дело здесь - шарах сразу прям туда и всё пофиг - для настоящих мужчин!

Таки ответ зависит от того, что на голове - ермолка или арафатка...

Я, например, больше к старорежимному склонен.

А чо никто не попенял на кривые макросы, ась?

#define minutes(x) (x*60000ul)
#define seconds(x) (x*1000ul)

Топикстартеру: ответь на вопрос - что будет, если использовать эти макросы вот так:

minutes(1+5)
seconds(100+23)

??? Сам догадаешься, где косяк, или прямо указать?

Код - полная фигня, а автор - молодец!

#define minutes(x) (x*60000ul)
#define seconds(x) (x*1000ul)

Наверное, никто не смотрел достаточно внимательно. 

А ведь и впрямь, что-то в этом есть от Великого :)

Макросы - зло. 

Может и зло, но не большее, чем "великие".

Hе, дида, наоборот - благодаря этому форуму N-ная часть разработчиков будет обкладываться скобками в макросах, считаю - это тот успех, за который боролся Великий Чма - просвещение ;) 

#define minutes(x) (x*60000ul)
#define seconds(x) (x*1000ul)

Я бы даже сказал - не "что-то", а чистейшей воды плагиат.