[Java] Meerdere classes string opvragen uit andere class

[Verwijderd]

ik zit met een probleem ben in Java een chat programma aan het programmeren.

Ik heb twee classes in de ene class heb ik een hoofdvenster in de andere class heb ik een schermpje waar je instellingen kan maken.
In het hoofdvenster heb ik een Subclass waar ik ip adressen in een string plaats. Deze moeten later uit een textbestand worden gehaald.

Maar ik wil nu in de andere class de ip adressen weer opvragen. Maar dit lukt mij niet.

Class 1:

code:

public class chat extends Frame implements ActionListener
{
    String LocalIp, LocalPort, RemoteIp, RemotePort, Nickname;

// meer codes.....

    public void loadinst()
    {
        LocalPort = "sdfdsds";
        RemoteIp = "127.0.0.1";
        RemotePort = "10101";
        Nickname = "No nickname";
    }

class 2:

code:

class instellingen extends Frame implements ActionListener

    public void saveinst()
    {
        JOptionPane.showMessageDialog(null, LocalPort);
    }

// meer codes.....

ik kan LocalPort niet opvragen. Ik heb al chat.LocalPort geprobeerd niets hielp.

Een vriend van mij zei plaats de message is in een andere class ook gepobeerd lukt maar ja hoe kan ik een subclass opvragen uit een andere class.

Want als saveinst() in een andere class staat kan ik die niet opvragen. Door middel van saveinst() en ook niet chat.saveinst() of main.saveinst() etc......

Wat doe ik fout??? Moet ik global of public voor de string typen???

En nog een hele andere vraag kun je het beste een class maken met alle subfuncties zo noem ik ze maar. Waar kun je het best codes typen die voor gemeen schappelijk gebruik zijn, dus meerdere venster en andere classes moeten dezelfde code kunnen lezen.

Bijvoorbaat dank voor al uw hulp!!!!!!

[Verwijderd]

moet je de string niet teruggeven met behulp van een getLocalPort() methode?? zoiets bijvoorbeeld:

code:

public String getLocalPort() {
  return LocalPort;
}

[Jelmer]

Ok, teneerste Classes moeten met een hoofd letter, variabelen met een kleine letter (Zie sun's java code conventions)

Om de variablen in een ander object op te vragen zul je een referentie naar je Chat object moeten hebben.

Kortom, als je new Intstellingen doet, moet je bijv als parameter chat (<- object van klasse Chat) mee geven.

Succes!

ohja, compiler output lezen en posten heeft vaak ook zin..

[ Voor 10% gewijzigd door Jelmer op 01-04-2003 18:09 ]

[Verwijderd]

Jelmer schreef op 01 April 2003 @ 18:08:
Ok, teneerste Classes moeten met een hoofd letter, variabelen met een kleine letter (Zie sun's java code conventions)

Om de variablen in een ander object op te vragen zul je een referentie naar je Chat object moeten hebben.

Kortom, als je new Intstellingen doet, moet je bijv als parameter chat (<- object van klasse Chat) mee geven.

Succes!

ohja, compiler output lezen en posten heeft vaak ook zin..

in de klasse waar je de info dus wilt gebruiken, doe je:

String localPort = chat.getLocalPort();

met LocalPort() gedefinieerd in de chat klasse zoals ik hierboven ergens al zei...

[Jelmer]

Ja, maar als je geen referenties hebt gaat dat niet echt lekker werken he darth_vader.... (tenzij je alles static maakt, maar dat gaat nou eenmaal niet op in dit verhaal)

[ Voor 4% gewijzigd door Jelmer op 01-04-2003 18:13 ]

[Verwijderd]

Jelmer schreef op 01 april 2003 @ 18:13:
Ja, maar als je geen referenties hebt gaat dat niet echt lekker werken he darth_vader.... (tenzij je alles static maakt, maar dat gaat nou eenmaal niet op in dit verhaal)

hoe bedoel je precies referenties?? doe es een voorbeeldje, snap het niet

edit: of bedoel je met implements ofzo??

[ Voor 6% gewijzigd door Verwijderd op 01-04-2003 18:19 . Reden: toevoeging ]

[Jelmer]

Nou vooruit dan maar, dit is wel errug basic java, maar goed... Dit is ook voor de TS van toepassing:

Java:

class ChatWindow
{
      private String localPort;
      .
      .
      .
      public String getLocalPort()
      {
            return localPort;
      }

      public static void main(String[] args)
      {
            ChatWindow chatWindow=new ChatWindow();

            InstellingenWindow instellingenWindow=new InstellingenWindow(chatWindow);
      }
}

class InstellingenWindow
{
        private ChatWindow chatWindow;

        InstellingenWindow(ChatWindow chatWindow)
        {
                this.chatWindow=chatWindow

        }

        public void saveInst()
        {
               chatWindow.getLocalPort();
        }
}

Hiermee heb je dus een referentie vanuit de main aan instellingenWindow meegeven naar de instantie van ChatWindow en kun je non-static variablen & methoden van ChatWindow gebruiken in andere classe.


enne, nee geen implements... daar mee geef je geïmplementeerde interfaces op.

[ Voor 15% gewijzigd door Jelmer op 01-04-2003 20:29 ]

[Verwijderd]

Jelmer schreef op 01 april 2003 @ 18:21:
Nou vooruit dan maar, dit is wel errug basic java, maar goed... Dit is ook voor de TS van toepassing:

Java:

code...

Hiermee heb je dus een referentie vanuit de main aan instellingenWindow meegeven naar de instantie van ChatWindow en kun je non-static variablen & methoden van ChatWindow gebruiken in andere classe.


enne, nee geen implements... daar mee geef je geïmplementeerde interfaces op.

aha...nu snap ik het. Op onze opleiding noemen we dat wat anders

[Verwijderd]

Ik krijg nu de volgende fout medling:

code:

chat.java:257: non-static method getLocalPort() cannot be 
referenced from a static context
                        LocalPort = chat.getLocalPort();
                                        ^
1 error

Ik heb deze code nu:

code:

// class chat ik heb de variable LocalPort gedeclareerd in deze class
public String getLocalPort()
{
    return LocalPort;
}

// dit staat in de andere class ik heb in de ze class ook 
// de variable LocalPort gedeclareerd.

LocalPort = chat.getLocalPort();
JOptionPane.showMessageDialog(null, LocalPort);

Sla ik nou iets over? Omdat het bij nou nog niet werkt. Zoals enkele van jullie opvalt heb ik nog niet de naam geving gewijzigd. Dit ga ik wel doen maar ik lees eerst de CodeConventions van sun door. Dit is voor mij nieuw ik kwam niet verder dan api's van de site java.sun.com.
Ik vind het wel intresant dus ik ga de naamgeving nog wijzigen. Als ik de oplossing heb zou ik met nieuwe naamgeving de code posten.

Ik wil jullie even bedanken voor de snelle reacties!!!
Maar wat doe ik nu nog fout?

[Jelmer]

Lees de code van mij nog maar eens goed door (heb er tevens nog een methode aan toegevoegd, te weten saveInst) en kijk voornl eens naar het hoofd en kleine letter gebruik (daar zit nl het verschil, niet dat de compiler het wat boeit als je classes met kleine letters doet en variablen met hoofdletters, maar wel als je ze door elkaar gaat gebruiken).

Wat jij probeert te doen is een instance-methode static aan te roepen.

[Verwijderd]

Ik krijg nu geen foutmelding meer.

Maar als ik tijdens het draaien op de des betrefende button klik dan krijg ik dit:

code:

Exception occurred during event dispatching:
java.lang.NullPointerException
        at instellingen.actionPerformed(ChatWindow.java:260)
        at java.awt.Button.processActionEvent(Button.java:324)
        at java.awt.Button.processEvent(Button.java:297)
        at java.awt.Component.dispatchEventImpl(Component.java:2588)
        at java.awt.Component.dispatchEvent(Component.java:2492)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:334)
        at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchTh
read.java:126)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThre
ad.java:93)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:88)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:80)

Maar wat doe ik nu nog fout dan???

Dit is de code tot nu toe:

Class1:

code:

    public static void main(String args[])
    {
        ChatWindow chatWindow = new ChatWindow();
    }

    public String getLocalPort()
    {
        return localPort;
    }

Class2:

code:

    // Variable
    String localPort;
    private ChatWindow chatWindow;

        //  de regel hieronder staat in de code die het windowtje tekend:
        this.chatWindow = chatWindow;

    public void actionPerformed(ActionEvent e)
    {
        if (e.getSource() == CmdApply)
        {
            localPort = chatWindow.getLocalPort();
            JOptionPane.showMessageDialog(null, localPort);
        }
    }

[Robtimus]

Lijkt me logisch: iets op regel 260 in ChatWindow.java is null. Als je even die regel markeert of zo, dan kunnen we precies zien wat het kan zijn.

[Jelmer]

// de regel hieronder staat in de code die het windowtje tekend:

Bedoel je daar de constructor mee? Kun je trouwens de methode definitie een posten? 't is nogal wiedes dat ie NULL blijft, als de methode zonder die parameter wordt aangeroepen.

't Heeft zie ik nu toch soms zo zn voordelen om je class-local variablen met (zoals een aantal guru's hier enigszins propageren) met een _ te beginnen. Dan kun je iig de fout van this.x=x en daarmee perongeluk dezelfde variable addreseren.

