java - repetir - Agregar números a un rango

numero entero aleatorio en java

Question

Estoy haciendo un programa donde puedo agregar una secuencia entera a un rango. Tengo problemas para agregar estos valores al Rango. Debería ejecutar el método addNodeAfter , pero no hace nada.

Luego, cuando quiero mostrar el Rango, obtengo una NullPointerException esta línea:

for (int i = 1; i <= manyNodes; i++){

¿Algun consejo?

Principal:

public class PDEMain {
/**
* @param args the command line arguments
*/
public static void main(String[] args) {
Scanner input = new Scanner(System.in);
System.out.print("Enter a start number: ");
Integer startNum = input.nextInt();
System.out.print("Enter end number: ");
Integer endNum = input.nextInt();
System.out.print("Enter increment: ");
Integer increment = input.nextInt();
Range obj = new Range(startNum, endNum, increment);
System.out.println(obj);
obj.display();
}
}

Distancia:

public class Range implements Cloneable {
private Integer data; // holds the data
private Range link; //holds the link
Range head; //refers to head of linked list
private Integer manyNodes;
private Integer startValue;
private Integer endValue;
private Scanner input;
public Range(Integer data, Range link){
this.data = data;
this.link = link;
}
public Range(Integer data, Range link, Range head) {
this.data = data;
this.link = link;
this.head = head;
manyNodes++;
}
public Range(Integer start, Integer end,Integer increment){
if(start == null){
startValue = 0;
}
if(increment == null){
if(start < end){
increment++;
}else{
increment--;
}
}
for (int i = start; i <= end; i+= increment){
addNodeAfter(i);
System.out.println(i);
}
}
public Integer getData() {
return data;
}
public void setData(Integer data) {
this.data = data;
}
public Range getLink() {
return link;
}
public void setLink(Range link) {
this.link = link;
}
public Range getHead() {
return head;
}
public void setHead(Range head) {
this.head = head;
}
public void addNodeAfter(Integer element){
this.link = new Range(element, this.link);
}
public void display(){
Range cursor = head;
for (int i = 1; i <= manyNodes; i++){ // NPE on this line
System.out.print(cursor.getData() + " ");
cursor = cursor.getLink();
}
System.out.println("");
}
}

Answer 1

Ha definido manyNodes como un Integer , no como un int . Esto significa que su valor predeterminado es null , no 0 , y nunca establece el valor en ningún lugar de su código.

Cuando intente utilizar el bucle como la variable de control en su método display() , la JVM lanzará un NPE cuando intente desempaquetar el null .

Una solución rápida sería cambiar el tipo a int :

private int manyNodes;

Esto resolverá el NPE inmediato, pero aún no mostrará nada, ya que en realidad nunca incrementa manyNodes en el constructor al que está llamando. Esto significa que el for-loop en su método de display() se cae y nunca imprime ninguno de los datos.

Recomendaría deshacerse por completo de head y manyNodes , y volver a escribir su método display() largo de las líneas de:

public void display() {
Range cursor = getLink();
while (cursor != null) {
System.out.print(cursor.getData() + " ");
cursor = cursor.getLink();
}
System.out.println("");
}

Tenga en cuenta que esto generará los datos "hacia atrás" debido a la forma en que agrega cosas en este constructor:

public static void main(String[] args) throws Exception {
Range obj = new Range(1, 10, 1);
obj.display(); // prints 10 9 8 7 6 5 4 3 2 1
}

Es posible que desee echar un vistazo a una implementación de lista enlazada existente para tener una mejor idea de cómo se escriben normalmente.