Create an account

Very important

  • To access the important data of the forums, you must be active in each forum and especially in the leaks and database leaks section, send data and after sending the data and activity, data and important content will be opened and visible for you.
  • You will only see chat messages from people who are at or below your level.
  • More than 500,000 database leaks and millions of account leaks are waiting for you, so access and view with more activity.
  • Many important data are inactive and inaccessible for you, so open them with activity. (This will be done automatically)


Thread Rating:
  • 645 Vote(s) - 3.42 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Dismissing a Presented View Controller

#1
I have a theoretic question. Now İ'm reading Apple's [ViewController][1] guide.

They wrote:

> When it comes time to dismiss a presented view controller, the
> preferred approach is to let the presenting view controller dismiss
> it. In other words, whenever possible, the same view controller that
> presented the view controller should also take responsibility for
> dismissing it. Although there are several techniques for notifying the
> presenting view controller that its presented view controller should
> be dismissed, the preferred technique is delegation.

But I can't explain, why I have to create a protocol in presented VC and add delegate varible, create delegate method in presenting VC for dismissing presented VC, instead of a simple call in **presented** view controller method

`[self dismissViewControllerAnimated:NO completion:nil]`?

Why is the first choice better? Why does Apple recommend it?


[1]:

[To see links please register here]

Reply

#2
In my experience, it comes in handy when you need to dismiss it from ***any*** ViewController you want and perform different tasks for each viewcontroller that dismisses it. Any viewController that adopts the protocol can dismiss the view in it's own way. (ipad vs iphone, or passing different data when dismissing from different views, calling different methods when dismissing, etc..)

Edit:

So, to clarify, if all you ever want to do is dismiss the view, I see no need to setup the delegate protocol. If you need to do different things ***after*** you dismiss it from different presenting view controllers, It would be your best way to go using the delegate.
Reply

#3
This is for view controller reusability.

Your view controller shouldn't care if it is being presented as a modal, pushed on a navigation controller, or whatever. If your view controller dismisses itself, then you're assuming it is being presented modally. You won't be able to push that view controller onto a navigation controller.

By implementing a protocol, you let the parent view controller decide how it should be presented/pushed and dismissed/popped.
Reply

#4
If you are using modal use view dismiss.

[self dismissViewControllerAnimated:NO completion:nil];
Reply

#5
Quote from View [Controller Programming Guide][1], "How View Controllers Present Other View Controllers".

> Each view controller in a chain of presented view controllers has
> pointers to the other objects surrounding it in the chain. In other
> words, a presented view controller that presents another view
> controller has valid objects in both its presentingViewController and
> presentedViewController properties. You can use these relationships to
> trace through the chain of view controllers as needed. **For example, if
> the user cancels the current operation, you can remove all objects in
> the chain by dismissing the first presented view controller.
> Dismissing a view controller dismisses not only that view controller
> but also any view controllers it presented.**

So on one hand it makes for a nice balanced design, good de-coupling, etc... But on the other hand it's very practical, because you can quickly get back to a certain point in navigation.

Although, I personally would rather use *unwinding segues* than try to traverse backwards the *presenting view controllers* tree, which is what Apple talks about in this chapter where the quote is from.

[1]:

[To see links please register here]

Reply

#6
One point is that this is a good coding approach. It satisfies many `OOP` principles, eg., SRP, Separation of concerns etc.

So, the view controller presenting the view should be the one dismissing it.

Like, a real estate company who gives a house on rent should be the authority to take it back.
Reply

#7
*Updated for Swift 3*

I came here just wanting to dismiss the current (presented) View Controller. I'm making this answer for anyone coming here with the same purpose.

# Navigation Controller

If you are using a navigation controller, then it is quite easy.


**Go back to the previous view controller:**

// Swift
self.navigationController?.popViewController(animated: true)

// Objective-C
[self.navigationController popViewControllerAnimated:YES];

**Go back to the root view controller:**

// Swift
self.navigationController?.popToRootViewController(animated: true)

// Objective-C
[self.navigationController popToRootViewControllerAnimated:YES];

(Thanks to [this answer][1] for the Objective-C.)

# Modal View Controller

When a View Controller is presented modally, you can dismiss it (from the second view controller) by calling

// Swift
self.dismiss(animated: true, completion: nil)

// Objective-C
[self dismissViewControllerAnimated:YES completion:nil];

The [documentation][2] says,

> The presenting view controller is responsible for dismissing the view
> controller it presented. If you call this method on the presented view
> controller itself, UIKit asks the presenting view controller to handle
> the dismissal.

So it works for the presented view controller to call it on itself. [Here][3] is a full example.

# Delegates

The OP's question was about the complexity of using delegates to dismiss a view.

- [This Objective-C answer][4] goes into it quite a bit.
- [Here][5] is a Swift example.

To this point I have not needed to use delegates since I usually have a navigation controller or modal view controllers, but if I do need to use [the delegate pattern][6] in the future, I will add an update.


[1]:

[To see links please register here]

[2]:

[To see links please register here]

:
[3]:

[To see links please register here]

[4]:

[To see links please register here]

[5]:

[To see links please register here]

[6]:

[To see links please register here]

Reply

#8
This is a lot of baloney. Delegation is fine when it is needed but if it makes the code more complex -- and it does -- then there needs to be a reason for it.

I'm sure Apple has its reasons. But it is clearer and more concise to simply have the presented VC do the dismiss unless there is a true reason for doing otherwise and no one here as of today has presented one that I can see.

Protocols are excellent when they're needed but object oriented design was never about having modules communicating unnecessarily with each other.

Tom Love (co-developer of Objective C) once commented that Objective C was "elegant", "small", "crisp" and "well-defined" (when comparing with C++). Easy for him to say. Delegation is a useful feature that seems to have been over-used "just because", and while I like working in the language, I dread the idea of felling compelled to use unnecessary syntax to make things more complex than they have to be.
Reply

#9
try this:

[self dismissViewControllerAnimated:true completion:nil];
Reply

#10
**Swift 3.0**
//Dismiss View Controller in swift

self.navigationController?.popViewController(animated: true)
dismiss(animated: true, completion: nil)
Reply



Forum Jump:


Users browsing this thread:
1 Guest(s)

©0Day  2016 - 2023 | All Rights Reserved.  Made with    for the community. Connected through