[NetForce1]

Ik denk dat hier de fout zit:

Java:

this.chatWindow = chatWindow;

ik weet of ze nog in een aparte scope zitten ofzo, maar zoals het hierboven staat is dat een beetje onduidelijk

/edit: waarom kan ik nou niet sneller een reply verzinnen

[ Voor 15% gewijzigd door NetForce1 op 01-04-2003 21:44 ]

[NetForce1]

quote ipv edit

[ Voor 94% gewijzigd door NetForce1 op 01-04-2003 21:43 ]

[Robtimus]

Misschien... Het zou ook kunnen dat ie die chatWindow parameter (ik denk dat het een parameter is) niet goed geinitialiseerd wordt.

[Verwijderd]

Maar je wilt een chatbox maken begrijp ik? Dan is het toch de taak van de server om IP adressen en ports bij te houden? Deze kun je dan in je eigen client wel weer opvragen uit de server en dan in een venstertje of waar ook weer uitlezen :-)

[Verwijderd]

Hierbij ff de hele code zonder alle objecten anders wordt het bericht zo groot is het nu al maar ja. Misschien wat makkelijker dat je alle codes hebt.


import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class ChatWindow extends Frame implements ActionListener
{
// Variabelen voor in het geheugen
private String localIp, localPort, remoteIp, remotePort, nickName;

public ChatWindow()
{
// Venster instellingen
setLayout(null);
setSize(415,565);
setLocation(100,0);
setResizable(false);
setVisible(true);
setTitle("Winsock Messenger");
show();

addWindowListener(new WindowAdapter() {
public void windowClosing(WindowEvent e) {
System.exit(0);
}
});

loadinst();
}

public static void main(String args[])
{
ChatWindow chatWindow = new ChatWindow();
}

public void actionPerformed(ActionEvent e)
{
if (e.getSource() == CmdZend)
{
new instellingen();
}
if (e.getSource() == MniAfsluiten)
{
System.exit(0);
}
if (e.getSource() == MniAbout)
{
JOptionPane.showMessageDialog(null, remoteIp);
JOptionPane.showMessageDialog(null, "Dit programma is gemaakt door ......");
}
}

public void loadinst()
{
localPort = "sdfdsds";
remoteIp = "127.0.0.1";
remotePort = "10101";
nickName = "No nickname";
}

public void setInstellingen()
{
JOptionPane.showMessageDialog(null, localPort);
}

public String getLocalPort()
{
return localPort;
}

}


class instellingen extends Frame implements ActionListener
{
// Variable
String localPort;
private ChatWindow chatWindow;

// Venster instellingen
setLayout(null);
setSize(400,400);
setLocation(100,100);
setResizable(false);
setVisible(true);
setTitle("Winsock Messenger - Instellingen");
show();

addWindowListener(new WindowAdapter()
{
public void windowClosing(WindowEvent e)
{
System.exit(0);
}
});

this.chatWindow = chatWindow;
}

public void actionPerformed(ActionEvent e)
{
if (e.getSource() == CmdApply)
{
localPort = chatWindow.getLocalPort();
JOptionPane.showMessageDialog(null, localPort);
}
}

}


Regel 260 = localPort = chatWindow.getLocalPort();
En hier ligt ook de fout als dus de fout medling. Wat doe ik nou fout dat ik deze foutmedling krijg???

ff heel wat anders:
Hoe krijg je eigenlijk regelsnummers voor code, welk woord moet ik typen tussen de vierkante hekjes?

[momania]

Je maakt dus nu een instantie van Instellingen aan, en hebt daarin een variabele voor ChatWindow, maar die wordt nergens geinitialiseerd of gevuld en blijft dus null.

[edit]
Ik zie nu zelfs dat de Class Instellingen niet eens een constructor heeft

