Question

I am having a problem trying to get the orientation of the device correct. I have to show an action sheet that should come depending on the orientation of the device. Here is the code I am using.

UIDeviceOrientation orientation = [UIDevice currentDevice].orientation;

  switch (orientation) 

  {
   case UIDeviceOrientationPortrait:
   case UIDeviceOrientationFaceUp:
   case UIDeviceOrientationFaceDown: 
   case UIDeviceOrientationUnknown:
   case UIDeviceOrientationPortraitUpsideDown:

   [self displayActionSheetInPotraitMode];
   break;

   case UIInterfaceOrientationLandscapeLeft:
   case UIInterfaceOrientationLandscapeRight:

   [self displayActionSheetInLandscapeMode];
   break;

   default:

   [self displayActionSheetInPotraitMode];
   break;
}

Answer 1

Let’s look what the UIDevice Class Reference tells us about the orientation property.

The value of this property always returns 0 unless orientation notifications have been enabled by calling beginGeneratingDeviceOrientationNotifications.

So, you should call

[[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications]

somewhere earlier in your code.

---

Also, I’d like to suggest a way to simplify your current code:

UIDeviceOrientation orientation = [UIDevice currentDevice].orientation;

if (UIDeviceOrientationIsLandscape(orientation)) {
    [self displayActionSheetInLandscapeMode];
} else {
    [self displayActionSheetInPotraitMode];
}