5 ms·

I have never found "X lines of code" to be impressive. Write clean code and polish it and make it readable. Who cares how long it is. Hackers brag about lines,

I have never found "X lines of code" to be impressive. Write clean code and polish it and make it readable. Who cares how long it is. Hackers brag about lines, not engineers.

gkoberger 11y ago
The actual number doesn't matter. It's just shorthand for "It's easy and quick! This isn't a mammoth 10k-line library".
- mbreese 11y ago
  It's also shorthand for "doesn't have feature X or support Y and hasn't really been tested!"
diggan 11y ago
Well, could be a reason this place is called Hackers News rather than Engineers News
georgestephanis 11y ago
In only 100 lines of PHP! Requires Symfony.
- MichaelApproved 11y ago
  Requiring libraries/frameworks shouldn't detract from something like this. I'd be more worried about bugs and feature support.
  - ihsw 11y ago
    Nah, requiring Symfony is like requiring Django. Doesn't make much sense.
    - deleted 11y ago
      [deleted]
    - mod 11y ago
      It's actually Silex. Maybe that makes more sense.
      - aikah 11y ago
        Silex is hardly a "micro framework", it's basically Symfony without the configuration files. So it's a LOT of code. I like it by the way, always hated the Spring like insanity of Symfony, but let's not pretend Silex + all Symfony libs the framework has to import are small. The irony is that for this very project, Both Silex and Symfony\Process are completely redundant, especially with PHP bloated core APIs.
        Gigablah 11y ago
        That's not correct. Silex replaces the Symfony DI (or service locator, if you will) component with something much simpler called Pimple. You can read the code within a few minutes. The only other required libraries for Silex to function are the event dispatcher, routing (which you can swap out for something like nikic/FastRoute), and the request/response stuff. I'm not sure if you think that's a "lot" of code, but if you're rolling your own you'd probably end up using the HTTPFoundation stuff anyway.
        aikah 11y ago
        > That's not correct. Silex replaces the Symfony DI (or service locator, if you will) component with something much simpler called Pimple. You can read the code within a few minutes. That's exactly what I said, what is the point of your message? With Silex there is no configuration through XML or YAML. What is not correct?
        Gigablah 11y ago
        You ranted about it being a "lot" of code. The default Silex installation doesn't even bring in the Security component, which is probably what prompted your comparison to Spring. Event dispatching + kernel (basically a handler) + request/response isn't particularly complex, is it?
- dangrossman 11y ago
  It requires Silex, a micro-framework, not Symfony. Symfony is a much larger framework that isn't required. He just used two of its modular components, Request for HTTP request parsing, and Process for spawning subprocesses.
- mschuster91 11y ago
  > Requires Symfony. Silex, not Symfony, and better to rely on trusted library code than to reinvent the wheel and introduce vulnerabilities...
  - Killswitch 11y ago
    But it's not 100 LOC if you don't reinvent the wheel! /s
mmanfrin 11y ago
People like to specialize and seek extremes of different metrics. Do you think that the demoscene is not impressive because they could do much more with more than 4k of code? This comment is an example of the type of negativity that is unhelpful and creates the toxic atmosphere HN is renown for, and I believe you know that considering you posted with a throwaway.

I have never found "X lines of code" to be impressive. Write clean code and polish it and make it readable. Who cares how long it is. Hackers brag about lines, not engineers.

gkoberger 11y ago

The actual number doesn't matter. It's just shorthand for "It's easy and quick! This isn't a mammoth 10k-line library".

mbreese 11y ago
It's also shorthand for "doesn't have feature X or support Y and hasn't really been tested!"

diggan 11y ago

Well, could be a reason this place is called Hackers News rather than Engineers News

georgestephanis 11y ago

In only 100 lines of PHP! Requires Symfony.

MichaelApproved 11y ago
Requiring libraries/frameworks shouldn't detract from something like this. I'd be more worried about bugs and feature support.
- ihsw 11y ago
  Nah, requiring Symfony is like requiring Django. Doesn't make much sense.
  - deleted 11y ago
    [deleted]
  - mod 11y ago
    It's actually Silex. Maybe that makes more sense.
    - aikah 11y ago
      Silex is hardly a "micro framework", it's basically Symfony without the configuration files. So it's a LOT of code. I like it by the way, always hated the Spring like insanity of Symfony, but let's not pretend Silex + all Symfony libs the framework has to import are small. The irony is that for this very project, Both Silex and Symfony\Process are completely redundant, especially with PHP bloated core APIs.
      - Gigablah 11y ago
        That's not correct. Silex replaces the Symfony DI (or service locator, if you will) component with something much simpler called Pimple. You can read the code within a few minutes. The only other required libraries for Silex to function are the event dispatcher, routing (which you can swap out for something like nikic/FastRoute), and the request/response stuff. I'm not sure if you think that's a "lot" of code, but if you're rolling your own you'd probably end up using the HTTPFoundation stuff anyway.
        aikah 11y ago
        > That's not correct. Silex replaces the Symfony DI (or service locator, if you will) component with something much simpler called Pimple. You can read the code within a few minutes. That's exactly what I said, what is the point of your message? With Silex there is no configuration through XML or YAML. What is not correct?
        Gigablah 11y ago
        You ranted about it being a "lot" of code. The default Silex installation doesn't even bring in the Security component, which is probably what prompted your comparison to Spring. Event dispatching + kernel (basically a handler) + request/response isn't particularly complex, is it?
dangrossman 11y ago
It requires Silex, a micro-framework, not Symfony. Symfony is a much larger framework that isn't required. He just used two of its modular components, Request for HTTP request parsing, and Process for spawning subprocesses.
mschuster91 11y ago
> Requires Symfony. Silex, not Symfony, and better to rely on trusted library code than to reinvent the wheel and introduce vulnerabilities...
- Killswitch 11y ago
  But it's not 100 LOC if you don't reinvent the wheel! /s

mmanfrin 11y ago

People like to specialize and seek extremes of different metrics. Do you think that the demoscene is not impressive because they could do much more with more than 4k of code? This comment is an example of the type of negativity that is unhelpful and creates the toxic atmosphere HN is renown for, and I believe you know that considering you posted with a throwaway.