Convertir variables char a int16

[PabloPic]

Hola muy buen dia, he estado probando un codigo que realmente no me sale, estoy intentando transformar una variable string (ejemplo char valor[5] a una variable entera tipo int16 duty, se que la funcion es duty=atol(valor); pero no me sale nada. Ayuda porfavor se los agradecere    , os dejo el codigo:

#include <18f4550.h>
#fuses INTRC,PUT,BROWNOUT
#use delay(INTERNAL=4M)
#use RS232 (BAUD=9600, XMIT=PIN_C6, RCV=PIN_C7, PARITY=N,BITS=8)
#include <lcd.c>
#include <stdlib.h>
char valor[9];
int16 duty;

#INT_RDA
void serial_isr()
{
   fgets(valor);
   delay_ms(50);
   puts(valor);
   
   duty = atol(valor);
   printf(lcd_putc,"%Lu",duty);
}


void main()
{
   enable_interrupts(GLOBAL);
   enable_interrupts(INT_RDA);
   set_tris_c(0x00);
   
   //CONFIGURACION PARA EL PWM, FRECUENCIA DE MOTOR A 1 KHz
   setup_timer_2(T2_DIV_BY_16,255,1);
   setup_ccp1(CCP_PWM);
   set_pwm1_duty(0);
   
   //INICIAR PRESENTACION
   lcd_init();
   lcd_gotoxy(3,1);
   printf(lcd_putc,"PIC18F4550");
   lcd_gotoxy(1,2);
   printf(lcd_putc,"PWM: PIC_LabVIEW");
   delay_ms(2000);
   printf(lcd_putc,"\f");
   lcd_gotoxy(1,1);
   printf(lcd_putc,"VALOR DEL DUTY:");
   lcd_gotoxy(1,2);

   while(TRUE)
   {
   }
}

[AngelGris]

  No estoy familiarizado con CCS, por lo cual tal vez esté equivocado en algunos puntos.

  Imagino que la función fgets, te devolverá un string terminado en el decimal (13) -que, creo, es el caracter de retorno de carro-.

  Ahora bien, por lo poco que la he utilizado, la función atoi o atol necesita una variable string terminada en 0 -caracter nulo- y tal vez allí es donde esté el error.

[MerLiNz]

es atoi no atol, la 2ª es para long, prueba a ver si te va

[PabloPic]

Pues no MerLiNz le puse atoi pero no me sale nada jeje. Mmm el valor que recibo desde la PC al PIC va de 0 a 1024, por lo que supongo yo es string en este caso valor, cambiaria a char valor[5]; ? jeje debido a que solo recibo caracteres de hasta 4 digitos y creo que el ultimo se completa con un caracter "NULL", aun asi no funciona jeje rayos 

[AngelGris]

  Para ir descartando posibles problemas, yo crearía una variable char pero ya asignándole un valor e intentaría emplear atoi o atol en dicha variable en lugar de aplicarla a algo recibido desde el UART.

Código: C

1. char palabra[10] = "1256";

 por ejemplo.

[PabloPic]

Muy bien AngelGris probe como me dijiste:

char valor[10]= "1024";
int16 duty;

despues viene la conversión:

duty=atol(valor);
printf(lcd_putc,"%Lu",duty);

En la pantalla display 16x2 me sale el valor convertido a una int16 solo me funciona con atol, probe con atoi y me salian puros ceros.
Por el momento se que atol funciona para int16, ahora como lo convierto y recibo ya con la UART

[AngelGris]

Muy bien AngelGris probe como me dijiste:

char valor[10]= "1024";
int16 duty;

despues viene la conversión:

duty=atol(valor);
printf(lcd_putc,"%Lu",duty);

En la pantalla display 16x2 me sale el valor convertido a una int16 solo me funciona con atol, probe con atoi y me salian puros ceros.
Por el momento se que atol funciona para int16, ahora como lo convierto y recibo ya con la UART

  Para mí el problema viene por lo que te comenté en la primer respuesta. fgets termina el string con un caracter distinto de 0 y por ello genera los inconvenientes.

  Si como indicas, sólo recibes datos de hasta 4 dígitos, puedes intentar lo siguiente antes de la conversión

Código: C

1. valor[4] = 0;

  con ello eliminarías el caracter de retorno de carro y colocarías el caracter nulo. Para que la función atol no genere inconvenientes.

[PabloPic]

Mu bien, pues lo intente AngelGris, si recibo un numero string de 4 digitos mas un retorno de carro en total serian 5 espacios osea char valor[5]; al compilarlo me salia error, puesto que no podia insertar un valor cualsea en esa posicion del arreglo osea valor[5]=0; sigo sin llegar a algun exito. Siento que quizas deberia probar alguna logica diferente o cambiar la funcion fgets(valor) por alguna otra 

[AngelGris]

  Si tu defines de la siguiente forma

Código: C

1. char valor[9];

indicas que valor tiene 9 espacios, pero el índice va desde 0 a 8. Por ello el quinto caracter va a tener como índice el número 4.

  No cambies tu definición de la variable, simplemente intenta escribir en el índice número 4 de la misma. Tal vez CCS no te permita hacerlo, no lo sé.

[PabloPic]

Bien, pues buscando y hallando otras maneras para resolver este problema, en alguna página y en algún codigo encontre que para trasformarla simplemente hacia una operacion de una variable tipo char y una entera, ¿Quien sabe porque?, simplemente no lo sé, solo se que funciona, pueden probarlo. Mas o menos quedaría así:

char valor;
int16 duty;

#INT_RDA
void serial_isr()
{
   valor=getchar();
   delay_ms(50);
   duty=(valor-48);

   printf(lcd_putc,"%Lu",duty);

[RALF2]

Yo lo haria usando casting  
Asi:

char caract = 'Z';
   int16 Num;
  
   Num = (int16)caract;
   printf("%lu ",Num);

Nota:
 El caracter 'Z' es igual a 90 (en decimal)

Saludos

[AngelGris]

Bien, pues buscando y hallando otras maneras para resolver este problema, en alguna página y en algún codigo encontre que para trasformarla simplemente hacia una operacion de una variable tipo char y una entera, ¿Quien sabe porque?, simplemente no lo sé, solo se que funciona, pueden probarlo. Mas o menos quedaría así:

char valor;
int16 duty;

#INT_RDA
void serial_isr()
{
   valor=getchar();
   delay_ms(50);
   duty=(valor-48);

   printf(lcd_putc,"%Lu",duty);

  En este caso simplemente conviertes 1 dígito , no 4 dígitos como tú necesitas. Y funciona por como es el código ascii.

código ascii                   número

48                                    0
49                                    1
50                                    2

y así sucesivamente. Por lo tanto si le restas 48 al valor del código ascii obtienes el valor decimal que te interesa.

[RALF2]

Asi mismo Angelgris, solo que el necesita, segun el codigo que coloco mas arriba, el resultado en un long asi que usando el casting mata dos pajaros de un solo tiro   

[AngelGris]

Asi mismo Angelgris, solo que el necesita, segun el codigo que coloco mas arriba, el resultado en un long asi que usando el casting mata dos pajaros de un solo tiro   

  Si claro, mi respuesta fue porque a él parecía asombrarle que funcione el último código que mostró.

[PabloPic]

Lo que no entiendo es que segun RALF2, emplenado casting quedaria de esta manera:

char valor;
int16 duty;

#INT_RDA
void serial_isr()
{
   valor=getchar();
   duty=(int16)valor;
   printf("%Lu,valor);
}

Pero no me funciona, o ¿Esta mal el codigo?