floomm / ghidrion

2 stars 0 forks source link

Loading init trace file #19

Closed floomm closed 1 year ago

floomm commented 1 year ago

Closes #5

I would like to discuss how the MVC pattern is currently implemented @riesentoaster. I'm more of a fan of the "fat controller" approach, where the view doesn't know the model directly. See below:

image

Right now, the TraceFileController is rather a static helper class. But I haven't changed anything yet, since I wanted to hear your opinion first. That's why I have just put everything into the CreateTraceFilePanel class for now.

floomm commented 1 year ago

The YamlToTraceFileConverter has two static public methods: