Java嵌套的循环重复不正确

时间:2014-03-12 14:06:35

标签: java arraylist nested-loops

我创建了这个包,问题是什么时候运行它会在bin之间重复相同的目录条目。任何帮助将不胜感激! 问题可能在详细的库存方法中。 它应该为Bin B创建一个新的随机bin项(类型,标题,艺术家),而不是像现在这样使用Bin A中的相同元素。除此之外,输出是正确的。正确生成并显示SKU编号。

它应该做什么:详细的Inventory方法应该使用for-each循环来遍历仓库中的所有bin。对于每个bin,s应该通过String“Bin”扩展,后跟bin的名称,后跟冒号和\ n以开始换行。然后它应循环遍历当前bin中的所有bin项,对于每一个扩展s的新文本,以在输入目录中查找当前bin项的SKU的结果开始,并继续使用逗号后跟bin项的String表示形式。

其他信息: *当i对应于目录时,它们是对象。它们实际上是不同类型的物理媒体,例如DVD或磁带,具有标题,艺术家和SKU属性。

  • 虽然看起来没有意义,但应该随机生成数据。 (这个类是说明继承的更大概念/课程的一部分。)

  • 我将共享相同SKU的bin中的所有项目的集合称为“bin项目”。

  • Bin类的每个对象(以完整代码显示)表示假装仓库中的bin。 Bin对象有两个实例变量:一个包含bin名称的String和一个包含bin中存储的每个SKU的BinItem的ArrayList。 Bin类的toString方法使用BinItem类的toString方法。它生成的列表仅涉及SKU和数量。

以下是我看到的输出:

Bin A:
DVD - Greatest Hits Volume 2 (Limp Bizkit), SKU 1234-0: 500
Cassette - The Best Of (Michael Jackson), SKU 1234-1: 25
DVD - Love Songs (Michael Jackson), SKU 1234-2: 7720
Bin B: DVD - Greatest Hits Volume 2 (Limp Bizkit), SKU 1234-3: 1000

Bin B下的bin项不应与A相同。

输出应如下所示:

Bin A:
CD - The Later Years (Shania Twain), SKU 1234-0: 500
Cassette - Greatest Hits Volume 2 (The Beatles), SKU 1234-1: 25
Cassette - Greatest Hits Volume 1 (Shania Twain), SKU 1234-2: 7720

Bin B: Cassette - Greatest Hits Volume 2 (Michael Jackson), SKU 1234-3: 1000

更新 的 我试图创建另一种方法来检索基于SKU的项目,但现在编译器在修改后的detailedInventory循环中的Bolded行上抱怨不兼容类型错误使用它?:

public static MusicMedia getMediaBySKU(ArrayList<MusicMedia> catalog, String sku)
   {
    for ( MusicMedia m : catalog ) {
        if ( m.getSKU().equals(sku) )
            return m;
    }
    return null;
}

修饰:

for ( Bin bn : warehouse ){
    s += "Bin " + bn.getName() + ":\n";

    for ( BinItem item : bn.getContents() ){
        MusicMedia mm = getMediaBySKU(catalog, item.getSKU());

        s += mm + ", " + item + "\n";
    }
}

public class testMusicMedia
{
    public static ArrayList<MusicMedia> MakeMusicCatalog( int size )
    {
       String[] titles =
         {
           "Greatest Hits Volume 1", "Greatest Hits Volume 2",
           "The Best Of", "Love Songs",
           "The Early Years", "The Later Years"
         };
       String[] artists = 
         {
           "Michael Jackson", "Eminem",
           "The Beatles", "Shania Twain",
           "Limp Bizkit"
         };
       ArrayList<MusicMedia> catalog = new ArrayList<MusicMedia>();
       Random rn = new Random();
       for ( int i = 0 ; i < size ; i++ )
       {
            MusicMedia m;
            int mediatype = rn.nextInt( 3 );
            String title = titles[ rn.nextInt( titles.length ) ];
            String artist = artists[ rn.nextInt( artists.length ) ];
            String sku = "1234-" + i;
            if ( mediatype == 0 )
                m = new CompactDisk( title, artist, sku );
            else if ( mediatype == 1 )
                m = new DigitalVideoDisk( title, artist, sku );
            else
                m = new CassetteTape( title, artist, sku );
            catalog.add( m );
       }
      return catalog;
    }   

