[java] array afleveren

[TheNameless]

ik wil in een methode een array vullen met een for-loop en daarna de array afleveren, zodat ik het kan gebruiken in een andere methode in een andere klassen.

code:

int[] vulkaartenRij(int [] kaartenRij)
  {
    for (int i=0;i<kaartenRij.length; i++)
    {
    kaartenRij[i]=i+1;
    }
    return kaartenRij[];
  }

ik dacht misschien zo, maar dat werkt niet.
iemand een idee?

[chem]

over welke taal hebben we het?

[FireFoz]

[bijdehand mode]
ik denk grieks
[/bijdehand mode]

[BierPul]

Op woensdag 17 oktober 2001 10:56 schreef FireFoz het volgende:
[bijdehand mode]
ik denk grieks
[/bijdehand mode]

[mbravenboer]

Java

return kaartenRij (zonder []) zou denk ik moeten werken...

[drm]

hmm. ziet eruit als Java

melp

[wasigh]

Op woensdag 17 oktober 2001 10:42 schreef MayaFreak het volgende:

ik dacht misschien zo, maar dat werkt niet.
iemand een idee?

Geef voortaan de foutmelding.
Nu is dit niet zo'n moeilijke error maar toch.

mbravenboer heeft het antwoord al gegeven..

[Verwijderd]

Of maak kaartenRij[] gewoon public...

[mbravenboer]

minne: Of maak kaartenRij[] gewoon public...

foei

[TheNameless]

Op woensdag 17 oktober 2001 10:59 schreef mbravenboer het volgende:
Java

return kaartenRij (zonder []) zou denk ik moeten werken...

thnx
hoe kan ik nu de return waarde gebruiken in andere methode?

[mbravenboer]

Check:

De Java Tutorial,Learning the Java Language, Object Basics and Simple Data Objects, Arrays

[drm]

ligt er aan wat je ermee wilt

code:

int [] result = new int [ aantal ];

result = vulkaartenRij ( andereArray );

for ( int a = 0; a < result.length; a++ )
   System.out.println ( a );

toch


* drm heeft al een tijdje geen java meer gedaan...

[wasigh]

Op woensdag 17 oktober 2001 14:03 schreef MayaFreak het volgende:

[..]

thnx
hoe kan ik nu de return waarde gebruiken in andere methode?

door hem mee te geven aan die andere methode..

[TheNameless]

code:

package memory;

import java.awt.*;
import java.applet.*;

public class Applet1 extends Applet
{
  int invoer = 20;
  TextField invoerveld;
  Button knopDeel;
  Label beschr;
  Spel spel= new Spel(invoer);

  public void init()
  {
    beschr = new Label("Vul hier het aantal paren in (1 tm 20)");
    invoerveld = new TextField("Hier intikken svp");
    knopDeel = new Button("Deel");
    add (beschr);
    add (invoerveld);
    add (knopDeel);
  }
  public void paint(Graphics g)
  {
    spel.tekenveld(g,invoer);
    spel.resetWaarden();
  }
  public boolean action(Event e,Object o)
  {
    if(e.target  == knopDeel)
    {
    repaint();
    return true;
    }
    return false;
  }
}
class Kaarten
{
  int invoer;
  int[] kaartenRij;

  Kaarten(int invoer)
  {
    this.invoer = invoer;
    System.out.println(invoer); //levert 0!!!!
    kaartenRij = new int[invoer];
  }
  int[] vulkaartenRij()
  {
    for (int i=0;i<kaartenRij.length; i++)
    {
    kaartenRij[i]=i+1;
    }
    return kaartenRij;
  }
}
class Spel
{
  int invoer;
  int xpos,ypos;
  int getal;
  int[] dumpRij;
  int[] volkaartenRij;
  Kaarten k = new Kaarten(invoer);

  Spel(int invoer)
  {
    this.invoer = invoer;
    System.out.println(invoer); //levert 20
  }
 /**
  * pakt een willekeurig getal uit volkaartenRij[]
  * en tekent een kaart en herhaald dat
 */
  public void tekenveld(Graphics g, int invoer)
  {
    volkaartenRij = k.vulkaartenRij();
    for(int i=0;i<volkaartenRij.length;i++)
    {
    volkaartenRij[(int) ((this.invoer-1)* Math.random())] = getal;
    g.drawString(""+getal,xpos+10,ypos+20);
    g.drawRect(xpos,ypos,30,30);
    xpos += 30;
    }
  }
  void resetWaarden()
  {
    xpos =0;
    ypos = 90;
  }
}

