Ungerade Zahlen

[NataLi]

Hallo,
ich komme nicht drauf wie man die ungeraden Zahlen aus der Liste löscht.
Wo habe ich einen Denkfehler?

Node z = new Node (9, new Node(-17, new Node( 4, new Node(41, null))));


public void removeEven(Node start) {
	Node act = start;
	Node before = null;
	
	while(act!=null) {
		
		if((start.wert)%2!=0) {before=act; act=act.next;
		
		removeEven(act);}
		
	    else .......//was kommt hier? 
	   
		}}

 

[Leroy42]

Wie ist deine Liste überhaupt definiert?

Doppeltverkettet?

Poste mal den Code der Liste

 

[SlaterB]

> Wo habe ich einen Denkfehler?

erstmal keinen, da du kein direktes Problem oder eine sonstige Frage beschrieben hast,
dass du eine bestimmte Aufgabe lösen sollst ist für sich noch kein 'Denkfehler'

> was kommt hier?

also ich persönlich weiß es nicht (außer ganz grob: removeEven(act.next); ) ,
wäre leichter wenn du mal ganz in Ruhe beschreibst, was ein Node ist,
was die Operation machen soll usw.,

ein vollständiges (kurzes) Programm mit Eingabe und gewünschter/ tatsächlicher Ausgabe wäre optimal

 

[NataLi]

es ist eine einfach verkettete Liste...

class Node {
int wert;
Node next;

// Konstruktor       	
Node(int wert, Node nextNode) {
this.wert = wert; this.next = nextNode;

 

[NataLi]

class Node { 
int wert; 
Node next; 

// Konstruktor           
Node(int wert, Node nextNode) { 
this.wert = wert; this.next = nextNode; }

public void removeEven(Node start) { 
   Node act = start; 
   Node before = null; 
    
   while(act!=null) { 
       
      if((start.wert)%2!=0) {before=act; act=act.next; 
       
      removeEven(act);} 
       
       else .......//was kommt hier? 
       
      }} }


class Aufgabe1{

public static void main (String [] args){
Node z = new Node (9, new Node(-17, new Node( 4, new Node(41, null))));
}}

 

[SlaterB]

hmm, du hast ja noch nicht mal eine Ausgabe, um die Ergebnisse zu überprüfen,
aber ich glaube ich lag mit meinem Tipp removeEven(act.next); gar nicht so schlecht

übrigens:
mit
> if((start.wert)%2!=0) {before=act; act=act.next;
veränderst du noch nicht die Originalkette der Nodes, sondern nur deine lokalen Variablen before und act,
du musst die next-Exemplarvariablen der Nodes bearbeiten

ersetze while durch if, du willst eh maximal einmal act prüfen

 

[NataLi]

Entschuldigung, habe die Ausgabe vergessen
Die Methode kommt in die KlassenDefinition Node rein.

void drucke(Node hilf2){
		
	while (hilf2 != null) {
      	System.out.print(hilf2.wert+" "); hilf2 = hilf2.next; }}