llevo 3 años realizando una app de un calendario para mi trabajo a base de turnos y equipos que funciona perfectamente pero.... Cada año voy añadiendo cosas nuevas y el código se hace cada vez más complicado y más repetitivo en algunos momentos. Sé que seguro que este código se puede acortar o asociar para que no sea tan repetitivo pero no encuentro la forma. Os adjunto un ejemplo de lo que digo para ver como me podéis echar una mano a resolver este problema.
} else if (id == R.id.item_navigation_drawer_grupo) {
EquipoMeses = EquipoLayout;
EquipoJuntoMeses = EquipoJuntoLayout;
mesMeses = MesLayout;
Intent Elegir_Grupo_Meses = new Intent(this, Elegir_Grupo_Meses.class);
Elegir_Grupo_Meses.putExtra("mes", mesMeses);
Elegir_Grupo_Meses.putExtra("mesletra", MesLetra);
Elegir_Grupo_Meses.putExtra("dia", diaMeses);
Elegir_Grupo_Meses.putExtra("dianulo", dianuloMeses);
Elegir_Grupo_Meses.putExtra("diaSemana", diaSemanaMeses);
Elegir_Grupo_Meses.putExtra("Equipo", EquipoMeses);
Elegir_Grupo_Meses.putExtra("EquipoJunto", EquipoJuntoMeses);
Elegir_Grupo_Meses.putExtra("ServicioActivado", ServicioActivadoMeses);
Elegir_Grupo_Meses.putExtra("PrimeraNotificacion", PrimeraNotificacion);
Elegir_Grupo_Meses.putExtra("AlarmaConectadaOnOff", AlarmaConectadaOnOffLayout);
Elegir_Grupo_Meses.putExtra("notificacionlanzada", notificacionlanzada);
startActivity(Elegir_Grupo_Meses);
finish();
} else if (id == R.id.item_navigation_drawer_notificaciones) {
EquipoMeses = EquipoLayout;
EquipoJuntoMeses = EquipoJuntoLayout;
mesMeses = MesLayout;
Intent QuitarNotificacion = new Intent(this, QuitarNotificacion.class);
QuitarNotificacion.putExtra("mes", mesMeses);
QuitarNotificacion.putExtra("mesletra", MesLetra);
QuitarNotificacion.putExtra("dia", diaMeses);
QuitarNotificacion.putExtra("dianulo", dianuloMeses);
QuitarNotificacion.putExtra("diaSemana", diaSemanaMeses);
QuitarNotificacion.putExtra("Equipo", EquipoMeses);
QuitarNotificacion.putExtra("EquipoJunto", EquipoJuntoMeses);
QuitarNotificacion.putExtra("ServicioActivado", ServicioActivadoMeses);
QuitarNotificacion.putExtra("PrimeraNotificacion", PrimeraNotificacion);
QuitarNotificacion.putExtra("AlarmaConectadaOnOff", AlarmaConectadaOnOffLayout);
QuitarNotificacion.putExtra("notificacionlanzada", notificacionlanzada);
startActivity(QuitarNotificacion);
finish();
} else if (id == R.id.item_navigation_drawer_bases) {
EquipoMeses = EquipoLayout;
EquipoJuntoMeses = EquipoJuntoLayout;
mesMeses = MesLayout;
Intent BackupRestore = new Intent(this, BackupRestore.class);
BackupRestore.putExtra("mes", mesMeses);
BackupRestore.putExtra("mesletra", MesLetra);
BackupRestore.putExtra("dia", diaMeses);
BackupRestore.putExtra("dianulo", dianuloMeses);
BackupRestore.putExtra("diaSemana", diaSemanaMeses);
BackupRestore.putExtra("Equipo", EquipoMeses);
BackupRestore.putExtra("EquipoJunto", EquipoJuntoMeses);
BackupRestore.putExtra("ServicioActivado", ServicioActivadoMeses);
BackupRestore.putExtra("PrimeraNotificacion", PrimeraNotificacion);
BackupRestore.putExtra("AlarmaConectadaOnOff", AlarmaConectadaOnOffLayout);
BackupRestore.putExtra("notificacionlanzada", notificacionlanzada);
startActivity(BackupRestore);
finish();
}
como veis este código se repite en este ejemplo hasta en 3 veces siendo lo mismo, pero esta situación se repite en 6 veces en esta activity y se repite en casi todas las activitys. ¿Habría alguna forma de crear un procedimiento para no tener que repetir siempre el mismo código y poderlo llamar de alguna manera simple?
if (MesCompManana == 6 ||MesCompNoche == 6) {
switch (Grupo) {
case 1:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 2:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 3:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 4:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 5:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 6:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 7:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
case 8:
if (MesCompManana == 6){
int CasillaCompManana1 = CasillaInicio + DiaCompManana - 1;
String imageCompManana1 = "dia_comp_manana";
int drawableIdManana1 = getResources().getIdentifier(imageCompManana1, "drawable", getPackageName());
Casilla[CasillaCompManana1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdManana1));
break;
} else if (MesCompNoche == 6){
int CasillaCompNoche1 = CasillaInicio + DiaCompNoche - 1;
String imageCompNoche1 = "dia_comp_noche";
int drawableIdNoche1 = getResources().getIdentifier(imageCompNoche1, "drawable", getPackageName());
Casilla[CasillaCompNoche1].setBackgroundDrawable(this.getResources().getDrawable(drawableIdNoche1));
break;
}
}
}
Aquí os muestro otro ejemplo de lo que os comento, este ejemplo se repite en 4 ocasiones cambiando solo una serie de variables pero el código es prácticamente exacto. Repetición de la misma función pero no tengo manera de reducir el código.
Ya os he comentado que la app funciona perfectamente, pero no me gusta como está quedando el código de cargado y cada vez me cuesta más entenderlo hasta a mí.
Me gustaría aprender más en este sentido de organización y distribución de código pero no se donde acudir ya que en mi ciudad no existen academias de programación.
Espero que me podáis ayudar y darme algunas ideas.
Aquí os añado otro trozo de código el cual no soy tampoco capaz de reducir por si me podéis dar alguna idea.
@Override
public boolean onLongClick(View v) {
if (MesLetra == MesAnoPasado || MesLetra == MesAnoViene){
ToastPersonalizado miToast = new ToastPersonalizado(this, Toast.LENGTH_SHORT);
miToast.show("En este mes no puedes introducir notas");
} else {
switch (v.getId()){
case R.id.TxtVCasilla1:
TextView txtvcasilla1 = (TextView)findViewById(R.id.TxtVCasilla1);
TextoCasilla = txtvcasilla1.getText();
if (txtvcasilla1.getText() == null) {
AquiNoPuedes();
break;
} else {
PasarDatosNotas();
return true;
}
case R.id.TxtVCasilla2:
TextView txtvcasilla2 = (TextView)findViewById(R.id.TxtVCasilla2);
TextoCasilla = txtvcasilla2.getText();
if (txtvcasilla2.getText() == null) {
AquiNoPuedes();
break;
} else {
PasarDatosNotas();
return true;
}
case R.id.TxtVCasilla3:
TextView txtvcasilla3 = (TextView)findViewById(R.id.TxtVCasilla3);
TextoCasilla = txtvcasilla3.getText();
if (txtvcasilla3.getText() == null) {
AquiNoPuedes();
break;
} else {
PasarDatosNotas();
return true;
}
...
case R.id.TxtVCasilla41:
TextView txtvcasilla41 = (TextView)findViewById(R.id.TxtVCasilla41);
TextoCasilla = txtvcasilla41.getText();
if (txtvcasilla41.getText() == null) {
AquiNoPuedes();
break;
} else {
PasarDatosNotas();
return true;
}
case R.id.TxtVCasilla42:
TextView txtvcasilla42 = (TextView)findViewById(R.id.TxtVCasilla42);
TextoCasilla = txtvcasilla42.getText();
if (txtvcasilla42.getText() == null) {
AquiNoPuedes();
break;
} else {
PasarDatosNotas();
return true;
}
}
}
return false;
}
Muchas gracias por vuestra ayuda por adelantado.
if
que muestras, el cual dices que se repite hasta 6 veces con los mismos datos lo podrías simplificar de la siguiente manera: Creando un método al cual le pasarás dos parámetros: 1) unMap
o unArray
con los datos que pones en elMap
tales comoQuitarNotificacion.putExtra("mes", mesMeses);
y 2) ElIntent
con el que vas a trabajar. De ese modo losif
no serían necesarios. Nota: Esto es sólo una sugerencia, aunque no sé lo que ocurre antes de llegar a losif
...