Question

我有一个MP3Track类成员：轨道号（int），轨道名称（字符串），艺术家字符串），在faourites（布尔）等。我的目录类包含两个ArrayLists，一个包含所有轨道（vectorMain），另一个包含fovuorites（vectorFav），它们是主目录中轨道的副本。

我希望用户可以选择交换两个轨道号（在两个阵列中都是唯一的）。

我添加了以下代码来执行此操作并且它可以工作（如果它们存在 - ATM）但它看起来相当麻烦。必须有更好的解决方案吗？

getMainIndex（） - 返回给定轨道号的ArrayList中的索引位置或-1。

public void swapTrack(){
    int t1 = -1, t2 = -1;
    System.out.println(face.getSwapTrackMenu());       
    System.out.print("1) Please enter first track number to swap: ");
    t1 = scan.readInt();                                
    System.out.print("2) Please enter second track number to swap: ");
    t2 = scan.readInt();                                

    if((getMainIndex(t1)!=-1)&&(getMainIndex(t2)!=-1)){
        String s1 = null;
        String s2 = null;
        for(MP3Track track : vectorMain){
            if(track.getTrackNo() == t1){
                s1 = track.getTitle();
            }
            if(track.getTrackNo() == t2){
                s2 = track.getTitle();
            }
        }

        for(MP3Track track : vectorMain){
            if(track.getTitle().equals(s1)){
                track.setTrackNo(t2);
            }
            if(track.getTitle().equals(s2)){
                 track.setTrackNo(t1);
            }
        }

        for(MP3Track track : vectorFav){
            if(track.getTitle().equals(s1)){
                track.setTrackNo(t2);
            }
            if(track.getTitle().equals(s2)){
                track.setTrackNo(t1);
            }
        }
    }
}//End moveTrack

我是Java的新手，所以任何建议，改进和评论赞赏。

非常感谢

好吧我也尝试过以下看起来更优雅且有效的???但是，有一种更好的方法来实现这一目标吗？我错过了什么？我在这里碰到了一堵砖墙！

    for(MP3Track track : vectorMain){
        if(track.getTrackNo() == t1){
            mp31 = vectorMain.get(getMainIndex(t1));
            //mp31.setTrackNo(t2);
        }
        if(track.getTrackNo() == t2){
            mp32 = vectorMain.get(getMainIndex(t2));
            //mp32.setTrackNo(t1);
        }
    }
    mp31.setTrackNo(t2);
    mp32.setTrackNo(t1);
    vectorMain.add(vectorMain.remove(getMainIndex(t1)));
    vectorMain.add(vectorMain.remove(getMainIndex(t2)));

    for(MP3Track track : vectorFav){
        if(track.getTrackNo() == t1){
            mp31 = vectorFav.get(getFavIndex(t1));
        }
        if(track.getTrackNo() == t2){
            mp32 = vectorFav.get(getFavIndex(t2));
        }
    }
    mp31.setTrackNo(t2);
    mp32.setTrackNo(t1);
    vectorFav.add(vectorFav.remove(getFavIndex(t1)));
    vectorFav.add(vectorFav.remove(getFavIndex(t2)));

非常感谢

Answer 1

你应该使用“indexOf”，它是ArrayList上的一个方法来检索对象的索引。我想你知道怎么去那里。 http://docs.oracle.com/javase/1.4.2/docs/api/java/util/ArrayList.html

将ArrayList对象成员与另一个成员交换 - Java

1 个答案: