`replaceChild` method in BinaryTreeNode is not correct
@DeoJin ya está trabajando en esto.
Desde el 17/3/2026.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- backend
Línea de trabajo
Start at the BinaryTreeNode.replaceChild method and inspect how its left and right child links are used. Confirm that a successful replacement also updates the replacement node's parent reference, then run the relevant BinaryTreeNode tests if available and verify both child positions and failure cases.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Missing update parent. It should be
if (!nodeToReplace || !replacementNode) {
return false;
}
if (this.left && this.left === nodeToReplace) {
this.left = replacementNode;
replacementNode.parent = this;
return true;
}
if (this.right && this.right === nodeToReplace) {
this.right = replacementNode;
replacementNode.parent = this;
return true;
}
return false;
}
- Lenguaje dominante
- JavaScript
- Estrellas
- 197k
- Forks
- 31k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de trekhleb/javascript-algorithms
-
BinarySearchTreeNode removingPosiblemente ocupada @bbonch la tomó hace 239 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
trekhleb/javascript-algorithms#2102 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
trekhleb/javascript-algorithms#2221 · 4 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
trekhleb/javascript-algorithms#2085 · 6 comentarios · 1 reacción ·
-
LinkedList methods.Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
trekhleb/javascript-algorithms#2065 · 1 comentario ·
-
Incorrect Cycle Path Construction in detectUndirectedCyclePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
trekhleb/javascript-algorithms#2057 · 1 comentario ·
Todos los issues de trekhleb/javascript-algorithms
Issues similares
-
[Feature]:Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
automated issue report
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
lirantal/discoprint#36 ·
Los mantenedores suelen responder en 1 día
-
accepting PR Content:HTML
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
mdn/content#45988 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
pnpm/pnpm#16635 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
txn2/mcp-data-platform#2030 ·
Los mantenedores suelen responder en 1 día