    public static String lookupMedia( ArrayList<MusicMedia> catalog,
        String sku )
    {
        for ( MusicMedia m : catalog )
        {
            if ( m.getSKU().equals( sku ))                  
            return "SKU is in catalog";
        }
        return "SKU not in catalog";
    }

    public static String detailedInventory( ArrayList<MusicMedia> catalog, ArrayList<Bin> warehouse )
    {
        String s = "";
        for ( Bin bn : warehouse ){
            s += "Bin " + bn.getName() + ":\n"; 
              for (int i = 0; i < bn.getContents().size(); i++){
                s += catalog.get(i) + ", " + bn.getContents().get(i) + "\n";
              }
        }
        s += "\n";
        return s;
    }  

    public static void main( String[] args )
    {
        ArrayList<MusicMedia> catalog = MakeMusicCatalog( 10 );
        ArrayList<Bin> warehouse = new ArrayList<Bin>();
        Bin a = new Bin( "A" );
        Bin b = new Bin( "B" );
        warehouse.add( a );
        warehouse.add( b );
        a.add( new BinItem( "1234-0", 500 ) );
        a.add( new BinItem( "1234-1", 25 ) );
        a.add( new BinItem( "1234-2", 7720 ) );
        b.add( new BinItem( "1234-3", 1000 ) );
        System.out.println( detailedInventory( catalog, warehouse ) ); 
    }
} 

完整代码:

public class MusicMedia
{
    private String myTitle,
    myArtist,
    mySKU;

    public MusicMedia( String title, String artist, String sku )
    {
        myTitle = title;
        myArtist = artist;
        mySKU = sku;
   }
   public String getTitle()
   {
       return myTitle;
    }
    public String getArtist()
    {
        return myArtist;
    }
    public String getMediaType()
    {
        return "Unknown";
    }
    public String getSKU()
    {
        return mySKU;
    }
    public String toString()
    {
        return " - " + getTitle() + " (" + getArtist() + ")";
    }
}


  public class Disk extends MusicMedia
{
    /**
     * Constructor for objects of class Disk
     */
    public Disk( String title, String artist, String sku )
    {
        super(title, artist, sku);
    }

    public String getMediaType()
    {
        return "Disk"; 
    }

    public String toString()
    {
        return getMediaType() + super.toString();
    }
}

我也有一个相同的CassetteTape类,它也扩展了MusicMedia。另外还有磁盘的其他两个子类,称为CompactDisk和DigitalVideoDisk。这两个也几乎相同,所以我粘贴了下面的DVD类。

public class DigitalVideoDisk extends Disk
{
    /**
     * Constructor for objects of class DigitalVideoDisk
     */
    public DigitalVideoDisk( String title, String artist, String sku )
    {
        super(title, artist, sku);
    }

    public String getMediaType()
    {
        return "DVD";
    }
}
public class BinItem
{
    private String mySKU;
    private int    myQuantity;
    public BinItem( String sku, int quantity )
    {
        mySKU = sku;
        myQuantity = quantity; 
    }
    public String getSKU()
    { 
        return mySKU;
    }
    public int getQuantity()
    {
        return myQuantity;
    }
    public String toString()
    {
        return "SKU " + getSKU() + ": " + getQuantity();
    }
 }

public class Bin
{
    private String  myName; //bin name
    private ArrayList<BinItem>   myContents; //contains a binItem
    public Bin( String name )
    {
      myName = name;
      myContents = new ArrayList<BinItem>();
    }   
    public String getName()
    {
      return myName;
    }    
    public ArrayList<BinItem> getContents()
    {
        return myContents;
    }   
    public void add( BinItem b )
    {        
        myContents.add( b );
    }    
    public String toString()
    {
        String s = "Bin " + myName + ":\n";
        for ( BinItem b : myContents )
            s += b + "\n";
        return s;
    }
}

0 个答案:

没有答案