problema con comparazione di 2 stringhe in c (strcmp )

[mattia281193]

Codice: Seleziona tutto

char uart_rd;
int res;
char txt[]= "a";
void main() {
  TRISB=0;
  UART1_Init(9600);
  Delay_ms(100);
  PORTB=0x01;
  Delay_ms(100);
  PORTB=0x00;


  while (1) {
    if (UART1_Data_Ready()) {
      uart_rd = UART1_Read();
      res = strcmp( uart_rd,txt);
      if (res==0) {
      portb=0x01;
    }
    else  {
    portb=0x00 ;
    }
  }
}
}

sto utilizzando un PIC 16f876a a 8 MHz , come compilatore MicroC Pro .
il programma viene compilato ma non si attiva la portb.0 .
ho letto che "res" è zero se sono uguali ma può essere sia maggiore che minore di 0 nel caso siano diverse , io l'ho considerata come fosse booleana e ho utilizzato una condizione if , ma penso che sia li la fregatura .
Io scrivo su mando da hyperterminal un file txt contenente solo "a" (senza le virgolette) tramite lo shield USB per xbee lo trasmetto via onde radio e un altro modulo xbee riceve e trasmette al PIC la stringa.
Lo xbee ricevente riceve il segnale, infatti si accende un led verde per indicare che sta in comunicazione .

[DirtyDeeds]

La strcmp vuole che le stringhe siano terminate dal carattere nullo '\0': uart_rd non lo è.

[mattia281193]

La strcmp vuole che le stringhe siano terminate dal carattere nullo '\0': uart_rd non lo è.

Codice: Seleziona tutto

char txt[]= "a\0'";

cosi per esempio?

[mattia281193]

ah scusa uart_rd deve essere terminato scusa ancora

[DirtyDeeds]

Codice: Seleziona tutto

char txt[]= "a\0'";

cosi per esempio?

No, decisamente no.

txt è una stringa, uart_rd è un carattere. Tra l'altro, perché vuoi usare la funzione di comparazione tra stringhe quando devi solo comparare un carattere?

[mattia281193]

errata

[mattia281193]

ah è vero una volta letto uart_rd è un carattere come posso comparare una char e una stringa?devo riconvertire uart_rd in stringa e utilizzare strcmp giusto?

[Candy]

Un carattere ed una stringa non sono comparabili, a meno che la stringa non contenga un unico carattere; che equivale a comparare un carattere:

Codice: Seleziona tutto

char a, b;
bool r;

a = '1';
b = '2';

a == b ? r = TRUE : r = FALSE; // r = FALSE
a == '1' ? r = TRUE : r = FALSE; // r = TRUE
b == '2' ? r = TRUE : r = FALSE; // r = TRUE
a = '2';
a == b ? r = TRUE : r = FALSE; // r = TRUE




[mattia281193]

Codice: Seleziona tutto

char uart_rd;
char txt[]= "a";
void main() {
  TRISB=0;
  UART1_Init(9600);
  Delay_ms(100);
  PORTB=0x01;
  Delay_ms(100);
  PORTB=0x00;


  while (1) {
    if (UART1_Data_Ready()) {
      uart_rd = UART1_Read();
      if (uart_rd==0x61) {
      portb=0x01;
    }
    else  {
    portb=0x00 ;
    }
  }
}
}

sono stato depistato dalla strcmp bastava questo ! grazie per le risposte e per la pazienza

[rusty]

Si ma a questo punto non ha piu' senso definire la variabile txt visto che non la usi, a meno che non scrivi:

Codice: Seleziona tutto

char txt= 'a';
...
   if (uart_rd==txt) {
      portb=0x01;
    }
...