It is. Extracting methods, and replacing the 3-5 line code with a meaningfully named function results later in much easier (and less) readability when we're looking for errors. But probably the biggest achievement is found when he makes members private and creates constructors for some of the objects and this place: ruclips.net/video/kO3LJGbQ3JE/видео.html - this part is extremely important, as until NumberOfActiveOffers is public and changeable from anywhere in the code, it's too easy for someone to mess up that data (same for the read-only list of AssignedOffers).
Do you think this code gets easer to read and will run faster, after changes made 19:35 and forward?
It is. Extracting methods, and replacing the 3-5 line code with a meaningfully named function results later in much easier (and less) readability when we're looking for errors.
But probably the biggest achievement is found when he makes members private and creates constructors for some of the objects and this place: ruclips.net/video/kO3LJGbQ3JE/видео.html - this part is extremely important, as until NumberOfActiveOffers is public and changeable from anywhere in the code, it's too easy for someone to mess up that data (same for the read-only list of AssignedOffers).
@@balazs.hideghety that is done till 19:35. Further it gets more difficult to read.