Waarom levert de eerste System.out.prinln(invoer) 0?

[wasigh]

Een erg interressante en goede vraag

code:

public class Applet1 extends Applet
{
  int invoer = 20;
  TextField invoerveld;
  Button knopDeel;
  Label beschr;
  Spel spel= new Spel(invoer);

Dit is het initialiseren van variabelen. Dit wordt gedaan op het moment dat de klass wordt aangemaakt. Bij bovenstaande code zou je dus verwachten dat invoer de waarde 20 heeft.. maar..
de compiler kan en mag de volgorde van deze regels omdraaiien. hij mag dus bijvoorbeeld eerst alle variabelen aanmaken en ze dan pas van waarde voorzien.

Dit verklaard waarom invoer o kan zijn

stel dat hij er de volgende regels van maakt:

code:

public class Applet1 extends Applet
{
  int invoer;
  Spel spel= new Spel(invoer);
  invoer = 20;

Dit is volkomen legaal, invoer krijgt de waarde 0 bij initialisatie. Daar word een nieuw spel mee aangemaakt. En dan wordt de waarde van invoer op 20 gezet.

(let op dit geld alleen bij ingebouwde typen)

Dit wetende moet je dus zorgen dat je van deze volgorde nooit afhankelijk bent.

Dus moet je wachten met het aanmaken van Spel tot je zeker weet dat invoer de juiste waarde heeft:

code:

public class Applet1 extends Applet
{
  int invoer = 20;
  TextField invoerveld;
  Button knopDeel;
  Label beschr;
  Spel spel;

