[java] coda di tracce, la classe thread non esegue correttamente
Codice:
import javax.sound.sampled.*; Codice:
import java.io.File; l problema è Codice:
pt.getPlayer().addToQueue(l); Codice:
if (this.q.size() > 0) { perchè? |
non capisco il ciclare che lavora per nulla. Non hai a disposizione gli eventi i quali ti avvertono quando c'è qualcosa da fare?
|
Quote:
dovrei implementare un pattern Observer/Listeners ma non so come si fa. |
Se decommento la riga
Codice:
//System.out.println("size: " + this.q.size() + " isplaying: " + this.isPlaying); |
Quote:
|
Se non vado errato (e potrebbe esserlo dato la giornata pesantina che mi è toccata) la soluzione al tuo problema potrebbe essere contenuta in tre parole: Java Memory Model :D
Hai una struttura dati condivisa - una linked list - in cui un thread (il main thread) pusha elementi - e un thread (il consumer, quello che gira in attesa degli eventi) consuma gli elementi pushati. Oltre alla giusta osservazione che ti hanno già fatto notare (un while true del genere usa CPU come se non ci fosse un domani) dal punto di vista del memory model Java non c'è garanzia di sincronizzazione tra la push di elemento e la pull dell'elemento stesso. In gergo tecnico questa è detta Happens-Before che garantisce che una modifica fatta da un thread sia visibile ad altri thread. In questo senso, non vi è garanzia alcuna che l'inserimento di un elemento nella coda sia visibile ad altri threads. Che ne dici? Quadra? |
Penso tu abbia ragione :)
Solo che io ho un solo thread, quello che cicla infinitamente. La stranezza è che se provo ad aggiungere una system out il tutto funziona invece se non aggiungo alcuna system out ma una qualsiasi altra istruzione come i++ non funziona. |
ma tu non puoi sapere come schedula i processi un SO, per questo devi complicarti leggermente la vita ed usare gli eventi, altrimenti non ne esci.
Qualsiasi programma che usa dati provenienti da socket usa gli eventi. Ci sono dati da leggere? Leggo i dati e poi faccio altre cose. Se vuoi proprio usare un while infinito, devi creare un thread specifico che cicla per fatti suoi e magari tenerlo fermo se non c'è nulla da fare, ma è pur sempre una strada sbagliata, si sprecano risorse. E' come se costantemente interrogassi la scheda di rete anche se non ci sono dati in arrivo, lavoro iutile. Lascia fare all'SO, è il suo compito e lo sa fare meglio di noi. Usa gli eventi ;) |
Quote:
|
Eventualmente potresti descrivere anche gli intenti del codice che hai scritto? Perché, oltre alla marea di errori di copia/incolla per cui non compila, è anche di difficile comprensione, le variabili sono male dichiarate e potrei andare avanti.
Onestamente non credo che questo forum sia il luogo adatto in cui sottoporre il proprio codice a peer-review, ma anche solo per avere un parere preciso un malcapitato sviluppatore che vuole aiutarti si ritrova a dovere decifrare codice e alla fine desiste ... postare qualcosa di leggibile e magari commentato (nel codice o post) è nell'interesse soprattutto tuo, per ricevere più risposte. Tutto questo detto in modalità Peace And Love, sia chiaro :) Un paio di suggerimenti: 1) prima fixerei la versione di codice che hai già scritto. sei conscio che hai scritto un codice multithreaded e che quindi sono obbligatorie tutte le accortezze del caso? Hai esperienza con lo sviluppo multithread? 2) fatto questo, puoi provare eventualmente a passare ad utilizzare, come ti è stato suggerito, IO asincrono; però vedi tu se nel contesto di utilizzo della tua applicazione un IO bloccante risulta un collo di bottiglia ... |
Quote:
Ora provo a togliere il thread e capire se l'errore sta nel popolare la coda. In che senso "difficile comprensione, le variabili sono male dichiarate e potrei andare avanti." La maggior parte del codice che è scritto è preso dagli esempi di oracle... va be, è difficile solo la parte dell'AudioInputStream. |
Cambia l'implementazione della coda utilizzando una struttura dati threadsafe e dovresti risolvere. La print su stdout aggiunge sincronizzazione, che dal punto di vista del memory model rendono visibili gli elementi pushati sulla coda al thread consumer. Ma, certamente, un codice non può fare affidamento sulla presenza di print su stdout per ottenere il risultato desiderato.
|
Quote:
Seguirò il tuo consiglio. Grazie per il supporto |
[Risolto][Java] coda di tracce, la classe thread non esegue correttamente
Quote:
Grazie @misterx e @Lampo89 del supporto |
Prego. In realtà non è l'unico problema di sincronizzazione per cui verificherei per bene tutto quanto, anche se in realtà mi immagino che questa classe sia il prototipo "work in progress" per una funzionalità più complessa e quindi ulteriori osservazioni potrebbero non valere ...
|
Tutti gli orari sono GMT +1. Ora sono le: 11:32. |
Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
Hardware Upgrade S.r.l.