Reducing Complexity Solution

[00:00:00]
>> Let's see the solution for this exercise. So, I'm gonna hop into the code and I'm going to start to Fix this monstrosity. Now, what I'm going to do as well, just as a reference. I'm going to look at the test. So I've gone ahead and I've kind of written test that should work, which in a way gives you an indication or a hint of what your code should look like.

[00:00:41]
And so you can see here just off right on the gate. You can see that we are looking for update widgets, get total price, add widget update widget and we have a pretty good idea of what we are looking for. So from here, let's go ahead and start to extract this.

[00:01:05]
So, what we need to do here is we'll go update widget. So this will be one method and then we'll do delete widget. And we're gonna go create widget. And then I believe as you can see here I already got this wrong, add widget. And there's one more I'm looking for get total price.

[00:01:44]
Add widget, get total price, Excel and then from here, what we can do is we can just copy this out and we can paste this in. Now, what I'm gonna do is a little bit of a kind of a mental thing is that in a method it's not always bad but when you find yourself referencing this dot something, just stop and check and see if you can possibly extract that out to a parameter.

[00:02:26]
And so in this case I can, so I don't need this. Instead, what I need to do is I need to go widgets, widget. All right, and so now this is a fairly pure function that takes a collection and return something and it's going to do the same thing every single time.

[00:02:47]
And from here, what we'll do is we're just going to go return this dot delete widget, and we'll go widgets, widget and so this is going to obviously break initially. But you can see here it's asking for widget. So let's update. Well, let's do these methods first and then we'll go through and continue.

[00:03:11]
I'm kind of coming from the bottom up, but you could also go from the top down. So from here to update, we're going to copy this right here. And We'll pass in widgets and widget. There we go. Clean this up. And now let's go to the next one.

[00:03:41]
And I can delete this instead of the return, return this dot update widget, widgets, widget. And from here, one more. And this one is a little funny in the sense of we're faking that we're pulling from a database. And so we're having to kind of hard code or not hardcore, but generate a UUID for it.

[00:04:19]
And so this is why you are seeing this little bit of gymnastics here, all right? And so now we have three methods that it takes widgets and it takes a widget and then it essentially performs that function and returns a new widgets collections. By the way, these are immutable operations.

[00:04:45]
We'll talk about these later. But I just wanted to point that out. And so from here, return this dot add widget, widget, widget. Okay, now let's go. Mode, widgets So we're gonna type this like so. And I believe mode is a string. There we go. And so now we are getting rid of mode here.

[00:05:28]
And let's also do this last one here Which will take just widgets. All right. So I've kind of broken it out but I still have the single responsibility principle violation here. And so let's see if I have any hints in here about what is going on. So here we have it, update widgets.

[00:06:15]
So let me go back to here and we're going to create a new method okay, widgets which takes Mode widgets and a widget. And I suppose I could have just renamed this one up here that, I wanted to show us actually moving this out. Oops, I'll paste this in here.

[00:06:49]
And then let's just pick this up. And we'll go here. So now we'll say this widgets equals this update widgets. Mode, widgets, widget, Widgets. There we go. And so just to walk through this real quick, I think I've done everything correctly. Other than I'm missing a mock widget.

[00:07:36]
No problem. You can copy this here, but let's actually let's look at the code. Let's run the test. I think we might have one little tiny issue. But we can check that. So notice in the recalculate total that I've extracted all of that functionality out into two methods.

[00:07:58]
And then I've taken the nested logic. I've extracted those out into standalone methods. And then from there, I've abstracted or extracted the mode from internal state or hidden state to a parameter that we provide via dependency injection. So let me save this. And let's go into the command line.

[00:08:24]
And let's run this. And let me real quick, let me make this bigger, there we go. So let's run this and see what happens. One failed, so right here, we need to make this a probably a mock widget. So let's hop back in our code. I thought that was going to happen.

[00:08:54]
Let's see if we can fix this. I'm just going to copy this. So we have a mock widget. Where is this angry? We'll just go here. So I've just defined a mock widget. So this is just me getting this test working. A small little hiccup and it's because I have strongly typed This, all right.

[00:09:47]
I'm not seeing any more. Let me set this right here. Save this. And so all I did is before I had not typed that as a widget and the tests were able to pass but since I went ahead and strongly typed it then it's got a little angry at me.

[00:10:13]
So back here, let's see if we can get this working. Every time I run unitest I start making deals with God like if this just passed I'll never eat another carbohydrate again, I promise to be good. But looks like I did eat pizza for lunch, but I'm glad my test pass.

[00:10:34]
All right.