Если оператор не работает - iOS

Question

У меня есть приложение iOS с несколькими простыми и логичными if-statements. Однако они не побегут, и я не могу понять, почему. Вот мой простой код:

-(void)viewDidLoad { 

    [super viewDidLoad]; 

    // Run setup code. 

    int day = [self currentDay]; 

    if ((day == (1 || 3 || 7) && (day != (2 || 4 || 5 || 6))) { 

     // Run the calendar setup code for 
     // Sunday/Tuesday OR saturday. 
     [self runSetupVX_4]; 
    } 

    else if ((day == (2 || 4 || 5 || 6)) && (day != (1 || 3 || 7))) { 

     // Run setup code for Monday 
     // wednesday, thursday and friday. 
     [self runSetupVX_2]; 
    } 
} 

-(int)currentDay { 

    NSDateComponents *component = [[NSCalendar currentCalendar] components:NSCalendarUnitWeekday fromDate:[NSDate date]]; 
    return [component weekday]; 
} 

Что я здесь делаю неправильно?

Answer 1

компилируется, но он делает что-то совершенно отличное от того, что вы пытаетесь достичь:

day == (1 || 3 || 7) && (day != (2 || 4 || 5 || 6) 

В заявлении ОШ 1, 3 и 7, и затем ORs 2, 4, 5 и 6, перед выполнением сравнений.

Правильный способ сделать это - сравнить day с 1, 3 и 7 отдельно. Если какой-либо из сравнений успешна, это также гарантирует, что день не 2, 4, 5 или 6:

if (day == 1 || day == 3 || day == 7) 
    ... 
if (day == 2 || day == 4 || day == 5 || day == 6) 
    ... 

Вы также можете переписать с switch:

switch(day) { 
    case 1: 
    case 3: 
    case 7: 
     // Run the calendar setup code for 
     // Sunday/Tuesday OR saturday. 
     [self runSetupVX_4]; 
     break; 
    case 2: 
    case 4: 
    case 5: 
    case 6: 
     // Run setup code for Monday 
     // wednesday, thursday and friday. 
     [self runSetupVX_2]; 
     break; 
} 

Answer 2

Вы можете поместить точку останова в оператор if, а затем проверить, вызывается ли оператор, вызываемый или нет.

Answer 3

Вы немного смущаете его.

if(day==1 || day==2 || day==3){ 
    [self runSetupVX_4]; 
}else{ 
    [self runSetupVX_2]; 
} 

Если сделать трюк

Answer 4

, если первая часть каждого оператора if истинна, тогда вторая часть будет ложной каждый раз. Поэтому вам не нужно проверять вторую часть.

Вы можете изменить свой код так:

-(void)viewDidLoad { 

[super viewDidLoad]; 

// Run setup code. 

int day = [self currentDay]; 

if (day ==1 || day==3 || day==7) { 

    // Run the calendar setup code for 
    // Sunday/Tuesday OR saturday. 
    [self runSetupVX_4]; 

} 
else{ 

     // Run setup code for Monday 
     // wednesday, thursday and friday. 
     [self runSetupVX_2]; 


}} 

-(int)currentDay { 

    NSDateComponents *component = [[NSCalendar currentCalendar]   components:NSCalendarUnitWeekday fromDate:[NSDate date]]; 
    return [component weekday]; 
} 

Answer 5

Надеется, что это работает для вас:

if (day == 1 || day == 2 || day == 3) { 

    // Run the calendar setup code for 
    // Sunday/Tuesday OR Saturday. 
    [self runSetupVX_4]; 
} 

else if (day == 2 || day == 4 || day == 5 || day == 6) { 

    // Run setup code for Monday 
    // Wednesday, Thursday and Friday. 
    [self runSetupVX_2]; 
} 

Answer 6

-(void)viewDidLoad { 

    [super viewDidLoad]; 

    // Run setup code. 

    int day = [self currentDay]; 
    //if ((day == (1 || 3 || 7) && (day != (2 || 4 || 5 || 6))) { 
    //here u will get 0 or 1 from this (1 || 3 || 7) && (day != (2 || 4 || 5 || 6))) 
    //And u r comparing with 0 or 1 with day so please check below code how to compare 

    if (((day == 1 || 3 || 7) && (day != 2 || 4 || 5 || 6)) { 

    // Run the calendar setup code for 
    // Sunday/Tuesday OR saturday. 
    [self runSetupVX_4]; 
    } 

    else if ((day == 2 || 4 || 5 || 6)&& (day != 1 || 3 || 7)) { 

    // Run setup code for Monday 
    // wednesday, thursday and friday. 
    [self runSetupVX_2]; 
    } 
} 

-(int)currentDay { 

    NSDateComponents *component = [[NSCalendar currentCalendar] components:NSCalendarUnitWeekday fromDate:[NSDate date]]; 
    return [component weekday]; 
} //you have problems with comparing variable please check it