Сижу пару дней размышляю. Есть функции которые исполняются по условию.
Скажем если энкодер для данного мотора активирован - проверяй позицию, если не активирован - то соответственно и проверять нечего.
можно записать так

или спрятать проверку условия в функцию

и потом


Вопрос как лучше?

Сообщение отредактировал Jenya7 - Sep 28 2017, 10:04

Неоднократно встречал рекомендацию: функция должна делать что-то одно. Функция, выполняющая 2 разных действия (проверяет позицию и регулирует ток) - моветон. Но если функция выставляет привод на указанную позицию с помощью регулировки тока - почему бы и нет.

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

Сообщение отредактировал Jenya7 - Sep 28 2017, 10:47

Вынесите проверку условия в функцию-обертку. Основную функцию сделайте статической, чтобы исключить возможность ее случайного вызова не из обертки, без проверки условия. В плюсах все это реализуется красивее защищенным наследованием. Если оставите условие вне функции - или забудете его написать или проверите что-то не то, к гадалке не ходите.

Тогда лучше пусть эта функция только проверяет позицию. А условие оставить снаружи. Потом понятнее будет читать.

я понял. спасибо.

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

static int Coordinate = 0;
RetCode = EncoderStateGet( &Coordinate, . . . . );

По коду возврата определяем, имеет ли смысл смотреть координату.
Кроме того такой подход позволяет последующее расширение функциональности, например,
выдавать "некритичные" ошибки, вроде "возможна потеря синхронизации" итп

Тут конечно, более интересно решение на ООП, энкодер+мотор==объект.