Új hozzászólás Aktív témák

• martonx veterán

  #3248

  #3247 Sk8erPeter

  2013-02-12 08:40:40

  Új Válasz

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  

  martonx

  veterán

  #3247 Sk8erPeter

  #3247 Sk8erPeter

  "csak igyekszek a S.O.L.I.D elvekből minimum a Dry-t betartani."
  De melyik a Dry a S.O.L.I.D.-ban?

  http://en.wikipedia.org/wiki/SOLID_(object-oriented_design)
  "Single responsibility, Open-closed, Liskov substitution, Interface segregation and Dependency inversion"

  De Don't Repeat Yourself (DRY) is van:
  http://en.wikipedia.org/wiki/Don't_repeat_yourself
  Gondolom erre gondoltál.

  Most ezt lehetne függvénybe rakni, amit Trisztán írt, hogy mondjuk a
  {
  mode: "javascript",
  theme: "default",
  lineNumbers: true
  }
  objektum legyen a default, de ebben az esetben van értelme? Ilyen az API, a readOnly-t még hozzá kell csapni, és két elem miatt írtál egy függvényt, ami aztán a CodeMirror API-t használja fel, és ki tudja, lehet, hogy később mondjuk rájössz, hogy az adott kódrészletnél nem is JavaScript-kiemelést szeretnél használni, hanem mondjuk PHP-set. Igaz, rakhatod a paraméterek passzolgatását függvénybe, de nem érzem, hogy azzal előrébb lennél, csak egy wrappert húznál kb. azonos mennyiségű kód köré, ami meg viszont már tök felesleges kódbázis-növelés.

  Bocs, akkor a D nem a DRY-t jelenti , de ettől még tarthatjuk magunkat hozzá.

  Lovagolhatunk azon, hogy mennyi karaktert spórolunk meg, de ettől még bizonyos elvekhez érdemes tartani magunkat, és szerintem már megérte, hogy felhoztam a témát, mert legalább beszélünk róla. Valahogy így csinálnám:

  $(document).ready(function(){
  
  // ... sok más jquery
  
  var UploadPageCodeMirror = cdmfromtexta('CodeSnippetBody', 'javascript', false);
  
  window.onload = function() {
  var ReadOnlyCodeMirror = cdmfromtexta(''CodeSnippetBodyReadOnly', 'javascript', true);
  }
  
  function cdmfromtexta(selector, mode, readonly){
  CodeMirror.fromTextArea(document.getElementById(selector), {
  mode: mode,
  theme: "default",
  lineNumbers: true,
  readOnly: readonly
  });
  }

Új hozzászólás Aktív témák