Salve ragaazzi! è brutto essere alle prime armi! mi viene spontaneo programmare con le regole del java.... dimentico che in android occorre rispettare la "discendenza" activity, intent, service ecc...
In pratica io ho programmato sempre in modo modulare. e vorrei fare così anche con android.
In pratica, io ho vari alertdialog che come spiegato nel tutorial li costruisco in uno switch e li richiamo nell'oncreate() con showdialog(int).
Ora io questo lo devo modularizzare, nel senso che devo poter accedere allo switch ogni qual volta che mi serve richiamare un alertdialog. Magari col codice mi spiego meglio:
codice alertdialogsystem dove è implementato lo switch, uguale a quello del tutorial, e a cui ringrazio Orochi :
questo invece è il codice esterno a AlertDialogsytem, in cui voglio chiamare l'alertdialog con int=DIALOG_PAUSED_ID. In pratica quando viene premuto il pulsante "si", dovrebbe apparire l'alertdialog. Ho fatto verie prove. Non badate all'uso del StartActivityOfNeeded() è una delle tante prove.codice:import android.app.Activity;
import android.app.AlertDialog;
import android.app.Dialog;
import android.content.DialogInterface;
import android.os.Bundle;
public class AlertDialogsystem extends Activity{
static final int DIALOG_PAUSED_ID = 0;
static final int DIALOG_GAMEOVER_ID = 1;
public Dialog onCreateDialog(int id) {
Dialog dialog = null;
switch(id) {
case DIALOG_PAUSED_ID:
// do the work to define the pause Dialog
AlertDialog.Builder builder = new AlertDialog.Builder(this);
builder.setMessage("Are you sure you want to exit?")
.setCancelable(false)
.setPositiveButton("Yes", new DialogInterface.OnClickListener() {
public void onClick(DialogInterface dialog, int id) {
// MyActivity.this.finish();
}
})
.setNegativeButton("No", new DialogInterface.OnClickListener() {
public void onClick(DialogInterface dialog, int id) {
dialog.cancel();
}
});
dialog = builder.create();
break;
case DIALOG_GAMEOVER_ID:
// do the work to define the game over Dialog
break;
default:
dialog = null;
}
return dialog;
}
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
showDialog(DIALOG_PAUSED_ID);
}
}
mi scuso se mi son dilungato un pò troppo nello spiegare.codice:si.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
int i=AlertDialogsystem.DIALOG_PAUSED_ID;
//showDialog(i, savedInstanceState);
//AlertDialogsystem.showDialog(i);
Intent audio_si=new Intent();
startActivityIfNeeded(audio_si, i);
}
});
Grazie in anticipo a chi saprà aiutarmi.
modularizzare codice android
Visualizzazione stampabile
-
03-01-11, 11:50allmysterymodularizzare codice android
-
03-01-11, 13:17Cymon
Perdona, non mi è tutto chiaro, ma ci provo... :-[
Tu vorresti un oggetto che fabbrichi dialog in base a un ID che gli passi tu così da centralizzare tutti gli alert che usi nella tua applicazione.
A questo punto devi proprio pensare in JAVA e non in termini di android ;)
Non è che perchè ti trovi nel framework android TUTTO deve essere una activity. Deve essere una activity tutto quello che deve essere gestito tramite un'intent(e quindi deve essere registrato sul manifest).
Quindi crea pure la tua AlertDialogsystem come una classe JAVA qualsiasi e poi usala all'interno delle activity come meglio credi (o perché i suoi metodi sono static o perché ne instanzi un oggetto...) -
03-01-11, 15:37allmystery
si si hai capito bene!
la prima cosa che ho fatto è pensarla in termini di java. allego i due file che contengono AlertDialogsystem che contiene lo switch e audio da cui richiamo un alert nello switch.
http://www.softmystery.altervista.org/audio.rar
Dato che non funziona , allora ho pensato che forse dovevo trattarli come activity/intent....
cmq nel file AlertDialogsystem se scrivo
AlertDialog.Builder builder=new AlertDialog.Builder(this);
eclipse dà errore not suggestion available -
03-01-11, 16:49CymonIl costruttore AlertDialogi.Builder vuole come parametro un context. Quindi non puoi usare this a meno che tu non lo stia invocando da dentro un'activity (che è un context).Quote:
Originariamente inviato da allmystery
Il modo migliore per svicolare da questa situazione è inserire nel costruttore del AlertDialogSystem il parametro context e salvarlo in un attributo della classe, tipo cntx.
Quando andrai a instanziare la classe da dentro l'activity basterà crearla come
A questo punto quando invochi l'AlertDialog.Builder puoi usare:
E' molto consueto dover costruire tutto con un context, è il modo in cui si può portare il framework android "fuori" dalle classi proprie del suo ciclo di vita. -
03-01-11, 17:33allmystery
ok, il context è sistemato ma comunque non visualizza gli alertDialog
http://www.softmystery.altervista.org/audio1.rar -
03-01-11, 20:05Orochi
Non so se c'entra con il problema che continua a non funzionare, ma se posso intervenire suggerirei di non conservare una variabile context dentro il tuo AlertDialogSystem ma piuttosto di passarlo da fuori quando ti serve, ovvero ogni volta che lo invochi. Questo perchè il context è un oggetto "pesante" che rischia di restare bloccato se per caso l'activity da cui l'hai ricavato viene distrutta (ad esempio perchè è ruotato il telefono) e quindi la vecchia istanza resterebbe appesa e non "pulita" dal garbage collector, a lungo andare questo provoca dei memory leak che possono consumarti e ridurre la memoria disponibile nell'applicazione!
Magari se è una piccola applicazione non ci fai caso, ma è meglio sempre sapere queste cose ;)
Vi linko un articolo interessante sempre dalla nostra bibbia:
Avoiding Memory Leaks | Android Developers -
03-01-11, 21:28allmystery
ok grazie del consiglio!
ecco i codici, in cui c'è ancora qualcosa che impedisce la visualizzazione dell'alterdialog:
e il "main"codice:import android.app.AlertDialog;
import android.content.Context;
import android.content.DialogInterface;
import android.view.View.OnClickListener;
public class AlertDialogsystem {
static final int DIALOG_PAUSED_ID = 0;
static final int DIALOG_GAMEOVER_ID = 1;
// static Dialog dialog = null;
AlertDialog.Builder builder ;
public AlertDialogsystem(OnClickListener onClickListener,int id) {
switch(id) {
case DIALOG_PAUSED_ID:
// do the work to define the pause Dialog
//builder= new AlertDialog.Builder(null);
builder=new AlertDialog.Builder((Context) onClickListener);
builder.setTitle("prova");
builder.setMessage("prova messaggio");
builder.setPositiveButton("ok", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int which) {
// TODO Auto-generated method stub
}
});
builder.setNegativeButton("cancel", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int which) {
// TODO Auto-generated method stub
}
});
builder.create();
builder.show();
break;
case DIALOG_GAMEOVER_ID:
// do the work to define the game over Dialog
break;
default:
builder= null;
}
}
}
è da stamattina che stò impazzendo, ma non funziona!!!!!!!!:'(:(codice:import android.app.Activity;
import android.app.AlertDialog;
import android.os.Bundle;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Button;
public class audio extends Activity {
/** Called when the activity is first created. */
AlertDialog.Builder builder ;
@Override
public void onCreate(final Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.audio);
//se clicca su si flag=si
//se clicca su no flag=no
//prima di cambiare il valore del flag attraverso il dialog label l'utente confermerà la propria azione
//nelle prox schermate fare l'if flag==si allora emana audio se invece è no allora non gestire audio
//richiamo bottoni da audio.xml
final Button si=(Button)findViewById(R.id.buttonsi);
Button no=(Button)findViewById(R.id.buttonno);
//associo un onclick ad ogni button devo gestire il dialog label
si.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
int i=AlertDialogsystem.DIALOG_PAUSED_ID;
new AlertDialogsystem(this,i);
}
});
no.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
}
});
}
}
-
04-01-11, 09:27Cymon
Aspetta un attimo... ma nell'Activity è scomparso sia l'onCreateDialog che la showDialog... perché dovrebbe funzionare? :p
-
04-01-11, 10:48allmystery
Alertdialogsystem è un semplice codice java(seguendo il cosiglio all'inizio del post) showdialog() l'ho sostituito con .show() e createdialog l'ho eliminato....
ok dai ora provo a cambiare. ma comunque non riesco a capire dove dovrei mettere showdialog(). Alertdialogsystem non è un activity e quindi non ha oncreate() con cui chiamare showdialog(). ovviamente correggetemi se scrivo fesserie! -
04-01-11, 11:05allmystery
Fatto! ma senza risultato... lo showdialog davvero non sò dove metterlo!
alertdialgsystem
audiocodice:import android.app.AlertDialog;
import android.content.Context;
import android.content.DialogInterface;
import android.view.View.OnClickListener;
public class AlertDialogsystem {
static final int DIALOG_PAUSED_ID = 0;
static final int DIALOG_GAMEOVER_ID = 1;
AlertDialog.Builder builder ;
public void onCreateDialog(OnClickListener onClickListener,int id) {
//public AlertDialogsystem(OnClickListener onClickListener,int id) {
switch(id) {
case DIALOG_PAUSED_ID:
builder=new AlertDialog.Builder((Context) onClickListener);
builder.setTitle("prova");
builder.setMessage("prova messaggio");
builder.setPositiveButton("ok", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int which) {
// TODO Auto-generated method stub
}
});
builder.setNegativeButton("cancel", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int which) {
// TODO Auto-generated method stub
}
});
builder.create();
builder.show();
break;
case DIALOG_GAMEOVER_ID:
// do the work to define the game over Dialog
break;
default:
builder= null;
}
}
}
lo sò che sono io a sbagliare a programmare... ma non capisco dove!codice:import android.app.Activity;
import android.app.AlertDialog;
import android.os.Bundle;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Button;
public class audio extends Activity {
/** Called when the activity is first created. */
AlertDialog.Builder builder ;
@Override
public void onCreate(final Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.audio);
//richiamo bottoni da audio.xml
Button si=(Button)findViewById(R.id.buttonsi);
Button no=(Button)findViewById(R.id.buttonno);
si.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
int i=AlertDialogsystem.DIALOG_PAUSED_ID;
AlertDialogsystem dialog=new AlertDialogsystem();
dialog.onCreateDialog(this, i);
//new AlertDialogsystem(this,i);
}
});
no.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
}
});
}
}
-
04-01-11, 11:37Cymon
Feeeeermi tutti.
La onCreateDialog() non è che puoi metterla a piacere in una classe e pensare che funzioni.
La onCreateDialog(int i) è un metodo proprio di una Activity e fa parte del ciclo di vita delle attività. Quindi deve essere necessariamente implementata su una classe che estenda Activity.
Quando all'inizio ti ho detto che AlertDialogsystem non doveva essere necessariamente una Activity intendevo che il wrapper che deve contenere la logica di fabbricazione i dialog non deve essere necessariamente una activity, ma la onCreateDialog(int i) DEVE necessariamente essere inserito in un'activity.
Quindi inserisci la onCreateDialog nella activity e lì dentro invochi la AlertDialogsystem dove fabbrichi il dialog.
la showdialog la metti nell'activity dove... bhe... dove deve apparire il dialog. -
04-01-11, 11:37Orochi
Io modificherei la classe AlertDialogSystem per farla funzionare a mo' di factory, ovvero costruire solo la dialog e la restituisce fuori, il codice che la invoca e la mostra resta nell'activity, la classe ti servirà solo a costruire il dialog dove serve.
-
04-01-11, 12:01CymonEsattamente il mio pensiero! :)Quote:
Originariamente inviato da Orochi
-
04-01-11, 16:57allmystery
ok Grazie ragazzi! mi avete non solo chiarito le idee, ma ridato un pò di speranza !!!!! :D
Per completezza , posto i codici funzionanti! Volevo seguire il consiglio di Orochi sul context, ma non sò ancora se quella che stò sviluppando dovrà essere un'applicazione "pesante"
alertdialogcodice:public class audio extends Activity {
/** Called when the activity is first created. */
AlertDialog.Builder builder ;
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.audio);
//richiamo bottoni da audio.xml
Button si=(Button)findViewById(R.id.buttonsi);
Button no=(Button)findViewById(R.id.buttonno);
si.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
int i=AlertDialogsystem.DIALOG_PAUSED_ID;
onCreateDialog(i);
showDialog(i);
}
});
no.setOnClickListener(new OnClickListener(){
@Override
public void onClick(View arg0) {
// TODO Auto-generated method stub
}
});
}
}
public Dialog onCreateDialog( int i){
AlertDialogsystem alert=new AlertDialogsystem();
Context cnt=this;
return alert.CreateDialog(cnt,i);
}
}
codice:public class AlertDialogsystem {
static final int DIALOG_PAUSED_ID = 0;
static final int DIALOG_GAMEOVER_ID = 1;
AlertDialog.Builder builder ;
Dialog dialog;
public Dialog CreateDialog(Context cnt,int id) {
switch(id) {
case DIALOG_PAUSED_ID:
builder=new AlertDialog.Builder( cnt);
builder.setTitle("prova");
builder.setMessage("prova messaggio");
builder.setPositiveButton("ok", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int which) {
// TODO Auto-generated method stub
}
});
builder.setNegativeButton("cancel", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int which) {
// TODO Auto-generated method stub
}
});
dialog= builder.create();
break;
case DIALOG_GAMEOVER_ID:
// do the work to define the game over Dialog
break;
default:
builder= null;
}
return dialog;
}
}
-
04-01-11, 18:53CymonCosì ci fai sentire Emergency roftlQuote:
Originariamente inviato da allmystery
P.S. Con tag JAVA invece del tag CODE puoi postare codice "colorato"*, più leggibile.
*: Che ci crediate o no, nonostante tutto il mio purismo, contro il codice colorato non ho niente. Anche se un giorno vi capiterà di lavorare via terminale monocromatico in vi puro, quindi tenete sempre a mente che potreste non averlo a disposizone. :p -
04-01-11, 21:04allmystery
ok lo terrò a mente ;)
-
05-01-11, 11:51allmystery
Rieccomi.... non sò se aprire un nuovo post, oppure continuare questo, perchè non ho capito se la questione adesso si è spostata sul context, oppure è rimasta la modularizzazione del codice. Magari poi segnalatemi voi...
La questione è questa: (piccolo riassunto) Nel file AlertDialogsystem io genero i dialog da restituire al chiamante, ovvero audio.java. Nell'alterdialog si implementa anche ciò che deve accadere quando si clicca sul pulsante si, e sul no.Io devo cambiare schermata, ovvero devo passare da un activity ad un'altra.
Ora sorge il problema. Alertdialogsystem non estende Activity. e io da subito ho pensato che dopo l'onclick chiamo un metodo implementato in audio.
senza nessun risultato.
Ovviamente nel metodo passa_descrizione_applicazione() c'è la creazione dell'intent e la chiamata di startActivity.
Ho fatto altre mille prove da stamattina. Come ad esempio, cercare di implementare l'intero Onclick in audio. ovvero:
Ma ovviamente poi AlertDialogsystem dà errore in
vuole per forza qui l'onclick.
dai dai datemi suggerimenti, che finalmente stò imparando! comunque secondo me è il context che forse dà problemi. -
05-01-11, 12:55Orochi
Potresti provare a passare il listener da fuori al metodo che costruisce il dialog, in modo da crearlo fuori e assegnarlo dentro col builder...
-
05-01-11, 15:08allmystery
misà che non ho capito molto bene.
e in audio che esteende activity
ps:io clicco su java al posto di CODE ma nn fà il testo rosso.... cmq .... -
05-01-11, 15:43CymonVuoi troppo... a me interessava la colorazione della sintassi ;)Quote:
Originariamente inviato da allmystery
Comunque credo che tu continui a fare con il framework cose un po' perverse :p
Se audio è una activity non credo sia molto bello che tu cerchi di istanziarla con una new e sperare di poter usare un suo metodo per lanciare un intent. E' tutto fuori framework.
Devi capire che la modularizzazione è una cosa buona e giusta, ma devi far fare agli elementi android le parti più strettamente legate al ciclo di vita android.
E qui interviene il fatto che hai dovuto passare il context alla tua classe. Secondo me dovresti:
- rendere il tuo alertdialogsystem affinchè implementi DialogInterface.OnClickListener
- Passare la classe medesima al listener
builder.setPositiveButton("ok", this);
poi implementare in alertdialogsystem il metodo onClick. E quando devi lanciare l'intent invocare un metodo del cnt che a questo punto devi salvarti da qualche parte mentre lo passi al metodo.
Sai una cosa? Quando la modularizzazione porta a soluzioni così contorte di solito io comincio a sospettare che sia un modo errato di modularizzare... o_O -
05-01-11, 16:54allmystery
niente non riesco!
in effetti tanto vale che metto lo switch direttamente nell'activity....
ok! grazie di tutto ragazzi! ;) -
26-07-13, 22:130raziel0Scusa e allora perche io ottenevo risultati migliori dopo aver aggiornato ?Quote:
Originariamente inviato da allmystery
La rom non centra nulla, o hai fatto un'installazione non pulita o la batteria e andata, qui non siamo in assistenza, se non ti fidi dei nostri consigli portalo in assistenza
