PIC18F26K83 LED does not work according to debugger

GunkutA
But in the first 17 trials in the for loop it will not enter the if condition and in the 18. it will enter and copy the value to the temperature. Am I wrong?

GunkutA
 
But the LED is ON how can it be possible? Because of ANSEL registers maybe? But other than that case LED is working perfectly. Maybe a bug?

#include <stdint.h>
  int i;
  int k;
 int temperature;
 unsigned int temp_data;
 short transmit_data1;
 short transmit_data2;
 uint16_t data_transmit;
 int temp_ADC_array[34]= { 259,293,332,377,428,487,555,632,720,821,934,1062,1203,1360,1531,1715,1910,2113,2320,2528,2731,2926,3108,3274,3422,3552,3663,3756,3833,3895,3945,3983,4013,4036};
 int temp_array[34]= {125,120,115,110,105,100,95,90,85,80,75,70,65,60,55,50,45,40,35,30,25,20,15,10,5,0,-5,-10,-15,-20,-25,-30,-35,-40 };

#define led (LATA.B2)



void Clk_62kHz (){
  NOSC2_BIT =1;
  NOSC1_BIT=1; //HFINTOSC
  NOSC0_BIT=0;
  FRQ3_BIT =0;
  FRQ2_BIT=0; //1 MHz
  FRQ1_BIT=0;
  FRQ0_BIT=0;
  NDIV3_BIT =0;
  NDIV2_BIT=1;
  NDIV1_BIT=0; //Divide 16 =62.5 kHz.
  NDIV0_BIT=0;

  }

void main(){
//Set the pins of the MCU
TRISA.B2=0; //LED
LATA.B2=0;
TRISA.B3=1; //Case Temp A
ANSELA.B3=1;
TRISC.B5=0; //SCLK
TRISC.B6=0; //CS
LATC.B6=1; //Deselect slave
TRISC.B7=0; //DIN
ANSELC.B5=0;
ANSELC.B6=0;
ANSELC.B7=0;
 temperature=0;
    Clk_62kHz();
   // Clk_8Mhz() ;
 //PPS Mapping
RC7PPS= 0b00011111 ; //DIN, RC7 = SDIPPS
RC6PPS= 0b00100000; //CS, RC6= SSPPS
RC5PPS= 0b00011110; //SCLK, RC5=SCKPPS
  //transmit_data= 58112;

transmit_data1=0b11100011;
   //buradan assagisi while loopun icindeydi!!!!!
   LATC.B6=0; //Select the slave


  transmit_data2=0b01111111;





 LATC.B6=1; //deselect the slave && update the data

while(1)
{
    temp_data=2000;
    for(i =0;i<34; i++)
    {
        if (temp_data>temp_ADC_array[33-i])
        {
            temperature = temp_array[33-i];
            if (temperature>50)
            {
                led=1;
            }
            else
            {
                led=0;
            }
        } 
        }
        delay_ms(5000);
    }
}

1and0
I'm sure you can provide a clearer picture. :(   And I think you meant LED_RED is connected to RB2, not RA2.  Give us the respect by providing better information!
 
Anyway, get back to basic.  Replace the entire while(1) body with led=1 and see what happen; then replace led=0 and see what happen.

1and0

GunkutA
 