[edit2]
't Is allemaal wel weer erg basic dit hoor, dit is de zoveelste die eerst knopjes wil zien voordat er echt geprogrammeerd gaat worden.
Dus tip: Ga eerst leuke console proggies coden en leer de taal goed.

[ Voor 52% gewijzigd door momania op 01-04-2003 23:30 ]

[momania]

Even leesbaarder:

Java:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class ChatWindow extends Frame implements ActionListener
{
    // Variabelen voor in het geheugen
    private String localIp, localPort, remoteIp, remotePort, nickName;

    public ChatWindow()
    {
        // Venster instellingen
        setLayout(null);
        setSize(415,565);
        setLocation(100,0);
        setResizable(false);
        setVisible(true);
        setTitle("Winsock Messenger");
        show();

        addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
            System.exit(0);
            }
        });

        loadinst();
    }

    public static void main(String args[])
    {
        ChatWindow chatWindow = new ChatWindow();
    }

    public void actionPerformed(ActionEvent e)
    {
        if (e.getSource() == CmdZend)
        {
            new instellingen();
        }
        if (e.getSource() == MniAfsluiten)
        {
            System.exit(0);
        }
        if (e.getSource() == MniAbout)
        {
            JOptionPane.showMessageDialog(null, remoteIp);
            JOptionPane.showMessageDialog(null, "Dit programma is gemaakt door ......");
        }
    }

    public void loadinst()
    {
        localPort = "sdfdsds";
        remoteIp = "127.0.0.1";
        remotePort = "10101";
        nickName = "No nickname";
    }

    public void setInstellingen()
    {
        JOptionPane.showMessageDialog(null, localPort);
    }

    public String getLocalPort()
    {
        return localPort;
    }

}

Java:

class instellingen extends Frame implements ActionListener
{
    // Variable
    String localPort;
    private ChatWindow chatWindow;

        // Venster instellingen
        setLayout(null);
        setSize(400,400);
        setLocation(100,100);
        setResizable(false);
        setVisible(true);
        setTitle("Winsock Messenger - Instellingen");
        show();

        addWindowListener(new WindowAdapter()
            {
                public void windowClosing(WindowEvent e)
                {
                    System.exit(0);
                }
            });

        this.chatWindow = chatWindow;
    }

    public void actionPerformed(ActionEvent e)
    {
        if (e.getSource() == CmdApply)
        {
            localPort = chatWindow.getLocalPort();
            JOptionPane.showMessageDialog(null, localPort);
        }
    }

}

[Verwijderd]

Uhm wil niet lullig doen ofzo, maar dit kan geen chatbox voorstellen?

[Verwijderd]

momania schreef op 01 April 2003 @ 23:26:
Je maakt dus nu een instantie van Instellingen aan, en hebt daarin een variabele voor ChatWindow, maar die wordt nergens geinitialiseerd of gevuld en blijft dus null.

[edit]
Ik zie nu zelfs dat de Class Instellingen niet eens een constructor heeft

[edit2]
't Is allemaal wel weer erg basic dit hoor, dit is de zoveelste die eerst knopjes wil zien voordat er echt geprogrammeerd gaat worden.
Dus tip: Ga eerst leuke console proggies coden en leer de taal goed.

Wat bedoel je met een constructor? Nou ik heb al enkele console programma's gemaakt. Maar ook in een console zou ik een variable van de ene class naar de andere willen halen. Dus misschien wil je me toch nog even helpen zodat ik dit voor elkaar kan krijgen?!

Verwijderd schreef op 01 april 2003 @ 23:31:
Uhm wil niet lullig doen ofzo, maar dit kan geen chatbox voorstellen?

Nou zoals ik al zei heb ik heel wat code geschrapt om het allemaal nog leesbaar op deze topic te kunnen krijgen. Dus nu kun je er idd geen chatbox meer van maken maar dat komt allemaal wel goed!

[Robtimus]

Ik wil je niet afzeiken of zo, maar als je niet eens weet wat een constructor is zou ik me eerst wat beter verdiepen in de beginselen van OO.

[Fostar82]

Regel 24 Instellingen van momania's post:
this.chatWindow = chatWindow;

volgens mij zit hier een mooie fout.
this.chatWindow is de pointer(verwijzing) naar een object van Chatwindow maar die is dus nog gelijk aan NULL, want een andere chatWindow dan this.chatWindow bestaat niet!!! Wel in main in de class Chatwindow maar Instellingen kan daar helemaal niet bij.

