renderer Method re-ogranization. Refactoring

[atherdon]

1. it has a strange name too :)

Our main goal here - to discuss how we can improve this function.
But firstly - please explain what exactly this function doing and why we put there a lot of data...

link to our chat: #48 (comment)

First of all - i can guess that this function is heavy - so the easiest way - to split it into a few functions. Second thing - we have a 3 params, and they are long. i mean - i can bet that we have a lot of similar data inside this params. let's make this a separated task.

[vadim9999]

@atherdon I think we can creacte a class that will contains mathods from Method.js. And creacte some setters to give data in class.

[atherdon]

ok, but i want to have here more details. like a step-by-step plan about how it can be improved. Not sure if'm 100% ok with idea of making class, but let's explore that way anyway...

[vadim9999]

We can add in dishByDay array with ingredients

[atherdon]

i should have more details.
i'm not very into this project. so i'm don't have clear understanding what exactly that methods doing. Please give more details, with codelinks maybe

[vadim9999]

method renderFunc() render a dish in one day and display modal with ingredients when you will click on a tile. Component ColumnRender specify what day should display renderFunc

[atherdon]

if we have 5 recipes - we also have 5 modals at our HTML page with different lists, or we have one modal - that refresh content?

…

On Sun, Oct 21, 2018 at 7:33 PM Vadim ***@***.***> wrote: method renderFunc() render a dish in one day and display modal with ingredients when you will click on a tile — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#50 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABZrDjoqz8tVjPmNNtMysYiolbCAgAmWks5unKHIgaJpZM4Xx6JU> .

[vadim9999]

We have 5 modules

[atherdon]

this thing is maybe not related to our current task, but we should think about it too. I'm still cannot figure out how to improve this method, without looking at it's code. Can you guide me somehow, so i can easily figure it out?

…

On Sun, Oct 21, 2018 at 8:05 PM Vadim ***@***.***> wrote: We have 5 modules — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#50 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABZrDmOHZqAfY8LPHYx-t3b-1sKFeW2Mks5unKlDgaJpZM4Xx6JU> .

[vadim9999]

rendererFunc() {
    let buffer = this.getOffset(this.state.dishByDay);

function getOffset calculate and return object with position for one column. (in code it's buffer)
For example in buffer

height: 150
start: 600
tag: "dinner"
text: "Tilapia with Corn on the Cob, Couscous and Pears"
time: "18:00-19:30"

Next create Modals that have ingredients

return <Modals mode={this.state.mode} data={buffer[j - 1]} key={item} ingredient={this.state.ingredient}/>;

In Modals we give name of meal, time, and ingredients. Modals is a tile. When you click it, open form with ingredients. In this way 5 tiles are 5 Modals in calendar.

[atherdon]

yep, this is what i assumed. Ok, no problem - we'll change it