JS: perché "Undefined"?

😈╰•ALBERTO•╯😈

Non riesco a trovare il problema da solo


<html>


<head>

</head>




<body>

<center>
<h1>Magazzino</h1>
<label>Codice del prodotto:</label><br><input id="text_uno" type="text" placeholder="Massimo 12 caratteri" maxlength="12"/> <br>
<br>
<label>Descrizione del prodotto:</label><br><textarea id="text_due" placeholder="Massimo 50 caratteri" maxlength="50"></textarea> <br>
<br>
<label>Quantità magazzino:</label><br><input id="text_tre" type="text" placeholder="es: 3"/> <br>
<br>
<label>Quantità sottoscorta:</label><br><input id="text_quattro" type="text" placeholder="es: 10"/> <br>
<br>
<button onclick="confermaordine();">Conferma ordine</button>
</center>


<script>

class Articolo {

	constructor(){

		this.codice = "";
		this.descrizione = "";
		this.quantitaMagazzino = 0;
		this.sottoScorta = 0;

	}

	verificaCodice(){
		if(this.codice.length > 12){
			return "Hai superato il numero massimo di caratteri consentiti!".fontcolor("red");
		}
	}
	
	verificaDescrizione(){
		if(this.descrizione.length > 50){
			return "Hai superato il numero massimo di caratteri consentiti!".fontcolor("red");
		}
	}	
	
	verificaQuantitaMagazzino(){
		if (this.quantitaMagazzino < this.sottoScorta){
			return (this.sottoScorta - this.quantitaMagazzino).fontcolor("red");
		}
	}
	acquistoDaFornitore(quantita){
		this.quantitaMagazzino += quantita;
	}
	
	ordineCliente(quantita){
		if(this.quantitaMagazzino < quantita){
			return (this.quantitaMagazzino - quantita + this.sottoScorta);
		
	
		}else{
		
			this.quantitaMagazzino -= quantita;
				return 0;
			}
		}
	
	}
	

class Magazzino{


	constructor(){
	
		this.articoli = [];
	}
	
	acquisto(articolo,quantita){
	
			let miaQuantita = articolo.ordineCliente(quantita);
			if(miaQuantita > 0){
				return "Emetti un ordine di " + miaQuantita;
		
		}
			
			
	
	}
	
	aggiungiArticoli(articolo){
	
		this.articoli.push(articolo);
		
	
	
	}

}
let articolo1 = new Articolo();

	articolo1.codice = "";
	articolo1.descrizione = "";
	articolo1.quantitaMagazzino = 0;
	articolo1.sottoScorta = 0;
	
let articolo2 = new Articolo();

	articolo2.codice = "";
	articolo2.descrizione = "";
	articolo2.quantitaMagazzino = 0;
	articolo2.sottoScorta = 0;
	
let magazzino = new Magazzino();

	magazzino.aggiungiArticoli(articolo1);
	magazzino.aggiungiArticoli(articolo2);
	
	
	
function confermaordine(){
	
		
	let codiceProdotto = document.getElementById("text_uno").value;
	let descrizioneProdotto = document.getElementById("text_due").value;
	let quantitaMagazzino = parseInt(document.getElementById("text_tre").value);
	let quantitaSottoscorta = parseInt(document.getElementById("text_quattro").value);
	
	articolo1.codice = codiceProdotto;
	articolo1.descrizione = descrizioneProdotto;
	articolo1.quantitaMagazzino = quantitaMagazzino;
	articolo1.quantitaSottoscorta = quantitaSottoscorta;
	
	let ordine = magazzino.acquisto(articolo1, quantitaMagazzino);
	
	alert(ordine);
	
}
	
		

	
</script>


</body>


</html>

@Giulio_M

Giulio_M

😈╰•ALBERTO•╯😈 occorre secondo me rivedere la funzione acquisto(articolo,quantita), infatti lasciando come hai scritto tu:
let ordine = magazzino.acquisto(articolo1, quantitaMagazzino); alert(quantitaMagazzino);
in pratica risponde "undefined" poiché non interpreta bene il contenuto di alert().
Mentre se ad esempio scrivi alert(quantitaMagazzino) funziona corettamente, vale a dire che anziché farlo tramite funzione (o magari lo farai dopo), prima puoi provare a "costruire" l'output dell'alert ad esempio in questo modo:
alert("Ordine con codice: "+codiceProdotto+"\nquantità sottoscorta: "+quantitaSottoscorta);
Oppure lo personalizzi come vuoi, il concetto comunque è questo, così funziona.

Ah, poi aggiungo: non usare il tag HTML <center>, oggi è deprecato. Meglio racchiudere tutto in un blocco <div> e applicare la proprietà di centratura tramite CSS (inline o in blocco che sia). Quindi ad esempio, al posto di <center> ... </center> basta che scrivi: <div style="text-align:center"> ... </div>

😈╰•ALBERTO•╯😈

Giulio_M

Ok grazie, poi provo e ti faccio sapere