[Java] Jlist meerdere selecties methode

[Verwijderd]

Ik ben bezig aan een project waarbij ik uit een database muziekstijlen moet halen bij een evenement (string vector) en deze muziekstijlen vervolgens moet selecteren in een Jlist die alle stijlen bevat. Deze jList wordt ook opgebouwd uit een string vector. Nu is het niet mogelijk om een methode als setSelectedValues() te gebruiken. Wel setSelectedIndices(). Daarom heb ik het volgende bedacht:

code:

private int[] compareVectors(String eventId){
        int[] incs = null;
        int x = 0;
        int y = 0;
        Vector jList = index.queryManager.getStyles();
        Vector eventStyles = index.queryManager.getStylesByEvent(eventId);
        
        while(x <= eventStyles.size()){
           
            if(jList.elementAt(y).equals(eventStyles.elementAt(x))){
                
                incs[x] = y;
                
                x++;
                y = 0;
            }else{
                y++;
            }

        }

setSelectedIndices() gebruikt een int[]. Ik wil dus de twee vectoren vergelijken en als er een overeenkomst is het zoveelste element uit de lijst achterhalen. Mijn methode werkt niet. De compiler geeft steeds een array out of bounds exception. Weet iemand waar de fout zit?

[Gerco]

Waarschijnlijk niet omdat je je incs[] array nooit aanmaakt, dan had je een NullPointerException gekregen, maar het is misschien wel een idee om daar even naar te kijken. De oorzaak is dan waarschijnlijker dat y groter is dan jList.size (what's in a name ).

[ Voor 19% gewijzigd door Gerco op 28-05-2008 13:27 ]

[sig69]

Op welke regel krijg je die exception? ik vermoed

code:

incs[x] = y;

, al had ik eerder een nullpointerexception verwacht.

[Verwijderd]

Dit is de error:

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 0 >= 0
at java.util.Vector.elementAt(Vector.java:427)
at view.View_modEventFrame.compareVectors(View_modEventFrame.java:300)

regel 300 is deze regel

code:

if(jList.elementAt(y).equals(eventStyles.elementAt(x))){

Dat met incs[] = null vind ik ook een beetje raar, maar netbeans begint de hele tijd te zeuren als ik het niet doe. Iemand een suggestie?

[Gerco]

Die incs moet je initialiseren:

Java:

int[] incs = new int[100];

Waarschijnlijk is dat niet wat je wilt, want je weet nog niet hoeveel entries erin gaan komen. Kijk dan eens naar ArrayList, daar kun je gewoon aan appenden en converteren naar een Array als je dat wilt.

[ Voor 21% gewijzigd door Gerco op 28-05-2008 13:49 ]

[sig69]

Verwijderd schreef op woensdag 28 mei 2008 @ 13:38:
Dit is de error:

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 0 >= 0
at java.util.Vector.elementAt(Vector.java:427)
at view.View_modEventFrame.compareVectors(View_modEventFrame.java:300)

regel 300 is deze regel

Voor de volgende keer: je post maar 20 regels, geef dan even aan welke vand eze 20 de exception gooit...

code:

if(jList.elementAt(y).equals(eventStyles.elementAt(x))){

y is dus groter dan het aantal elementen in jlist, of x is groter dan eventStyles. Zet er een breakpoint op en je weer het.

[ Voor 9% gewijzigd door sig69 op 28-05-2008 13:53 ]

[Robtimus]

Java:

private int[] compareVectors(String eventId)
{
    int eventStylesSize = eventStyles.size(); // gecached omdat je die vaker nodig zult hebben
    int jListSize = jList.size(); // idem
    int[] incs = new int[eventStylesSize]; // geinitialiseerd
    int x = 0;
    int y = 0;
    Vector jList = index.queryManager.getStyles();
    Vector eventStyles = index.queryManager.getStylesByEvent(eventId);
        
    while(x < eventStylesSize && y < jListSize) // x <, niet x <=, tevens check op y
    {
        if (jList.elementAt(y).equals(eventStyles.elementAt(x)))
        {
            incs[x] = y;
            x++;
            y = 0;
        }
        else
        {
            y++;
        }
}

Zomaar wat kleine veranderingen die je hoe dan ook nodig zult hebben. Belangrijkste is de while conditie.

[Confusion]

Volgens mij is Vector.indexOf() een stuk handiger...

[Verwijderd]

@IceManX

Het werkt Ik ben erg blij met je tip. Tnx

[ Voor 4% gewijzigd door Verwijderd op 29-05-2008 16:40 ]

[Confusion]

Verwijderd schreef op donderdag 29 mei 2008 @ 15:34:
@IceManX

Het werkt Ik ben erg blij met je tip. Tnx

Begrijp ik daaruit dat je dit moeilijk te begrijpen algoritme gaat laten staan en niet vervangt door een loop waarin de indexOf methode van de Vector klasse wordt gebruikt?

[Verwijderd]

@Confusion

Ik ben nog niet zo heel ervaren met java, deze methode is mij niet eerder opgevallen. Het project is al ingeleverd. Iig bedankt voor de tip. Ik zal er de volgende keer als ik zoiets moet doen eens naar kijken