iOS7 UISwitch его событие ValueChanged: Непрерывный вызов - это ошибка или что?

93

редактировать

Теперь это исправлено
Не делайте никаких настроек, чтобы исправить это.

Edit2

Видимо такая же проблема возникает снова в iOS 8.0 и 8.1.

Редактировать3

Теперь это исправлено
Не делайте никаких настроек, чтобы исправить это.


Привет Сегодня я видел в UISwitch'sСобытии ValueChanged:вызова в continuously то время как я изменение в Onдо Offили OffВкл и мой палец перемещается еще на правой стороне, а также с левой стороны. Я сделал изображение GIF для большей ясности с помощью NSLog.

введите описание изображения здесь

Мой метод изменения значения:

- (IBAction)changeSwitch:(id)sender{

    if([sender isOn]){
        NSLog(@"Switch is ON");
    } else{
        NSLog(@"Switch is OFF");
    }
    
}

iOS6 тот же код Switch работает нормально, как мы и ожидали:

введите описание изображения здесь

так что кто-нибудь может предложить мне, чтобы вызвать только один раз его состояние Вкл или Выкл. или это баг что ли ..?

ОБНОВИТЬ

Вот моя демонстрация:

программный Добавить UISwitch

из XIB добавив UISwitch

Нитин Гохель
источник
1
им все еще получаю эту ошибку в iOS7.1 на тренажере, Havent попытался устройства пока, работает Xcode 5.1.1
Fonix
3
У меня такая же проблема с 7.1.2 ipad
Hassy
7
Я вижу идентичную / аналогичную проблему с UISwitch в iOS 8.0 и 8.1
Морское побережье Тибета
2
Все еще здесь, в версии 9.1. Пожалуйста, отправьте копию openradar.appspot.com/15555929 всем. Это единственный способ исправить это.
Гийом Альгис,
1
Кажется, что он вернулся в 9.3
Бен Легжеро

Ответы:

44

См. Следующий код:

-(void)viewDidLoad
{
    [super viewDidLoad];    
    UISwitch *mySwitch = [[UISwitch alloc] initWithFrame:CGRectMake(130, 235, 0, 0)];    
    [mySwitch addTarget:self action:@selector(changeSwitch:) forControlEvents:UIControlEventValueChanged];
    [self.view addSubview:mySwitch];
}

- (void)changeSwitch:(id)sender{
    if([sender isOn]){
        NSLog(@"Switch is ON");
    } else{
        NSLog(@"Switch is OFF");
    }
}
AeroStar
источник
спасибо за ответ, поскольку я сказал, что пробовал оба способа и получил тот же результат. atlist Я знаю, как добавить swtich программно, а также из xib, сэр.
Нитин Гохель
12

Здесь та же ошибка. Думаю, я нашел простой обходной путь. Нам просто нужно использовать новый, BOOLкоторый сохраняет предыдущее состояние, UISwitchи оператор if в нашем IBAction(Value Changed fired), чтобы проверить, действительно ли значение переключателя изменилось.

previousValue = FALSE;

[...]

-(IBAction)mySwitchIBAction {
    if(mySwitch.on == previousValue)
        return;
    // resetting the new switch value to the flag
    previousValue = mySwitch.on;
 }

Больше никаких странных поступков. Надеюсь, это поможет.

nnarayann
источник
2
это должно просто сказать, если (mySwitch.on == previousValue)
Киган Джей
12

Вы можете использовать свойство UISwitch's, .selectedчтобы убедиться, что ваш код выполняется только один раз при изменении фактического значения. Я думаю, что это отличное решение, потому что оно позволяет избежать создания подклассов или добавления новых свойств.

 //Add action for `ValueChanged`
 [toggleSwitch addTarget:self action:@selector(switchTwisted:) forControlEvents:UIControlEventValueChanged];

 //Handle action
- (void)switchTwisted:(UISwitch *)twistedSwitch
{
    if ([twistedSwitch isOn] && (![twistedSwitch isSelected]))
    {
        [twistedSwitch setSelected:YES];

        //Write code for SwitchON Action
    }
    else if ((![twistedSwitch isOn]) && [twistedSwitch isSelected])
    {
        [twistedSwitch setSelected:NO];

        //Write code for SwitchOFF Action
    }
}

А вот и в Swift:

func doToggle(switch: UISwitch) {
    if switch.on && !switch.selected {
        switch.selected = true
        // SWITCH ACTUALLY CHANGED -- DO SOMETHING HERE
    } else {
        switch.selected = false
    }
}
itsji10dra
источник
6
Я считаю это самым простым.
zekel
+1, потому что это привело меня к использованию аналогичного решения для заполнения значения .tag, когда я хочу игнорировать логику переключения. Мне нужна логика для срабатывания иногда как во включенном, так и в выключенном режиме, поэтому приведенного выше было недостаточно.
Дэвидетелл
9

Если вы используете так много переключателей в своем приложении, тогда возникает проблема с изменением кода во всех местах, где определен метод действия t для UISwitch. Вы можете сделать пользовательский переключатель и обрабатывать события только при изменении значения.

CustomSwitch.h

#import <UIKit/UIKit.h>

@interface Care4TodayCustomSwitch : UISwitch
@end

CustomSwitch.m

@interface CustomSwitch(){
    BOOL previousValue;
}
@end

@implementation CustomSwitch



- (id)initWithFrame:(CGRect)frame
{
    self = [super initWithFrame:frame];
    if (self) {
        // Initialization code
        previousValue = self.isOn;
    }
    return self;
}


-(void)awakeFromNib{
    [super awakeFromNib];
    previousValue = self.isOn;
    self.exclusiveTouch = YES;
}


- (void)setOn:(BOOL)on animated:(BOOL)animated{

    [super setOn:on animated:animated];
    previousValue = on;
}


-(void)sendAction:(SEL)action to:(id)target forEvent:(UIEvent *)event{

    if(previousValue != self.isOn){
        for (id targetForEvent in [self allTargets]) {
            for (id actionForEvent in [self actionsForTarget:targetForEvent forControlEvent:UIControlEventValueChanged]) {
                [super sendAction:NSSelectorFromString(actionForEvent) to:targetForEvent forEvent:event];
            }
        }
        previousValue = self.isOn;
    }
}

@end

Мы игнорируем события, если значение совпадает с измененным значением. Поместите CustomSwitch во все классы UISwitch в раскадровке. Это решит проблему и вызовет цель только один раз при изменении значения.

кодер
источник
Это сработало для меня. Он идеален в том смысле, что не требует ручного добавления постороннего кода в ваши файлы реализации, поскольку он может использоваться повторно и скрывает свою реализацию внутри реализации класса. Это просто хороший дизайн. Было бы неплохо, если бы в этом ответе было больше комментариев, потому что я действительно не понимаю, почему здесь весь код.
Джеймс С.
Спасибо, у меня сработало. Не могли бы вы немного подробнее объяснить код. @codester
Сваямбху
7

У меня много пользователей, которые сталкиваются с одной и той же проблемой, так что, возможно, это ошибка, UISwitchпоэтому я только что нашел ее временное решение. Я нашел одно gitHubиндивидуальное KLSwitchиспользование этого сейчас, надеюсь, что Apple исправит это в следующем обновлении xCode: -

https://github.com/KieranLafferty/KLSwitch

Нитин Гохель
источник
6

Если вам не нужно мгновенно реагировать на изменение значения переключателя, решением может быть следующее:

- (IBAction)switchChanged:(id)sender {
  [NSObject cancelPreviousPerformRequestsWithTarget:self];

  if ([switch isOn]) {
      [self performSelector:@selector(enable) withObject:nil afterDelay:2];
  } else {
      [self performSelector:@selector(disable) withObject:nil afterDelay:2];
  }
}
предварительно
источник
Работает как шарм в iOS 11.2. В моей ситуации он запустил 2 события подряд (состояние: выключено, смахивание: выключено, 1-е событие: включено, 2-е событие: выключено), поэтому задержки в 0,1 с достаточно для меня и незаметно для пользователя.
Павел
3

