1. Java / Говнокод #6658

    +147

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (nn.isleaf) {
          this.traverse_ssector((BSPLeaf) nn);
          return;
        }
        final BSPBranch n = (BSPBranch) nn;

    правильное наследование:
    в родительском классе BSPNode определено поле isleaf
    в конструкторе класса BSPLeaf оно выставляется true и никогда не меняется
    в конструкторе класса BSPBranch соответственно false

    а ведь так просто сделать проверку instanceof

    Запостил: Lure Of Chaos, 14 Мая 2011

    Комментарии (7) RSS

    Добавить комментарий