Вопрос Помогите с++ ардуино пж

[Geogral]

#include <Stepper.h>



// Указываем количество шагов на один оборот

const int stepsPerRevolution = 2048; // для 28BYJ-48 с редуктором



// Подключение к выводам драйвера ULN2003

Stepper myStepper(stepsPerRevolution, 8, 10, 9, 11);



// Параметры подъема

float height = 0.5; // высота подъема в метрах

float timeToLift = 10; // время подъема в секундах

float pulleyRadius = 0.02; // радиус шкива в метрах (например, 2 см)



void setup() {

myStepper.setSpeed(15); // скорость в об/мин (можно подбирать)

Serial.begin(9600);



// Вычисляем длину троса

float ropeLength = height;



// Вычисляем, сколько оборотов нужно сделать

float pulleyCircumference = 2 * 3.1416 * pulleyRadius;

float requiredRotations = ropeLength / pulleyCircumference;



// Вычисляем общее количество шагов

int totalSteps = requiredRotations * stepsPerRevolution;



// Вывод в сериал для отладки

Serial.print("Total steps: "

Serial.println(totalSteps);



// Движение мотора

myStepper.step(totalSteps);

}



void loop() {

// ничего не делаем

}



Как оптимизировать данный код ?

 

[voland008]

#include <Arduino.h>
#include <Stepper.h>
#include <math.h> // M_PI

/***** Аппаратные константы *****/
constexpr long STEPS_PER_REV = 2048; // 28BYJ‑48 + редуктор
constexpr byte PIN_IN1 = 8;
constexpr byte PIN_IN2 = 10;
constexpr byte PIN_IN3 = 9;
constexpr byte PIN_IN4 = 11;

/***** Габариты механики *****/
constexpr float PULLEY_R = 0.02f; // радиус шкива, м

/***** Параметры подъёма (можно менять) *****/
constexpr float HEIGHT_M = 0.50f; // метры
constexpr float LIFT_TIME_S = 10.0f; // секунды

/***** Производные константы (считаются на этапе компиляции) *****/
constexpr float PULLEY_CIRC = 2.0f * M_PI * PULLEY_R; // м/об
constexpr float ROTATIONS_N = HEIGHT_M / PULLEY_CIRC; // об
constexpr long TOTAL_STEPS = lround(ROTATIONS_N * STEPS_PER_REV); // шаги

/* нужные об/мин, чтобы пройти высоту за LIFT_TIME_S */
constexpr float TARGET_RPM = (HEIGHT_M / LIFT_TIME_S) / PULLEY_CIRC * 60.0f;

Stepper motor(STEPS_PER_REV, PIN_IN1, PIN_IN2, PIN_IN3, PIN_IN4);

/*------------------------------------------------------------------*/
void setup()
{
Serial.begin(115200);

motor.setSpeed(TARGET_RPM); // RPM -> библиотека сама считает задержку
Serial.println(F("=== Stepper lift demo ===&#34);
Serial.print (F("RPM set to: &#34); Serial.println(TARGET_RPM);
Serial.print (F("Total steps: &#34); Serial.println(TOTAL_STEPS);

motor.step(TOTAL_STEPS); // единоразовый подъём
}
/*------------------------------------------------------------------*/
void loop() { /* пусто */ }
/*------------------------------------------------------------------*/

 

[Froct1025]

Как оптимизировать данный код ?Заменить float на int и считать в целых числах. Поскольку слабый микроконтроллер в ардуине float-ы крайне медленно считает

 

[Пиздюк]

const int stepsPerRevolution = 2048; это удали
stepsPerRevolution а это замени на 2048

 

[тима)]

Ваш код уже достаточно хорошо структурирован, но его можно немного оптимизировать для улучшения читаемости и функциональности. Вот несколько рекомендаций по оптимизации:

1. Избегайте дублирования кода: Разбейте задачи на функции для лучшей читаемости.
2. Избавьтесь от магических чисел: Определите константы для чисел, используемых в расчетах.
3. Добавьте комментарии: Убедитесь, что код остается понятным другим разработчикам.

Вот оптимизированная версия вашего кода:

#include <Stepper.h>

// Указываем количество шагов на один оборот
const int stepsPerRevolution = 2048; // для 28BYJ-48 с редуктором

// Подключение к выводам драйвера ULN2003
Stepper myStepper(stepsPerRevolution, 8, 10, 9, 11);

// Параметры подъема
const float height = 0.5; // высота подъема в метрах
const float pulleyRadius = 0.02; // радиус шкива в метрах (например, 2 см)
const float pi = 3.1416; // значение числа Пи

void setup() {
myStepper.setSpeed(15); // скорость в об/мин
Serial.begin(9600);

int totalSteps = calculateSteps(height, pulleyRadius);

// Вывод в сериал для отладки
Serial.print("Total steps: "
Serial.println(totalSteps);

// Движение мотора
myStepper.step(totalSteps);
}

void loop() {
// ничего не делаем
}

// Функция для вычисления общего количества шагов
int calculateSteps(float liftHeight, float radius) {
float ropeLength = liftHeight;
float pulleyCircumference = 2 * pi * radius;
float requiredRotations = ropeLength / pulleyCircumference;
return requiredRotations * stepsPerRevolution;
}


### Изменения и улучшения:
- Функция calculateSteps: Эта функция рассчитывает общее количество шагов, что позволяет изолировать логику вычислений.
- Константы: Добавлен const для pi, чтобы избежать использования "магического числа" при вычислениях.
- Комментарии: Убеждены, что код обеспечивает ясность.

Такие изменения делают код более структурированным и простым для поддержки.