<div dir="ltr">Már beláttam a dolgot. Sajnos abból indultam ki, hogy nem lehet nem default konstruktort hívni. De lehet. Szóval nincs szükség a sok kódra.</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013. január 31. 13:59 István Székely írta, <span dir="ltr">&lt;<a href="mailto:istvan.szekely@i-logic.hu" target="_blank">istvan.szekely@i-logic.hu</a>&gt;</span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <div>Szia!<br>
      <br>
      Hogy ne csak &quot;miért nem lehet konstruktor&quot; kérdést járjuk körbe,
      hanem az eredeti kérdésedre is legyen válasz, íme az én
      véleményem. (De ez csak szigorúan a magánvéleményem.)<br>
      <br>
      Igen, rossz vagy. Over-engineered. Minimum kétszer annyi kódot
      írsz, mint amit akkor kéne, ha elneveznéd az osztályodat (nested
      class). Gondolj arra, mi lesz, ha egy junior meglátja a kódot.
      Gondolj arra, hogy mi lesz, amikor te magad egy fél év múlva
      meglátod a kódot. Nálam a code review-n nem menne át.<br>
      <br>
      Üdv,<br>
      Stivi<br>
      <br>
      <br>
      On 2013-01-31 13:04, Gábor Lipták wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Sziasztok!
        <div><br>
        </div>
        <div>Szóval adott a névtelen osztály, aminek nem
          lehetnek konstruktor argumentumai. És adott egy abstract
          osztály amit névtelen osztályként akarok használni, ám muszály
          bizonyos dolgokat megkapnia a nem absztrakt metódusnak ahhoz,
          hogy működjön.</div>
        <div><br>
        </div>
        <div>Konkrét példa: ICheckStateProvider JFace-ben.
          Akarok egy implementációt ami a isGrayed metódusra fix
          megoldást ad, de az isChecked metódust szabadon hagyja. Ezt
          ötlöttem ki:</div>
        <div>
          <br>
        </div>
        <div>
          <div>public abstract class
            ContentBasedGrayingCheckStateProvider {</div>
          <div><span style="white-space:pre-wrap"> </span>protected
            InnerDelegate delegate = new InnerDelegate();</div>
          <div><span style="white-space:pre-wrap"> </span>protected
            ITreeContentProvider contentProvider;</div>
          <div><span style="white-space:pre-wrap"> </span></div>
          <div><span style="white-space:pre-wrap"> </span>public
            abstract boolean isChecked(Object element);</div>
          <div><br>
          </div>
          <div><span style="white-space:pre-wrap"> </span>public
            boolean isGrayed(Object element) {</div>
          <div><span style="white-space:pre-wrap"> </span>//Itt
            tudok valamit csinálni a contentProviderrel</div>
          <div><span style="white-space:pre-wrap"> </span>return
            false;</div>
          <div><span style="white-space:pre-wrap"> </span>}</div>
          <div><span style="white-space:pre-wrap"> </span></div>
          <div><span style="white-space:pre-wrap"> </span>public
            ICheckStateProvider withContentProvider(
            ITreeContentProvider contentProvider ) {</div>
          <div><span style="white-space:pre-wrap"> </span>this.contentProvider
            = contentProvider;</div>
          <div><span style="white-space:pre-wrap"> </span>return
            delegate;</div>
          <div><span style="white-space:pre-wrap"> </span>}</div>
          <div><br>
          </div>
          <div><span style="white-space:pre-wrap"> </span>protected
            class InnerDelegate implements ICheckStateProvider {</div>
          <div><br>
          </div>
          <div><span style="white-space:pre-wrap"> </span>@Override</div>
          <div><span style="white-space:pre-wrap"> </span>public
            boolean isChecked(Object element) {</div>
          <div><span style="white-space:pre-wrap"> </span>return
            ContentBasedGrayingCheckStateProvider.this.isChecked(element);</div>
          <div><span style="white-space:pre-wrap"> </span>}</div>
          <div><br>
          </div>
          <div><span style="white-space:pre-wrap"> </span>@Override</div>
          <div><span style="white-space:pre-wrap"> </span>public
            boolean isGrayed(Object element) {</div>
          <div><span style="white-space:pre-wrap"> </span>return
            ContentBasedGrayingCheckStateProvider.this.isGrayed(element);</div>
          <div><span style="white-space:pre-wrap"> </span>}</div>
          <div><span style="white-space:pre-wrap"> </span>}</div>
          <div>}</div>
          <div><br>
          </div>
          <div>Felhasználni így kell majd:</div>
          <div><br>
          </div>
          <div>
            <div>ITreeContentProvider cp = ...;</div>
            <div>new CheckboxTreeViewer(...).setCheckStateProvider( new
              ContentBasedGrayingCheckStateProvider() {</div>
            <div><span style="white-space:pre-wrap"> </span></div>
            <div><span style="white-space:pre-wrap"> </span>@Override</div>
            <div><span style="white-space:pre-wrap"> </span>public
              boolean isChecked(Object element) {</div>
            <div><span style="white-space:pre-wrap"> </span>// TODO
              Auto-generated method stub</div>
            <div><span style="white-space:pre-wrap"> </span>return
              false;</div>
            <div><span style="white-space:pre-wrap"> </span>}</div>
            <div><span style="white-space:pre-wrap"> </span>}.withContentProvider(cp));</div>
            <div><br>
            </div>
            <div>Mi a véleményetek? Nem akarok annyi nevesített
              osztályt gyártani. Rossz vagyok? Jobb ötletek van?</div>
            <div>
              <br>
            </div>
            <div>Gábor</div>
          </div>
        </div>
      </div>
      <br>
    </blockquote>
    <br>
  </div>

<br>_______________________________________________<br>
Javalist mailing list<br>
<a href="mailto:Javalist@lists.javaforum.hu">Javalist@lists.javaforum.hu</a><br>
<a href="http://lists.javaforum.hu/mailman/listinfo/javalist" target="_blank">http://lists.javaforum.hu/mailman/listinfo/javalist</a><br>
<br></blockquote></div><br></div>