No the LED is connected to RA2 see LATA.B2 from the code and also in hardware it is connectedo to RA2. :(

My apology. I hate the bit struct of that compiler.  I see B2 and think RB2.
 

LED is working fine. When I try to make it on and off without that if(temperature>50) statement it works fine. Maybe there is a variable conflict? But both of them are int so

It that is the case, then the if(temperature) statement is not working correctly. Can you post the disassembly listing of it?
 

;
L_main0:
;BRE70111_gosterge.c,68 :: temp_data=2000;
 MOVLW 208
 MOVWF _temp_data+0 
 MOVLW 7
 MOVWF _temp_data+1 
;BRE70111_gosterge.c,69 :: for(i =0;i<34; i++)
 CLRF _i+0 
 CLRF _i+1 
L_main2:
 MOVLW 128
 XORWF _i+1, 0 
 MOVWF R0 
 MOVLW 128
 SUBWF R0, 0 
 BTFSS STATUS+0, 2 
 GOTO L__main11
 MOVLW 34
 SUBWF _i+0, 0 
L__main11:
 BTFSC STATUS+0, 0 
 GOTO L_main3
;BRE70111_gosterge.c,71 :: if (temp_data>temp_ADC_array[33-i])
 MOVF _i+0, 0 
 SUBLW 33
 MOVWF R3 
 MOVF _i+1, 0 
 MOVWF R4 
 MOVLW 0
 SUBFWB R4, 1 
 MOVF R3, 0 
 MOVWF R0 
 MOVF R4, 0 
 MOVWF R1 
 RLCF R0, 1 
 BCF R0, 0 
 RLCF R1, 1 
 MOVLW _temp_ADC_array+0
 ADDWF R0, 0 
 MOVWF FSR2L+0 
 MOVLW hi_addr(_temp_ADC_array+0)
 ADDWFC R1, 0 
 MOVWF FSR2L+1 
 MOVF POSTINC2+0, 0 
 MOVWF R1 
 MOVF POSTINC2+0, 0 
 MOVWF R2 
 MOVF _temp_data+1, 0 
 SUBWF R2, 0 
 BTFSS STATUS+0, 2 
 GOTO L__main12
 MOVF _temp_data+0, 0 
 SUBWF R1, 0 
L__main12:
 BTFSC STATUS+0, 0 
 GOTO L_main5
;BRE70111_gosterge.c,73 :: temperature = temp_array[33-i];
 MOVF _i+0, 0 
 SUBLW 33
 MOVWF R3 
 MOVF _i+1, 0 
 MOVWF R4 
 MOVLW 0
 SUBFWB R4, 1 
 MOVF R3, 0 
 MOVWF R0 
 MOVF R4, 0 
 MOVWF R1 
 RLCF R0, 1 
 BCF R0, 0 
 RLCF R1, 1 
 MOVLW _temp_array+0
 ADDWF R0, 0 
 MOVWF FSR0L+0 
 MOVLW hi_addr(_temp_array+0)
 ADDWFC R1, 0 
 MOVWF FSR0L+1 
 MOVF POSTINC0+0, 0 
 MOVWF _temperature+0 
 MOVF POSTINC0+0, 0 
 MOVWF _temperature+1 
;BRE70111_gosterge.c,74 :: }
L_main5:
;BRE70111_gosterge.c,69 :: for(i =0;i<34; i++)
 INFSNZ _i+0, 1 
 INCF _i+1, 1 
;BRE70111_gosterge.c,75 :: }
 GOTO L_main2
L_main3:
;BRE70111_gosterge.c,77 :: if (temperature<0)
 MOVLW 128
 XORWF _temperature+1, 0 
 MOVWF R0 
 MOVLW 128
 SUBWF R0, 0 
 BTFSS STATUS+0, 2 
 GOTO L__main13
 MOVLW 0
 SUBWF _temperature+0, 0 
L__main13:
 BTFSC STATUS+0, 0 
 GOTO L_main6
;BRE70111_gosterge.c,79 :: led=1;
 BSF LATA+0, 2 
;BRE70111_gosterge.c,80 :: }
 GOTO L_main7
L_main6:
;BRE70111_gosterge.c,83 :: led=0;
 BCF LATA+0, 2 
;BRE70111_gosterge.c,84 :: }
L_main7:
;BRE70111_gosterge.c,85 :: delay_ms(5000);
 MOVLW 110
 MOVWF R12, 0
 MOVLW 146
 MOVWF R13, 0
L_main8:
 DECFSZ R13, 1, 1
 BRA L_main8
 DECFSZ R12, 1, 1
 BRA L_main8
 NOP
 NOP
;BRE70111_gosterge.c,86 :: }
 GOTO L_main0
;BRE70111_gosterge.c,87 :: }
L_end_main:
 GOTO $+0
; end of _main

GunkutA
 

;BRE70111_gosterge.c,77 :: if (temperature<0)
 MOVLW 128
 XORWF _temperature+1, 0 
 MOVWF R0 
 MOVLW 128
 SUBWF R0, 0 
 BTFSS STATUS+0, 2 
 GOTO L__main13
 MOVLW 0
 SUBWF _temperature+0, 0 
L__main13:
 BTFSC STATUS+0, 0 
 GOTO L_main6