Mercurial > altnet-hispano
annotate Agendas/trunk/src/Agendas.Domain/Agenda.cs @ 37:90f0cab1febc
Sugerencia de resharper sobre optimización
author | nelo@MTEySS.neluz.int |
---|---|
date | Thu, 17 Mar 2011 17:04:03 -0300 |
parents | 65bbf1ab2b24 |
children | 3c5657d99727 |
rev | line source |
---|---|
1 | 1 using System; |
2 | 2 using System.Collections.Generic; |
4 | 3 using AltNetHispano.Agendas.Domain.Exceptions; |
15 | 4 using AltNetHispano.Agendas.Domain.Repositories; |
0
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
5 |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
6 namespace AltNetHispano.Agendas.Domain |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
7 { |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
8 public class Agenda |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
9 { |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
10 private readonly IPublicador _publicador; |
1 | 11 private readonly IRecordador _recordador; |
8
cae27d7eb697
Se agregó ISeguridad para validar autenticaciones de usuarios.
juanjose.montesdeocaarbos
parents:
7
diff
changeset
|
12 private readonly ISeguridad _seguridad; |
15 | 13 private readonly IEventoRepository _eventosRepository; |
24 | 14 private readonly IPonenteRepository _ponenteRepository; |
2 | 15 |
24 | 16 public Agenda(IPublicador publicador, IRecordador recordador, ISeguridad seguridad, IEventoRepository eventosRepository, IPonenteRepository ponenteRepository) |
0
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
17 { |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
18 _publicador = publicador; |
24 | 19 _ponenteRepository = ponenteRepository; |
15 | 20 _eventosRepository = eventosRepository; |
1 | 21 _recordador = recordador; |
8
cae27d7eb697
Se agregó ISeguridad para validar autenticaciones de usuarios.
juanjose.montesdeocaarbos
parents:
7
diff
changeset
|
22 _seguridad = seguridad; |
0
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
23 } |
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
24 |
24 | 25 public void Publicar(string titulo, string ponenteNombre, DateTime? fecha) |
26 { | |
27 var ponente = GetPonente(ponenteNombre); | |
3 | 28 |
29
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
29 var evento = _eventosRepository.GetPropuestaByTitulo(titulo) ?? new Evento {Titulo = titulo}; |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
30 evento.Ponente = ponente; |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
31 evento.Fecha = fecha; |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
32 |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
33 if (!evento.Fecha.HasValue) |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
34 throw new ValidationException(); |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
35 if (NoEstaAutenticado(_seguridad)) |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
36 throw new UsuarioNoAutenticadoException(); |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
37 if (string.IsNullOrWhiteSpace(evento.Ponente.Nombre)) |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
38 throw new ValidationException(); |
1 | 39 |
29
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
40 if (_publicador != null) |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
41 _publicador.Publicar(evento); |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
42 _eventosRepository.Save(evento); |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
43 _ponenteRepository.Save(ponente); |
016b9b9b8d3a
Cuando se publica una van con el mismo titulo que una propuesta, se utiliza esta
nelo@MTEySS.neluz.int
parents:
26
diff
changeset
|
44 } |
4 | 45 |
21
43360bf09b1a
unificando criterio de parámetros en Agenda
nelo@MTEySS.neluz.int
parents:
18
diff
changeset
|
46 public void Recordar(Guid eventoId) |
1 | 47 { |
21
43360bf09b1a
unificando criterio de parámetros en Agenda
nelo@MTEySS.neluz.int
parents:
18
diff
changeset
|
48 var evento = _eventosRepository.Get(eventoId); |
1 | 49 if (_recordador != null) |
50 _recordador.Recordar(evento); | |
0
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
51 } |
4 | 52 |
26 | 53 public void Proponer(string titulo, string ponenteNombre) |
4 | 54 { |
26 | 55 var ponente = GetPonente(ponenteNombre); |
56 var evento = new Evento {Titulo = titulo, Ponente = ponente}; | |
21
43360bf09b1a
unificando criterio de parámetros en Agenda
nelo@MTEySS.neluz.int
parents:
18
diff
changeset
|
57 |
12 | 58 if (string.IsNullOrWhiteSpace(evento.Titulo)) |
4 | 59 throw new ValidationException(); |
11
9d6b28a696d1
corrección en método que verifica la autenticación del usuario
nelo@MTEySS.neluz.int
parents:
9
diff
changeset
|
60 if (NoEstaAutenticado(_seguridad)) |
18
8ed4a806ebe0
corrección de excepeción lanzada si no hay usuario autenticado al proponer un evento
nelo@MTEySS.neluz.int
parents:
16
diff
changeset
|
61 throw new UsuarioNoAutenticadoException(); |
15 | 62 _eventosRepository.Save(evento); |
26 | 63 _ponenteRepository.Save(ponente); |
4 | 64 } |
65 | |
6
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
66 public IList<Evento> GetEventosPropuestos() |
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
67 { |
15 | 68 return _eventosRepository.GetEventosSinFecha() ?? new List<Evento>(); |
6
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
69 } |
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
70 |
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
71 public IList<Evento> GetEventosPublicados() |
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
72 { |
15 | 73 return _eventosRepository.GetEventosConFecha() ?? new List<Evento>(); |
6
2912c1dd0e6b
Se verificó mediante un test que las propuestas y las publicaciones se estaban mezclando. Se corrigió el bug.
juanjose.montesdeocaarbos
parents:
4
diff
changeset
|
74 } |
9
c90492faf268
Se implementa seguridad a través del constructor y no como parámetro
juanjose.montesdeocaarbos
parents:
8
diff
changeset
|
75 |
15 | 76 private static bool NoEstaAutenticado(ISeguridad seguridad) |
77 { | |
37
90f0cab1febc
Sugerencia de resharper sobre optimización
nelo@MTEySS.neluz.int
parents:
36
diff
changeset
|
78 return seguridad == null || seguridad.GetPrincipal() == null |
11
9d6b28a696d1
corrección en método que verifica la autenticación del usuario
nelo@MTEySS.neluz.int
parents:
9
diff
changeset
|
79 || string.IsNullOrWhiteSpace(seguridad.GetPrincipal().Identity.Name); |
9
c90492faf268
Se implementa seguridad a través del constructor y no como parámetro
juanjose.montesdeocaarbos
parents:
8
diff
changeset
|
80 } |
14 | 81 |
24 | 82 public void ModificarEvento(Guid id, string titulo, string ponenteNombre, DateTime? fecha) |
14 | 83 { |
15 | 84 var evento = _eventosRepository.Get(id); |
14 | 85 if (evento == null) |
86 throw new EventoNotFoundException(); | |
24 | 87 |
88 var ponente = GetPonente(ponenteNombre); | |
89 | |
14 | 90 evento.Titulo = titulo; |
91 evento.Ponente = ponente; | |
92 evento.Fecha = fecha; | |
36
65bbf1ab2b24
cuando se modifica un evento, también debe ser publicada dicha modificación
nelo@MTEySS.neluz.int
parents:
29
diff
changeset
|
93 |
65bbf1ab2b24
cuando se modifica un evento, también debe ser publicada dicha modificación
nelo@MTEySS.neluz.int
parents:
29
diff
changeset
|
94 if (_publicador != null) |
65bbf1ab2b24
cuando se modifica un evento, también debe ser publicada dicha modificación
nelo@MTEySS.neluz.int
parents:
29
diff
changeset
|
95 _publicador.Publicar(evento); |
14 | 96 } |
11
9d6b28a696d1
corrección en método que verifica la autenticación del usuario
nelo@MTEySS.neluz.int
parents:
9
diff
changeset
|
97 |
26 | 98 public Evento GetEvento(Guid id) |
14 | 99 { |
15 | 100 return _eventosRepository.Get(id); |
14 | 101 } |
24 | 102 |
103 public void RegistrarPonente(string nombre, string mail, string twitter, string blog) | |
104 { | |
105 var ponente = new Ponente {Nombre = nombre, Mail = mail, Twitter = twitter, Blog = blog}; | |
106 _ponenteRepository.Save(ponente); | |
107 } | |
108 | |
109 private Ponente GetPonente(string nombre) | |
110 { | |
111 return _ponenteRepository.GetByNombre(nombre) ?? new Ponente { Nombre = nombre }; | |
112 } | |
113 | |
26 | 114 public void ModificarPropuesta(Guid id, string titulo, string ponenteNombre) |
115 { | |
116 ModificarEvento(id, titulo, ponenteNombre, null); | |
117 } | |
14 | 118 } |
119 | |
120 public class EventoNotFoundException : Exception | |
121 { | |
122 } | |
123 | |
124 public class UsuarioNoAutenticadoException : Exception | |
11
9d6b28a696d1
corrección en método que verifica la autenticación del usuario
nelo@MTEySS.neluz.int
parents:
9
diff
changeset
|
125 { |
9d6b28a696d1
corrección en método que verifica la autenticación del usuario
nelo@MTEySS.neluz.int
parents:
9
diff
changeset
|
126 } |
0
3f7dbb633b71
Test sobre la publicación de Eventos al registrarlos en la Agenda
nelopauselli
parents:
diff
changeset
|
127 } |