Question

两种可能的解决方案，但哪一种？还是有更好的？

我有一个带有成员的MP3Track类：Track number（int），track name（String），artist String），faourites（boolean）等。我的目录类包含两个ArrayLists，一个包含所有轨道（vectorMain）和其他包含fovuorites（vectorFav），它们是主目录中曲目的副本。

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

（解决方案1）我已经添加了以下代码来执行此操作并且它可以工作（如果它们存在 - 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

（溶液2）

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();
    MP3Track mp31 = null;
    MP3Track mp32 = null;
    //But are these references or not?????
    for(MP3Track track : vectorMain){
        if(track.getTrackNo() == t1){
            mp31 = vectorMain.get(getMainIndex(t1));
        }
        if(track.getTrackNo() == t2){
            mp32 = vectorMain.get(getMainIndex(t2));
        }
    }
    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)));

}

这更优雅，但对我来说似乎也很麻烦。或者我在这里缺少什么？我撞墙了！

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

非常感谢

Answer 1

对我来说似乎过于复杂。

你应该有两个单独的类：Catalog用于可用的MP3世界。收藏夹似乎应该与User相关联。多个用户都应该能够共享Catalog中的收藏，但Catalog班级不需要了解任何内容。

如果您保留IO，这种方法会更有用。将其移动到驱动程序类中，然后将所需的内容传递给方法。

哪个解决方案用于交换ArrayList对象成员与另一个成员 - Java

1 个答案: