1

Estoy intentando simplificar el contenido de la función movimiento() a dos bucles, pero al hacerlo en bucles no me hace nada. Mi código es el siguiente:

int movimimiento(){
    mueve_servo(&patas[1][1], mov_home[1][1]);
    mueve_servo(&patas[1][2], mov_home[1][2]);
    mueve_servo(&patas[3][1], mov_home[3][1]);
    mueve_servo(&patas[3][2], mov_home[3][2]);
    mueve_servo(&patas[5][1], mov_home[5][1]);
    mueve_servo(&patas[5][2], mov_home[5][2]);

}

Mi duda es como puedo poner el contenido en forma de bucles para reducir código y sobre todo que tenga la misma funcionalidad. He hecho lo siguiente, pero no me hace nada:

int numero_patas[3] = {1, 3, 5};
int num_art[2] = {1, 2};

for(int pata = 0; pata < sizeof(numero_patas); pata++) {
    for(int art = 0; art < sizeof(num_art); art++) {
        mueve_servo(&patas[numero_patas[pata]][num_art[art]], mov_home[numero_patas[pata]][num_art[art]]);
    }
}

Muchas gracias!!

1

1 respuesta 1

3

Estas haciendo algo simple de una forma totalmente complicada.

No necesitas un array (y menos dos) para hacer ese for.

Solamente tenes que mover tus variables la cantidad de pasos necesarios, y cortar donde corresponde.

Seria algo asi:

for(int pata = 1; pata < 7; pata=pata+2) {
    for(int art = 1; art < 3; art++) {
        mueve_servo(&patas[pata][art], mov_home[pata][art]);
    }
}
7
  • Te comento por aquí, las llamadas a patas y mov_home no se podría quitar porque son llamadas a los pines de la placa y a los valores de movimiento del servo. SoftPWM patas[6][3] ={{SoftPWM(coxa_1),SoftPWM(femur_1),SoftPWM(tibia_1)},{SoftPWM(coxa_2),SoftPWM(femur_2),SoftPWM(tibia_2)},{SoftPWM(coxa_3),SoftPWM(femur_3),SoftPWM(tibia_3)},{SoftPWM(coxa_4),SoftPWM(femur_4),SoftPWM(tibia_4)},{SoftPWM(coxa_5),SoftPWM(femur_5),SoftPWM(tibia_5)},{SoftPWM(coxa_6),SoftPWM(femur_6),SoftPWM(tibia_6)}}; int mov_home[6][3] = { {120,90,90}, {85,90,90}, {60,90,90}, {60,90,90}, {95,90,90}, {130,90,90}};
    – jose
    Commented el 11 nov. 2018 a las 22:59
  • Lo único que podría cambiar serían los valores de dentro del array: mueve_servo(&patas[1][1], mov_home[1][1]); ¿se podría de alguna manera?
    – jose
    Commented el 11 nov. 2018 a las 23:00
  • ahi edite la respuesta @jose. Lo principal era explicarte que tu for esta mal.
    – gbianchi
    Commented el 11 nov. 2018 a las 23:03
  • Y esta siguiente función como quedaría? void avanza(){ mueve_servo(&patas[1][SERVO_COX], mov_avanza[1 [SERVO_COX]); mueve_servo(&patas[3][SERVO_COX], mov_avanza[3][SERVO_COX]); mueve_servo(&patas[5][SERVO_COX], mov_avanza[5][SERVO_COX]); mueve_servo(&patas[0][SERVO_COX], mov_retrocede[0][SERVO_COX]); mueve_servo(&patas[2][SERVO_COX], mov_retrocede[2][SERVO_COX]); mueve_servo(&patas[4][SERVO_COX], mov_retrocede[4][SERVO_COX]); }
    – jose
    Commented el 12 nov. 2018 a las 16:58
  • @jose si los numeros son siempre simetricos, y siempre el mismo orden, vale mas que escribas las lineas de codigo y no te mezcles en tratar de hacer vectores y for...next que no sean logicos. No tiene mucho sentido.
    – gbianchi
    Commented el 12 nov. 2018 a las 17:00

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.