Δυαδική οπτικοποίηση Αναζήτηση δέντρο σε Java

Question

Δυαδική οπτικοποίηση Αναζήτηση δέντρο σε Java

ψήφοι

1

Γεια σου Είμαι σήμερα κάνει το στάδιο των δοκιμών του έργου μου (Αλγόριθμος Visualization Tool). Έχω πάρει ένα πρόβλημα με τη διαγραφή μέθοδο της BST μου.

 public boolean delete(String key) {
boolean deleted = true;
boolean finished=false;
BNode current = root;
BNode prev = null;
while (!finished) {
  if (key.compareTo(current.key) > 0) {
    prev = current;
    current = current.right;
    this.repaint();
  }
  else if (key.compareTo(current.key) < 0) {
    prev = current;
    current = current.left;
    this.repaint();
  }
  else if (key.compareTo(current.key) == 0) {
      finished=true;
      this.repaint();
  }

}

if (check(current) == 0) {
    if(current==root)
    {
        root=null;
        xPos=400;
        yPos=60;
        this.repaint();
    }
    else
    {
        if (current.key.compareTo(prev.key) > 0) {
            prev.right = null;
            this.repaint();
        }
        else if(current.key.compareTo(prev.key) < 0) {
            prev.left = null;
            this.repaint();
        }
    }

}
else if (check(current) == 1) {
    if(current==root)
    {
        prev=current;
        if (current.left != null) {
            current=current.left;
            prev.key=current.key;
            prev.left = current.left;
            this.repaint();
        }
        else {
            current=current.right;
            prev.key=current.key;
            prev.right = current.right;
            this.repaint();
        }
    }
    else
    {

    if (current.key.compareTo(prev.key) > 0) {
    if (current.left != null) {
      prev.right = current.left;
      this.repaint();
    }
    else {
      prev.right = current.right;
      this.repaint();
    }
  }
  else if(current.key.compareTo(prev.key) < 0) {
    if (current.left != null) {
      prev.left = current.left;
      this.repaint();
    }
    else {
      prev.left = current.right;
      this.repaint();
    }
  }
    }
}
else if (check(current) == 2) {
  BNode temp = inord(current);
  if(current==root)
  {
      root.key=temp.key;
      this.repaint();
  }
  else
  {

      if (current.key.compareTo(prev.key) > 0) {
      prev.right.key = temp.key;
      this.repaint();
    }
    else {
      prev.left.key = temp.key;
      this.repaint(0);
    }
    }
}

return deleted;}

Ο κώδικας για την ίδια BST κατηγορία είναι πολύ μεγαλύτερη. Τα πάντα λειτουργούν καλά εκτός από το ότι όταν προσπαθώ να διαγράψω ένα κόμβο χωρίς παιδί, έχω μια εξαίρεση nullpointer όταν χρησιμοποιώ για παράδειγμα, 9 και 10 ως είσοδος (προσπαθήστε να ντελ 10) ή 5 και 12 (προσπαθήστε να ντελ 12), αλλά ποτέ δεν αν χρήστη, για παράδειγμα 4 και 8 (προσπαθήστε να ντελ 8) ή 9, 6 και 5. νομίζω ότι το πρόβλημα είναι με compareTo.

int check(BNode a) {
int ret;
if ( (a.left != null) && (a.right != null)) {
  ret = 2;
}
else if ( (a.left == null) && (a.right == null)) {
  ret = 0;
}
else {
  ret = 1;
}
return ret;}

Θα πρέπει πραγματικά να βοηθήσει με this.I μπορείτε να δημοσιεύσετε ολόκληρη την τάξη, αν χρειάζεται να .. Σας ευχαριστούμε!

java swing visualization binary-search-tree

Δημοσιεύθηκε 24/03/2011 στις 17:42
πηγή χρήστη user630167

Σε άλλες γλώσσες...

1 απαντήσεις

Thomas · Accepted Answer · 2011-03-24 18:04

Μόλις μερικές σημειώσεις:

Αν περνάτε null να ελέγξετε, θα παίρνατε μια NPE εκεί.
if( check(current) == 0) κλπ -> θα πρέπει να ελέγχετε μία φορά και στη συνέχεια να εκτελέσει το αν (ή ακόμα και ένα διακόπτη)

Παράδειγμα για 2 .:

 int result = check(current);
 switch(result) {
  case 0:
    //do whatever is appropriate
    break;
  case 1:
    //do whatever is appropriate
    break;
  case 2:
    //do whatever is appropriate
    break;
  default:
    //should never happen, either leave it or throw an exception if it ever happens
}

Επεξεργασία: // Στην πραγματικότητα, ξεχάστε αυτή την επεξεργασία, μόλις είδα αυτό δεν πρέπει να συμβεί, αλλά δεν είναι ακόμα ένα καλό στυλ

Μπορείτε επίσης να έχουν τα πράγματα όπως αυτό στον κώδικά σας:

if (current.left != null) {
    current=current.left;
    prev.key=current.key;
    prev.left = current.left;
    this.repaint();
}
else {
    current=current.right; //this might be null
 ...
}

Αν current.leftείναι άκυρη και current.rightείναι μηδενική, currentθα είναι μηδενική μετά.