  public void init()
  {
    spel = new Spel(invoer);
    beschr = new Label("Vul hier het aantal paren in (1 tm 20)");

edit:
bij nader inzien klinkt het verhaal van mbravenboer logischer

[mbravenboer]

MayaFreak:

code:

class Spel
{
  int invoer;
  ...
  Kaarten k = new Kaarten(invoer);

  Spel(int invoer)
  {
    this.invoer = invoer;
  }

De instantie variabele invoer krijgt een waarde op het moment dat de constructor wordt aangeroepen. Voordat de de constructor wordt aangeroepen worden echter al de declaraties in je klasse verwerkt. De variabele invoer heeft als beginwaarde dus 0. Met de waarde wordt ook het object Kaarten aangemaakt die opgeslagen wordt in de variabele k. Daarna wordt pas de invoer variabele ingesteld in de constructor.

In het algemeen is het onverstandig en onduidelijk om instantie variabelen gelijk bij hun declaratie in de klasse een waarde te geven. Doe dat liever in de constructor .

edit: hum we waren tegelijk bezig

[mbravenboer]

wasigh: Dit is het initialiseren van variabelen. Dit wordt gedaan op het moment dat de klass wordt aangemaakt. Bij bovenstaande code zou je dus verwachten dat invoer de waarde 20 heeft.. maar..
de compiler kan en mag de volgorde van deze regels omdraaiien. hij mag dus bijvoorbeeld eerst alle variabelen aanmaken en ze dan pas van waarde voorzien.

Dit verklaard waarom invoer o kan zijn

Ook dit geeft alweer aan dat het gewoon erg onhandig en onduidelijk is om variabelen al bij de declaratie in de klasse gelijk te initialiseren. Doe dit gewoon in de constructor, daar zijn ze voor .

Dus moet je wachten met het aanmaken van Spel tot je zeker weet dat invoer de juiste waarde heeft

We zijn het eens

[TheNameless]

code:

package memory;

import java.awt.*;
import java.applet.*;

public class Applet1 extends Applet
{
  int invoer;
  TextField invoerveld;
  Button knopDeel;
  Label beschr;
  Spel spel;

  public void init()
  {
    spel= new Spel(invoer);
    beschr = new Label("Vul hier het aantal paren in (1 tm 20)");
    invoerveld = new TextField("Hier intikken svp");
    knopDeel = new Button("Deel");
    add (beschr);
    add (invoerveld);
    add (knopDeel);
  }
  public void paint(Graphics g)
  {
    spel.tekenveld(g,invoer);
    spel.resetWaarden();
  }
  public boolean action(Event e,Object o)
  {
    if(e.target  == knopDeel)
    {
    invoer = Integer.parseInt(invoerveld.getText());
    repaint();
    return true;
    }
    return false;
  }
}
class Kaarten
{
  int invoer;
  int[] kaartenRij;

  Kaarten(int invoer)
  {
    this.invoer = invoer;
    System.out.println(invoer); //levert 0!!!!
    kaartenRij = new int[invoer];
  }
  /**
   * vult de array met de waarden 1- max 20
   */
  int[] vulkaartenRij()
  {
    for (int i=0;i<kaartenRij.length; i++)
    {
    kaartenRij[i]=i+1;
    }
    return kaartenRij;
  }
}
class Spel
{
  int invoer;
  int xpos,ypos;
  int getal;
  int[] dumpRij; //om te checken welke getallen er al getekend zijn
  int[] volkaartenRij;
  Kaarten k = new Kaarten(invoer);

  Spel(int invoer)
  {
    this.invoer = invoer;
    System.out.println(invoer); //levert 20
  }
 /**
  * pakt een willekeurig getal uit veld[]
  * en tekent een kaart en herhaald dat
 */
  public void tekenveld(Graphics g, int invoer)
  {
    volkaartenRij = k.vulkaartenRij(); //volkaaartenRij wordt de returnwaarde kaartenRij
    for(int i=0;i<volkaartenRij.length;i++)
    {
    volkaartenRij[(int) ((this.invoer-1)* Math.random())] = getal;
    g.drawString(""+getal,xpos+10,ypos+20);
    g.drawRect(xpos,ypos,30,30);
    xpos += 30;
    }
  }
  void resetWaarden()
  {
    xpos =0;
    ypos = 90;
  }
}

(Ik heb alleen wat verandert in boolean action)
Nu geven beide System.out.println 0 (dat is logisch!), maar als je via het textvak (bijvoorbeeld) 5 invuld blijven de beide op 0 staan

[mbravenboer]

Ik zie zo niet wat je precies probeert te doen en waar het mis gaat...

Wel een paar tips:

1. Gebruik geen public boolean action(bla) want dat is heel erg deprecated. De goede manier om akies af te vangen is via een ActionListener. Je moet een ActionListener implementeren voor die knop en daarna aanmelden op de knop. Je events komen dan binnen via de methode actionPerformed.

2. Die variabele invoer is op heel veel plekken te vinden en wordt zelfs ook nog op een aantal plaatsen als parameter meegegeven. Dat werkt erg verwarrend en zorgt snel voor fouten. Probeer eens of je de status van je spel niet in een model (1 of meer klassen) kan representeren. Dat is een stuk makkelijker en de code wordt duidelijker.

3. Probeer het aantal instantie-variabelen te beperken. Je hebt vrij veel variabelen in de klassen zelf gedefinieerd.

[wasigh]

opmerkingen in het algemeen:

> maak variabelen private en gebruik getters en setters
> kies logische namen niet k
> geef je class een duidelijke naam en niet Applet1
> gebruikt voor action's een actionHandler en geen deprecated methode

code:

public void tekenveld(Graphics g, int invoer)
  {
    volkaartenRij = k.vulkaartenRij(); //volkaaartenRij wordt de returnwaarde kaartenRij
    for(int i=0;i<volkaartenRij.length;i++)
    {
    volkaartenRij[(int) ((this.invoer-1)* Math.random())] = getal;
    g.drawString(""+getal,xpos+10,ypos+20);
    g.drawRect(xpos,ypos,30,30);
    xpos += 30;
    }

tuurlijk blijft invoer van spel op 0 staan je wijzigd hem toch ook niet?

[TheNameless]

Ik heb hem werkende!

Bedankt voor de tips, en wat betreft boolean action: dat moeten we gebruiken van de school!

Maar hij ziet er voor de rest wel goed uit?

[mbravenboer]

MayaFreak: Ik heb hem werkende!

Mooi

Bedankt voor de tips, en wat betreft boolean action: dat moeten we gebruiken van de school!

Hum.... ach, het is pas 3 jaar deprecated .

Maar hij ziet er voor de rest wel goed uit?

Het is nog een behoorlijke brei, waardoor het programmeren juist erg lastig gaat... Als je OO echt goed te pakken gaat krijgen (dat zal vast wel komen ) dan wordt het allemaal een stuk eenvoudiger en zou je dit inzichtelijker kunnen oplossen... Voorlopig voldoet het wel . Ik wil je niet teleurstellen, maar over een jaar of 2 wil je deze code niet meer zien .