Reactivecocoa: '액션'에 대한 제어 바인딩입니다.

동등한 기능을 위해서는 다음도 필요하다는 것을 잊지 마십시오.

confirmButton.rac.isEnabled <~ commitAction.isEnabled

흠, 나는 그것을 간과했다. 그러면 Action 로 올인해야 할 것 같습니다.

@andersio 그게 무슨 말인지, 왜 이 문제가 종료되었는지 잘 모르겠습니다.

@sharplet이 제기하는 isEnabled 입력으로 값 전파를 분리하는 것이 좋다고 생각합니다.

그 사실을 편리한 방법/패턴에 숨기는 것은 제 생각에 우리에게 그다지 도움이 되지 않습니다.

https://github.com/ReactiveCocoa/ReactiveCocoa/pull/3210/commits/f983a746f915a9a517e9e076c19581724075c562 에서 구현되었습니다

강한 참조와 함께 액션을 소유하는 뷰를 갖는 것은 잘못된 것 같습니다. 작업의 소유권(CocoaAction을 통해 간접적으로)은 원래 레이어(VC 또는 VM)에 남겨두어야 합니다.

동의하지 않는 것 같아요. 그게 왜 당신에게 잘못 느껴지나요?

약한 참조는 제안된 API의 작은 비트로, 명령 패턴을 사용하여 모든 컨트롤을 포함하는 단일 세트의 UIControl Action 합니다.

Rex에서는 특정 구현이 포함된 UIButton 전용이었습니다.

Action AFAIK는 일반적으로 명령을 나타내고 뷰 모델이 소유하며 해당 상태에 대한 내부 및 외부 종속성을 모두 가질 수 있습니다. Cocoa target-action 패턴과 마찬가지로 명백한 소유자가 있을 때 강력한 참조가 되어야 하는 이유를 모르겠습니다.

강력한 참조를 사용하는 Rex API는 CocoaAction 작동 방식의 인공물일 가능성이 높으며, 값이 래핑된 Action 전파될 수 있도록 유지해야 합니다.

여기에 제안된 API는 대신 Action 직접 사용합니다.

렉스 0.12:

extension UIButton {
    public var rex_pressed: MutableProperty<CocoaAction> { get }

    // This one would become `BindingTarget`. Irrelevant to this issue.
    public var rex_title: MutableProperty<String> { get }
}

제안됨(모든 UIControl ):

extension Reactive where Base: UIControl {
    /// The action is weakly referenced internally.
    public func setAction<Input, Output, Error>(
        _ action: Action<Input, Output, Error>,
        for controlEvents: UIControlEvents,
        inputTransform: (Self, UIControlEvents) -> Input
    )

    public func removeAction()

    /// ... some convenience overloads of `setAction(_:for:inputTransform:)`.
}

일반적으로 우리는 세 가지 UI 상태 클래스와 Action 명령 패턴을 위한 클래스가 있습니다.

1. 관찰할 수단이나 값이 없는 변경 가능한 키입니다.
   예: UIControl.isEnabled , UILabel.text , UIView.isHidden .

BindingTarget s로 모델링합니다.

1. 이벤트 및 메서드 호출 알림을 제어합니다.
   예: UIControl.trigger(for: .valueChanged) , UITableViewCell.trigger(for: #selector(prepareForReuse))

Signal s로 모델링합니다.

1. 변경 가능한 키 + 사용자 상호 작용 시 게시되는 해당 제어 이벤트.
   예: UITextField.text , NSTextField.text , UISwitch.isOn

#3229

1. [ _ Action 를 컨트롤과 연결합니다.
   (이것은 [1]과 [2] 위에 빌드된 편의 패턴일 뿐입니다.)
   예: UIButton.rac_command

_이 문제. _

@andersio 저는 불행히도 removeAction() 이 작업을 제거하려는 제어 이벤트를 지정해야 한다고 생각합니다. _또는_ 1.0의 경우 이름을 removeAllActions() 바꾸고 setAction() 는 컨트롤이 현재 이 "간편한 API"를 사용하여 연결되는 단일 작업만 처리함을 나타냅니다. 어쨌든 99.9%의 사람들이 이를 사용하기 때문입니다.

그것은 논쟁의 여지가 있습니다. 이 경우 여러 작업이 isEnabled 동시에 조작할 수 있기 때문에 IMO 여러 작업이 의미가 없을 수 있습니다(= 다소 불확실함).

주어진 작업을 직접 설정하는 (Swift) 속성을 추가하지 않는 이유는 무엇입니까?

extension Reactive where Base: UIButton {
    var pressed: CocoaAction {
        get { … }
        set { … }
    } 
}

아니면 some convenience overloads 섹션에 포함되었을 수 있습니까?

제네릭 API를 추가하고 싶다면 괜찮을 것 같지만, 편의성 API가 더 중요하다고 생각합니다. 그것이 제가 세상에 자랑하고 싶은 API입니다. 나는 다음과 같은 것이 매우 설득력 있다고 생각합니다.

button.reactive.pressed = CocoaAction(viewModel.action)

setAction(…) 를 사용하는 한 제거는 nil 작업을 설정하여 수행해야 합니다. 별도의 API가 필요하지 않습니다.

의심할 여지 없이 이게 더 좋아 보입니다. 이것들은 반드시 지켜져야 합니다. 나는 여전히 CocoaAction 가 Action 약한 참조를 갖도록 제안할 것입니다.

setAction 가 업데이트되고( 790b3e8 ) pressed 가 다시 돌아왔습니다( 790b3e8 ).

extension Reactive where Base: UIButton {
    public var pressed: CocoaAction<Base> { get nonmutating set }
}

extension Reactive where Base: UIControl {
    public var action: CocoaAction<Base>? { get }
    public var controlEventsForAction: UIControlEvents? { get }

    /// `CocoaAction` is retained by `self`, but `CocoaAction` weakly references `Action`.
    public func setAction(
        _ action: CocoaAction<Base>?,
        for controlEvents: UIControlEvents
    )
}

API가 여러 작업을 지원해야 한다고 생각합니다. 따라서 이것은 유효해야 합니다.

button.reactive.setAction(action1, for: .touchUpInside)
button.reactive.setAction(action2, for: .touchUpOutside)
// button now has 2 actions set for 2 different events

이유는 무엇입니까 CocoaAction 약하게 참조 Action ?

@mdiep

// button now has 2 actions set for 2 different events

이는 button 의 활성화 상태가 action1.isEnabled 및 action2.isEnabled 모두에 바인딩됨을 의미합니다. 이것이 우리가 원하는 것인지 잘 모르겠습니다. 활성화 상태는 어떻게 결정해야 합니까? 모든 Action 의 활성화 상태 중 AND ?

CocoaAction이 Action을 약하게 참조하는 이유는 무엇입니까?

되돌렸습니다. 나는 그것이 지금 (그리고 Rex의 역사 동안) 문제를 일으키지 않는다는 것을 인정할 것입니다.

이는 버튼의 활성화 상태가 action1.isEnabled 및 action2.isEnabled 둘 다에 바인딩됨을 의미합니다. 이것이 우리가 원하는 것인지 잘 모르겠습니다. 활성화 상태는 어떻게 결정해야 합니까? 모든 작업의 ​​활성화 상태의 AND?

그리고 나에게 옳지 않은 것 같습니다.

옵션은 다음과 같습니다.

1. 활성화를 완전히 포기하십시오
2. AND 모든 작업의 ​​활성화 상태
3. 또는 모든 작업의 ​​활성화 상태
4. 정확히 1개의 작업이 있는 경우 활성화
5. .pressed 등에서 활성화하고 setAction(:for:) 직접 활성화하지 마십시오.

(5) 나에게 가장 좋은 해결책인 것 같다. 일반 케이스는 쉽고 편리합니다. 더 복잡한 것을 하고 싶다면 여전히 할 수 있습니다.

(5) 는 setAction 가 여러 actionN <~ reactive.trigger(for: .specificEvent) 를 수행하는 것과 차이가 없음을 의미합니다. 또한 컨트롤에 여러 편의 명령 슬롯이 포함되어 있으면 어떻게 됩니까?

다음과 같은 점에서 여전히 다릅니다.

1. 취소가 다릅니다
2. 이벤트당 하나의 작업만 허용합니다.
3. CocoaAction 대한 입력으로 컨트롤을 전달합니다.

그렇지 않으면 거의 동일합니다. 하지만 괜찮다고 생각합니다.

계획이 둘 이상의 작업을 지원하는 것이라면 이름을 addAction 변경해야 합니다.

propagateEnablingState 인수를 setAction 하면 isEnabled 의 벡터를 처리하는 방법을 결정해야 합니다. (구성 가능? 예: 및/또는 정의되지 않음?)

여기에서 논의되는 "편의성 속성"에서 1개 이상의 작업을 지원할 가치가 있는지 잘 모르겠습니다. 둘 이상의 작업이 필요한 사람들을 위해 대체 사용법( trigger(for:) 작업을 개별적으로 트리거)을 남겨두었기 때문입니다.

사용하기 쉬운 대안(다른 Signal/BindingTarget 항목의 사용과 일치하는 경우도 있음)을 감안할 때 이 방식으로 버튼에 둘 이상의 작업을 추가해야 하는 타당한 이유를 볼 수 없습니다.

기본적으로 "간단한 API"를 원하면 여기와 같이 action 속성을 사용합니다. 활성화된 상태는 버튼에 반영됩니다.

>1 작업에 버튼을 연결하려면 개별 trigger 를 사용한 다음 isEnabled BindingTarget 에 연결하기 위해 자신의 map 를 연결합니다. BindingTarget . 합리적으로 들리나요?

기본적으로 "간단한 API"를 원하면 여기와 같이 action 속성을 사용합니다. 활성화된 상태는 버튼에 반영됩니다.

>1 작업에 버튼을 연결하려면 개별 트리거를 사용하고 고유한 맵을 연결하여 isEnabled BindingTarget에 연결합니다. 합리적으로 들리나요?

그것은 나에게 합리적으로 들린다.

그렇다면 setAction(:for:) 가 필요합니까?

그러면 숨길 것입니다. 간단하게 유지하세요.