Hoe je dit kan oplossen is door Instellingen een constructor te geven. (De constructor wordt altijd direct aangeroepen nadat je een object maakt van de betreffende class.) Die constructor geef je dan je chatwindow aangemaakt in main mee.
Public Instellingen(ChatWindow hetChatWindow)
{
chatWindow = hetChatWindow; //nu wijst de pointer chatWindow naar het //Object van chatWindow wat je in de main //hebt aangemaakt.
}

en in ChatWindow maak je instellingen aan door
new Instellingen(this);

Hiermee laat je ChatWindow zich zelf meegeven aan de instance van Instellingen.
Nu kun je dus bij de methoden van chatWindow... dus moet de aanroep op regel 31 in Instellingen van getLocalPoort() werken.

Suc6
Hoop dat ik het allemaal goed heb. Programmeer alleen nog maar in C++

[momania]

Fostar82 schreef op 02 april 2003 @ 15:59:
Hoop dat ik het allemaal goed heb. Programmeer alleen nog maar in C++

't Is goed gekeurd

Ik wou het alleen zelf niet posten, omdat ik vind dat dit te basic is.

[Fostar82]

momania schreef op 02 april 2003 @ 16:02:
[...]

't Is goed gekeurd

Ik wou het alleen zelf niet posten, omdat ik vind dat dit te basic is.

Hehe dank je

[Verwijderd]

Oké ik heb nu wat werkends! Ik heb nu nog een vraagje welke namen geven jullie aan een class

OpslaanIn slaopin = new OpslaanIn();

Ik heb de class OpaslaanIn slaopin van gemaakt maar kun je het best gewoon de class met kleine letters schrijven of een hele andere naam geven in dit geval? Wat gebruiken jullie. Want ik kan op de site van Sun niet wijzer uitworden als ik bij de Code Conventions kijk.

http://java.sun.com/docs/codeconv/html/CodeConvTOC.doc.html

Allemaal heel erg bedankt!!!

Java:

/*
 *  Voorbeeld programma
 *  Geschreven door Darksilence
 */

public class OpslaanIn {
    String str1, str2;

    public static void main(String[] args)
    {
        System.out.println ("Voorbeeld programma");

        OpslaanIn slaopin = new OpslaanIn();
        slaopin.opslaan();


        GeefWeer weergave = new GeefWeer();
        weergave.string1(slaopin);
        weergave.string2(slaopin);
    }

    void opslaan()
    {
        str1 = "doet bij?";
        str2 = "Jaha hij doet echt!";
    }
}

class GeefWeer
{

    private OpslaanIn slaopin;

    void string1(OpslaanIn slaopin)
    {
        this.slaopin = slaopin;
        System.out.println (slaopin.str1);
    }

    void string2(OpslaanIn slaopin)
    {
        this.slaopin = slaopin;
        System.out.println (slaopin.str2);
    }
}

[momania]

Verwijderd schreef op 02 April 2003 @ 17:08:
Ik heb de class OpaslaanIn slaopin van gemaakt maar kun je het best gewoon de class met kleine letters schrijven of een hele andere naam geven in dit geval? Wat gebruiken jullie. Want ik kan op de site van Sun niet wijzer uitworden als ik bij de Code Conventions kijk.

Ik denk dat dit wel wat voor je is:
http://islab.ruca.ua.ac.be/bart/javacursus/inleiding.html (net ff gevonden op Google)
Is een simpele starterscursus java (in het nederlands), waar je denk een hoop van kan leren
Staat vast ook wel wat tussen over code conventions.

code:

/*
 *  Voorbeeld programma
 *  Geschreven door Darksilence
 */

public class OpslaanIn {
    String str1, str2;

    public static void main(String[] args)
    {
        System.out.println ("Voorbeeld programma");

        OpslaanIn slaopin = new OpslaanIn();
        slaopin.opslaan();


        GeefWeer weergave = new GeefWeer();
        weergave.string1(slaopin);
        weergave.string2(slaopin);
    }

    void opslaan()
    {
        str1 = "doet bij?";
        str2 = "Jaha hij doet echt!";
    }
}

class GeefWeer
{

    private OpslaanIn slaopin;

    void string1(OpslaanIn slaopin)
    {
        this.slaopin = slaopin;
        System.out.println (slaopin.str1);
    }

    void string2(OpslaanIn slaopin)
    {
        this.slaopin = slaopin;
        System.out.println (slaopin.str2);
    }
}

gebuik voortaan ook even de [code=java] tags aub, dan leest de code nog wat beter.

Succes!