Programma biblioteca java

[mcfede88]

Salve a tutti,
ho iniziato a studiare java dal libro di Deitel, e mi è venuta l'intenzione di scrivere un semplice programma che prendesse in input i titoli e la durata di alcuni film.
Sicuramente c'è qualcosa che non ho compreso bene della programmazione ad oggetti visto che vengo dal C.
Ho provato tutto ma non ne vengo a capo.
Grazie mille

Ho scritto la classe beni:

public class beni {
	private String titolo;
	private String anno;
	
	
	public beni (String titolo, String anno) {
		this.titolo = titolo;
		this.anno = anno;
	}
public String getTitolo(){
	return titolo;
}

public String getAnno(){
	return anno;
}

public void setTitolo(String p){
	titolo 
= p;
}

public void setAnno(String p){
	anno = p;
}

public String getBeni(){
	return titolo+ " " +anno;
}


}

poi ho scritto il main per provare questa classe, visto che è una prova è scritta senza un ciclo for o while

import java.util.Scanner;
public class test {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner utente = new Scanner (System.in);
		
		beni uno = new beni(String titolo, String anno);
		
		
		System.out.println("Inserisci nuovo bene.\n Nome:");
		uno.setTitolo=utente.next();
		System.out.println("Anno:");
		uno.setAnno=utente.next();
		
		

		 
		 
		 System.out.println(beni.getBeni);

	}

}

[Patrick Jane]

Originariamente inviato da mcfede88
Salve a tutti,
ho iniziato a studiare java dal libro di Deitel, e mi è venuta l'intenzione di scrivere un semplice programma che prendesse in input i titoli e la durata di alcuni film.
Sicuramente c'è qualcosa che non ho compreso bene della programmazione ad oggetti visto che vengo dal C.
Ho provato tutto ma non ne vengo a capo.
Grazie mille

Ottieni errori in fase di runtime o di compilazione?

Ricordati sempre che una classe dichiarata come public deve essere dichiarata in un file a se stante.
E ricordati che il nome della classe deve essere uguale al nome del file.

Così a occhio, l'errore che ti ho trovato dovrebbe essere qui

beni uno = new beni(String titolo, String anno);

Li devi passare 2 stringhe, non creare 2 stringhe!!
Ovvero, dovresti passare

beni uno = new beni(titolo,anno);

Ricordati che un costruttore inizializza un oggetto durante la creazione.

Ovviamente devi ottenere prima il loro valore

[mcfede88]

ho fatto ma mi da sempre errore, su uno di setTitolo e setAnno ed errore su titolo, anno di beni uno = new beni.

Ci deve essere qualche errore nei metodi set o nello scanner.

Perchè se faccio

beni uno = new beni ("Federico","100");
System.out.println(uno.getBeni());

Il tutto funziona

[Patrick Jane]

Non mi è per nulla chiaro perchè chiami il metodo in questo modo

uno.setTitolo=utente.next();

Per settare il titolo ti basterebbe fare

uno.setTitolo();

PS: posta l'errore che ottieni

[mcfede88]

chiamo

 uno.setTitolo=utente.next();

perchè lo devo prendere da tastiera.

comunque ho cambiato come dici tu e mi dici
"uno cannot be resolved"
"titolo cannot be resolved"
"anno cannot be resolved"

[Patrick Jane]

Commetti i classici errori di chi passa da un linguaggio come C a Java.

i metodi setTitolo() e setAnno() ricevono degli argomenti, quindi devi chiamarli inserendo degli argomenti.

Il tuo codice corretto

import java.util.Scanner;
class beni {
	private String titolo;
	private String anno;
	
	
	public beni (String titolo, String anno) {
		this.titolo = titolo;
		this.anno = anno;
	}
public String getTitolo(){
	return titolo;
}

public String getAnno(){
	return anno;
}

public void setTitolo(String p){
	titolo = p;
}

public void setAnno(String p){
	anno = p;
}

public String getBeni(){
	return titolo+ " " +anno;
}


}

public class test {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner utente = new Scanner (System.in);
		
		beni uno = new beni("Ciao","10");
		
		
		System.out.println("Inserisci nuovo bene.\n Nome:");
		uno.setTitolo(utente.next());
		System.out.println("Anno:");
		uno.setAnno(utente.next());
		
		

		 
		 
		 System.out.println(uno.getBeni());

	}

}

utente.next() devi chiamarlo all'interno della funzione setTitolo() e setAnno() in modo tale che il dato inserito venga passago come parametro al metodo
Comunque ti consiglio di controllare i dati in input.

[mcfede88]

ok grazie,
ma non ho risolto un errore
se io cambio

beni uno = new beni("Ciao","10");

in

beni uno = new beni (titolo, anno);

mi da sempre
anno e titolo cannot be resolved

[Patrick Jane]

Originariamente inviato da mcfede88
ok grazie,
ma non ho risolto un errore
se io cambio

beni uno = new beni("Ciao","10");

in

beni uno = new beni (titolo, anno);

mi da sempre
anno e titolo cannot be resolved

Bè scusa mi sembra normale!!
Come fai a passare titolo ed anno se non esistono?
Nel costruttore devi sempre utilizzare utente.next() sia per titolo che per anno. Oppure puoi creare 2 stringhe e ricevere li i valori:
String nome = utente.next();
String anno = utente.next()

beni uno = new beni(nome,anno);

è buona norma l'iniziale maiuscola per il nome della classe

[mcfede88]

Continuo con la stesura del mio programma.
Praticamente devo gestire sia libri che dvd quindi ho fatto delle classi DVD e libri che estendono la classe beni.

Adesso vorrei che il mio programma assegnasse un id ai DVD che inserisco:
Esempio:
Adesso io ho questo:

	public static void InserimentoDVD (){
		
	
	Scanner utente = new Scanner (System.in);
		
	System.out.println("Inserisci nuovo DVD.\n Titolo: ");
	String nome = utente.next();
	
	System.out.println("Anno:");
	String anno = utente.next();
	
	System.out.println("Durata:");
	int durata = utente.nextInt();
	

	DVD uno =  new DVD(nome, anno, durata);
	uno.setTitolo(nome);
	uno.setAnno(anno);
	uno.setDurata(durata);
	
 }

E vorrei fare una cosa del genere solo che non riesco a farla funzionare:

	public static void InserimentoDVD (){
		
	
	Scanner utente = new Scanner (System.in);
		
	System.out.println("Inserisci nuovo DVD.\n Titolo: ");
	String nome = utente.next();
	
	System.out.println("Anno:");
	String anno = utente.next();
	
	System.out.println("Durata:");
	int durata = utente.nextInt();
	int idDVD = 0;

	DVD [idDVD] =  new DVD(nome, anno, durata);
	[idDVD].setTitolo(nome);
        [idDVD].setAnno(anno);
	[idDVD].setDurata(durata);
        idDVD++;
	}

[Patrick Jane]

	int idDVD = 0;

	DVD [idDVD] =  new DVD(nome, anno, durata);
	[idDVD].setTitolo(nome);
        [idDVD].setAnno(anno);
	[idDVD].setDurata(durata);
        idDVD++;
	}

Il primo problema che vedo è sicuramente la creazione dell'istanza.
Usi un riferimento DVD (ti pregherei di postare anche la classe DVD, in modo da poter vedere il suo funzionamento ), ma non crei la variabile d'istanza.

Non mi è chiaro perchè utilizzi un costruttore passando i dati appena ricevuti dall'input (nome, anno, durata), e poi sotto chiami i metodi..

Prima di poterti essere d'aiuto ho bisogno di sapere perchè vuoi utilizzare un array di DVD..