Question

我创建了这个包，问题是什么时候运行它会在bin之间重复相同的目录条目。我认为问题出在详细的Inventory方法中。应该是：它应该为Bin B创建一个新的随机bin项（类型，标题，艺术家），而不是使用Bin A中的相同元素。除此之外，输出是正确的。正确生成并显示SKU编号。

**当i对应目录时，它们是对象。它们实际上是不同类型的物理媒体，例如DVD或磁带，具有标题，艺术家和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相同。

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 ) ); 
    }
}

我将bin中共享相同SKU的所有项目的集合称为“bin项目”。 Bin类的每个对象（未示出）表示假装仓库中的bin。 Bin对象有两个实例变量：一个包含bin名称的String和一个包含bin中存储的每个SKU的BinItem的ArrayList。 Bin类的toString方法使用BinItem类的toString方法。它生成的列表仅涉及SKU和数量。

此方法应使用for-each循环来循环仓库中的所有箱柜。对于每个bin，s应该由String＆＃34; Bin＆＃34;然后是bin的名称，后跟冒号和\ n以开始新行。然后它应循环遍历当前bin中的所有bin项目通过新的文本行扩展s，该文本行以在输入目录中查找当前bin项目的SKU的结果开始，并继续使用逗号，后跟bin项目的String表示。

输出应如下所示：

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

完整代码：

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;
    }
}

Answer 1

好的，根据你的编辑：

...在输入目录中查找当前bin项目的SKU的结果......

你现在没有这样做，这是我们需要了解程序应该做什么的关键点。现在，您只是使用catalog从i检索元素，基本上是任意的。

因此，您需要做的第一件事就是创建一个帮助方法，在catalog中搜索某个SKU的MusicMedia对象。你有一个非常类似于lookupMedia的方法，所以我只是把它修改为这个稍微不同的规范。这将返回m而不是String值：

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

现在您可以根据SKU检索项目，您可以修改detailedInventory循环以使用它：

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

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

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

（不知道如何从BinItem获取SKU所以我猜到了，但我希望你能得到这个想法。如果你没有一个方法可以返回SKU，你可能需要制作一个。）

Java嵌套for循环的重复输出

1 个答案: