in reply to Re^7: Speeding up RecDesent parser for Perl Code
in thread Speeding up RecDesent parser for Perl Code

Until now I allready have about 150 additional Perl Modules, not including the XML setup files, GUI Resource files, images, external scripts and templates. Surely I can and will refactor some functions but under 3000 lines is not possible for the main modul (about one third of the file are comments too). Funny thing is, that this "main modul" is only a plugin in a larger application (which is mostly empty by now).

Andre
  • Comment on Re^8: Speeding up RecDesent parser for Perl Code

Replies are listed 'Best First'.
Re^9: Speeding up RecDesent parser for Perl Code
by adrianh (Chancellor) on Sep 08, 2006 at 07:11 UTC
    Surely I can and will refactor some functions but under 3000 lines is not possible for the main modul (about one third of the file are comments too).

    3000 lines with no duplication? 3000 lines with no multiple responsibilities? I have to admit that I find this hard to believe. Main modules tend to be the smallest, rather than the largest bits of functionality in my experience - small wrappers around a bunch of objects/libraries that do the real work. Not being able to get something under 3000 lines smells of a design problem to me.

[reply]
      Yepp and it is small in comparison to the total size of the application (something between 100.000 and 150.000 lines, I guess).

      But I dont think that the quality of an design depend on the number of lines in a modul.

      Maybe this would be a nice poll: "What is a good design?" :-). It is one of those questions, that you can argue month about.
      For me 3000 lines are easy to maintain, but then this is only my opinion and everybody has its own.

      Humble blessings,

      Andre
[reply]
        But you *did* complain that it was slow to parse 3000 lines, so clearly it's not your preferred option.
[reply]

In Section Seekers of Perl Wisdom