Эта проблема все еще существует в бета-версии iOS 9.3. Если вы не возражаете, что пользователь не может перетащить его за пределы переключателя, я считаю, что использование, .TouchUpInsideа не .ValueChangedработает надежно.

Ник Корн
источник
2

Я все еще сталкиваюсь с той же проблемой в iOS 9.2

У меня есть решение и я представляю, как это может помочь другим

  1. Создать переменную count для отслеживания количества вызовов метода

    int switchMethodCallCount = 0;
    
  2. Сохранить значение bool для значения переключателя

    bool isSwitchOn = No;
    
  3. В методе изменения значения Switch выполняйте действие желания только для первого вызова метода. Когда значение переключателя снова изменяет установленное значение счетчика и значение переменной bool на значение по умолчанию

    - (IBAction)frontCameraCaptureSwitchToggle:(id)sender {
    
    
    
    //This method will be called multiple times if user drags on Switch,
    //But desire action should be perform only on first call of this method
    
    
    //1. 'switchMethodCallCount' variable is maintain to check number of calles to method,
    //2. Action is peform for 'switchMethodCallCount = 1' i.e first call
    //3. When switch value change to another state, 'switchMethodCallCount' is reset and desire action perform
    
    switchMethodCallCount++ ;
    
    //NSLog(@"Count --> %d", switchMethodCallCount);
    
    if (switchMethodCallCount == 1) {
    
    //NSLog(@"**************Perform Acction******************");
    
    isSwitchOn = frontCameraCaptureSwitch.on
    
    [self doStuff];
    
    }
    else
    {
    //NSLog(@"Do not perform");
    
    
    if (frontCameraCaptureSwitch.on != isSwitchOn) {
    
        switchMethodCallCount = 0;
    
        isSwitchOn = frontCameraCaptureSwitch.on
    
        //NSLog(@"Count again start");
    
        //call value change method again 
        [self frontCameraCaptureSwitchToggle:frontCameraCaptureSwitch];
    
    
        }
    }
    
    
    }
    
Четан Коли
источник
Я тоже все еще сталкиваюсь с этой проблемой в версии 9.2. Я реализовал вашу логику, и теперь она работает так, как я задумал.
Ник Корн
2

Эта проблема меня беспокоит, когда я привязываю переключатель к другому поведению. Обычно вещи не переходят от onк on. Вот мое простое решение:

@interface MyView : UIView
@parameter (assign) BOOL lastSwitchState;
@parameter (strong) IBOutlet UISwitch *mySwitch;
@end

@implementation MyView

// Standard stuff goes here

- (void)mySetupMethodThatsCalledWhenever
{
    [self.mySwitch addTarget:self action:@selector(switchToggled:) forControlEvents:UIControlEventValueChanged];
}

- (void)switchToggled:(UISwitch *)someSwitch
{
    BOOL newSwitchState = self.mySwitch.on;
    if (newSwitchState == self.lastSwitchState)
    {
        return;
    }
    self.lastSwitchState = newSwitchState;

    // Do your thing
}

Только не забудьте также установить self.lastSwitchStateкаждый раз, когда вы вручную меняете mySwitch.on! :)

Бен Легжеро
источник
1

Этот тип проблемы часто вызывается ValueChanged. Вам не нужно нажимать кнопку для запуска функции. Это не прикосновение. Каждый раз, когда вы программно переключаете переключатель в положение включения / выключения, значение изменяется, и функция IBAction снова вызывается.

@RoNiT дал правильный ответ:

Swift

func doToggle(switch: UISwitch) {
    if switch.on && !switch.selected {
        switch.selected = true
        // SWITCH ACTUALLY CHANGED -- DO SOMETHING HERE
    } else {
        switch.selected = false
    }
}
Дэйв Джи
источник
0
DispatchQueue.main.async {
        self.mySwitch.setOn(false, animated: true)
    }

Это работает нормально и больше не вызывает функцию селектора.

анураагджайн
источник
0

Это сработало для меня.

DispatchQueue.main.asyncAfter(deadline: DispatchTime.now()){
    self.switch.isOn = true
}
user3305074
источник