Your constructor returns void and is misspelled.
Code: Select all
public void allitems(String name){
this.name = name;
}
There should not be a semicolon after for and you're suddenly using "movies" instead of "videos". Also, array.length always returns the length of the array, in this case 100, causing your app to crash.
Code: Select all
public void list(String s){
if(s == "book")
for(int x = 0 ; x<books.length ; x++);
books[x].printDetails();
if(s == "music")
for(int x = 0 ; x<music.length ; x++);
music[x].printDetails();
if(s == "video")
for(int x = 0 ; x<movies.length ; x++);
movies[x].printDetails();
if(s == "software")
for(int x = 0 ; x<softwares.length ; x++);
softwares[x].printDetails();
}
The first error's still there though...
A couple of other notes, please state the error when you're asking us to figure it out, it's kind of annoying to compile it ourself when we could've figured the mistakes out by seeing the compiler output.
Your indentation is often completely random and you're mixing whitespace and tabs. It's almost as if different persons wrote the code, one used tabs and the other spaces.
The
Java class naming convention is camel-case, which means your classes should be named this way:
Code: Select all
class Application { ... }
class AllItems { ... }
class Product { ... }
...
Also, your member variables are all public, which is the opposite to the
java programming practises. Keep the members private and write get/set functions (or whatever the situation requires).
An array is not a good data structure for what you're doing, in fact as you seem to have realized at some point, it will crash when you add more than 100 products of one type. To fix that, I suggest using one of these structures instead:
http://java.sun.com/j2se/1.4.2/docs/api ... yList.html,
http://java.sun.com/j2se/1.4.2/docs/api ... ector.html .
helppppppp
