Objective-C：爲什麼我的if語句不起作用？

Question

在這裏我有一些代碼，檢查如果iPhone有一個相機閃光燈。然後檢查，如果我的交換機是通過使用此代碼：

if(switch1.on){ 

       } 

的指示燈亮起罰款沒有這個代碼，我也嘗試過這種代碼以及：

if(!(switch1.on)){ 

        } 

，併成功地令我在相機閃光燈也打開它，甚至當開關被設置爲關閉

這裏是我的全碼：

-(IBAction)torchon:(id)sender{ 
    AVCaptureDevice *flashlight = [AVCaptureDevice defaultDeviceWithMediaType:AVMediaTypeVideo]; 
    if ([flashlight isTorchAvailable] & [flashlight isTorchModeSupported:AVCaptureTorchModeOn]) { 

     BOOL success = [flashlight lockForConfiguration:Nil]; 
     if(success){ 
      if(switch1.on){ //// or (!(switch1.on)) 

       on.hidden = YES; 
       [UIScreen mainScreen].brightness = 1.0; 
       [flashlight setTorchMode:AVCaptureTorchModeOn]; 
       [flashlight unlockForConfiguration]; 
      } 
     } 
    }  
} 

任何幫助，將理解

**EDIT:** 

下面是其中i將開關設置爲沒有使用ibactions當開關的值已經改變的代碼。此代碼成功：

-(IBAction)switch1{ 
    if (switch1.on) { 
     switch1.on = YES; 
     switch2.on = NO; 
    } else{ 
     switch2.on = YES; 
     switch1.on = NO; 
    }  
} 

-(IBAction)switch2{ 
    if (switch2.on) { 
     switch2.on = YES; 
     switch1.on = NO; 
    } else{ 
     switch1.on = YES; 
     switch2.on = NO; 
    } 
} 

回答

加上一個布爾值：

bool yes; 

然後設置布爾在viewdidload

- (void)viewDidLoad 
{ 
    [super viewDidLoad]; 
    // Do any additional setup after loading the view, typically from a nib. 
    yes = YES; 
} 

之後，將其設置爲當ibaction被調用並設置時爲YES爲NO一旦開關被關閉：

-(IBAction)switch1{ 
    if (swich1.on) { 
    swich1.on = YES; 
    swich2.on = NO; 
     yes = YES; 
    } 

} 


-(IBAction)switch2{ 
    if (swich2.on) { 

    swich2.on = YES; 
    swich1.on = NO; 
     yes = NO; 
    } 

} 

最後，它檢查if(switch1.on){}你需要將其更改爲if(yes==YES){}這樣在一起，如果聲明是這樣的：

-(IBAction)torchon:(id)sender{ 
    AVCaptureDevice *flashlight = [AVCaptureDevice defaultDeviceWithMediaType:AVMediaTypeVideo]; 
    if ([flashlight isTorchAvailable] & [flashlight isTorchModeSupported:AVCaptureTorchModeOn]) { 

     BOOL success = [flashlight lockForConfiguration:Nil]; 
     if(success){ 
      if(yes==YES){ 
       on.hidden = YES; 
       [UIScreen mainScreen].brightness = 1.0; 
       [flashlight setTorchMode:AVCaptureTorchModeOn]; 
       [flashlight unlockForConfiguration]; 

      } 
     } 

    } 

} 

Answer 1

的UISwitch還有另外一個getter方法on屬性。

@property(nonatomic, getter=isOn) BOOL on; 

這可能是更好地使用.isOn而不是.on對所獲得的價值。

此外，一些你的代碼可以簡化爲：

-(IBAction)switch1{ 
    [switch2 setOn:!switch1.isOn];  
} 

-(IBAction)switch2{ 
    [switch1 setOn:!switch2.isOn]; 
} 

，如果你使用的動作sender在你的方法可以變得更好。

Answer 2

看來您的IBAction方法與交換機的getter類似。這可能會導致不希望的副作用。重構你的方法，一個方法，更符合良好的編程習慣線：

-(IBAction)switchDidChange:(UISwitch*)aSwitch { 
    UISwitch* otherSwitch = (aSwitch == switch1)? switch2 : switch1; 
    [otherSwitch setOn:!aSwitch.on animated:YES]; 
} 

此外，在代碼中on財產並沒有真正似乎會影響什麼是開啓或關閉。包括這個條件。