Im Array zählen und verändern

[canyakan95]

Hallo,
meine aufgabe ist es in einem Array die negativen Werte zu zählen und wenn meine zählvariable >=4 ist , soll es das ganze Array verdoppeln.
Die Aufgabe habe ich in zwei schritten hinbekommen, aber wollte fragen ob man das auch nur in einer methode hinkriegen kann.

    public static int count( int[]arr)
    {
        int count =0;
        if(arr.length%2==0)
        {
            for(int i=0;i<arr.length;i++)
            {
                if(arr[i]<0)
                {
                    count++;
                }
            }
        }
        return count;
    }
    public static int[] countandchange(int [] arr)
    {
        int counts=count(arr);
        for(int i=0;i<arr.length;i++)
        {
            if(counts>=4)
            {
                arr[i]*=2;
            }
        }
        return arr;
    }

 

[canyakan95]

Und wenn ja, wäre es nett wenn ihr mir vllt eine mögliche Lösung zeigen könntet. Habe es nämlich net hinbekommen .
Mfg

 

[Tarrew]

Du hättest deinen Code aus "count" doch einfach in die andere Methode kopieren können?

Ansonsten zB so:

public static int countAndChange( int[]arr)
	    {
	      int count = 0;
	      for(int i : arr){
	    	  if(i%2!=0){
	    		  count++;
	    	  }
	      }
	      if(count>=4){
	    	  for(int j=0; j<arr.length; j++){
	    		  arr[j]*=2;
	    	  }
	      }
	      return count;
	    }

In deinem Code zählst du übrigens nicht die negativen Werte sondern die ungeraden Werte. Deswegen hab ichs auch mal mit ungeraden Werten gemacht.

 

[Tarrew]

public static int count( int[]arr)
	    {
	      int count = 0;
	      for(int i : arr){
	    	  if(i<0){
	    		  count++;
	    	  }
	      }
	      if(count>=4){
	    	  for(int j=0; j<arr.length; j++){
	    		  arr[j]*=2;
	    	  }
	      }
	      return count;
	    }

Sry hatte bisschen gepennt und nur "%2" gelesen. Zählst natürlich doch die negativen Werte. Müsste also so sein.

 

[fhoffmann]

Ich verstehe nicht, warum du das alles in eine Methode quetschen willst. Jede Methode sollte eine fest begrenzte Aufgabe haben (damit sie auch wiederverwendbar ist).
Sinnvoll wäre es also, folgende Methoden zu schreiben:
- zaehleNegative
- verdopple
- verdoppleBeiMehrAlsSoundsovielNegativen (diese ruft die anderen beiden Methoden auf)

 

[bartzer]

Hi canyakan95,

wozu brauchst du die Abfrage in der Methode "count" ob deine Array-Größe gerade ist?

 if(arr.length%2==0)

Die zweite Methode würde ich einwenig anders strukturieren

public static int[] countAndChange(int [] arr){         
  if(count(arr)<4){
    return arr;
  }
  for(int i=0;i<arr.length;i++){
    arr[i] *= 2;
  }
  return arr;
}

Ich lasse eigentlich nur die deine Hilfsvariable "counts" weg und schreibe den Methodenaufruf von count direkt in die Abfrage. Außerdem finde ich es in der Regel übersichtlicher if-Bedingungen als Ausnahmen zu gestalten. In diesem Beispiel also: beende die Methode wenn count < 4 (und somit nichts weiter passieren soll). Andernfalls, also wenn die Bedingung nicht zutrifft -> verdopple alle Werte des Arrays. So lassen sich meiner Erfahrung nach oft (unübersichtliche) Verschachtelungen vermeiden

- - -

Natürlich kannst du das Ganze auch in eine Methode schreiben, aber wie schon erwähnt wurde ist das nicht besser. Umso verständlicher eine Methode ist desto besser. Also am Besten garnicht erst angewöhnen Alles in eine Methode zu schreiben sondern sinnvoll auf mehrere aufteilen.