interface-vs-implementation/ about dmitripavlutin.com-comments HOT 11 OPEN

I'd love to see an example of this concept without Classes...

from dmitripavlutin.com-comments.

Hi @panzerdp. Great Article as always :)

from dmitripavlutin.com-comments.

Although this example seems object-oriented, the principle and implementation reminds me of what I understand to be functional programming. Either way, very cool and great article!

from dmitripavlutin.com-comments.

Hi @panzerdp. Great Article as always :)

Thanks @taniot!

from dmitripavlutin.com-comments.

Awesome article! I really like your takes on software design, and I hope you will write more of them in the future :)

from dmitripavlutin.com-comments.

Awesome article! I really like your takes on software design, and I hope you will write more of them in the future :)

Thanks, @Jaanilj! Yes, I'm learning a lot about software design and will try to write more useful things in the future.

from dmitripavlutin.com-comments.

Good explanation. It seems the last version of the render method is a violation of the Single responsibility principle. Am I wrong?

from dmitripavlutin.com-comments.

Good explanation. It seems the last version of the render method is a violation of the Single responsibility principle. Am I wrong?

Nice catch @Red02Raccoon! Seems like the last version violates SRP because it renders the list, and also invokes the sorting.

from dmitripavlutin.com-comments.

It can also be understood as extracting reusable methods

from dmitripavlutin.com-comments.

Hi. I guess it's a wrong link "For example, if the ListRenderer will always sort the names alphabetically in ascending order: then stop at the design presented at the beginning of section 2. Do not add unnecessary complexity." => 2. Have a nice day :) Thanks

from dmitripavlutin.com-comments.

I see the last comment was made almost a year ago, but thought I'd add my 2 cents in case anybody is still interested.

@sosioo, here's one possible solution without classes:

const localeCompareAsc = (s1: string, s2: string) => s1.localeCompare(s2);
const localeCompareDesc = (s1: string, s2: string) => localeCompareAsc(s2, s1);
const renderListItem = (item: string) => `  <li>${item}</li>`;
const renderList = (items: string[]): string =>
  ["<ul>", ...items.map(renderListItem), "</ul>"].join("\n");

const ascending = false; // Perhaps obtained via system input
const comparator = ascending ? localeCompareAsc : localeCompareDesc;
const names = ["Catwoman", "Joker", "Batman"];
const renderedNames = renderList([...names].sort(comparator));

console.log(renderedNames);
// <ul>
//   <li>Joker</li>
//   <li>Catwoman</li>
//   <li>Batman</li>
// </ul>

This also addresses @Red02Raccoon's point about single responsibility because the renderList function only renders the list. Sorting is handled independently, before the list is passed to renderList.

from dmitripavlutin.com-comments.