search

itproto / ilsrep

Automatically exported from code.google.com/p/ilsrep
0 stars 0 forks source link

sharpTeam current issues r93 #6

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
1. ConnectionThread it's supposed to be named PollHandler imho, handler is
in client-server terminology better name

2. Model* folder it's not good place for ConnectionThread. Model is
abstraction for "buiz. data" classes or domain objects, responsible for
internal state of application, something that we should save often to
database. For poll application I suppose - Poll, Choice, Pollsession. It's
real abstraction for users, ConnectionThread it's smth. internal. Sorry
maybe it's bad explanation, I'll discuss this topic on meeting

3. I would ask you to use sockets. As far I remember I've discussed that
(from my experience TcpListener it's simple dummy wrapper). Ok, it's my
internal requirement

===
Model is adopted by me from MVC pattern:
http://en.wikipedia.org/wiki/Model-view-controller

Original issue reported on code.google.com by ukra...@gmail.com on 26 Jun 2008 at 7:48

GoogleCodeExporter commented 8 years ago 
Fixed tasks 1 and 2 at the revision 95

Original comment by kostje...@gmail.com on 27 Jun 2008 at 10:34

GoogleCodeExporter commented 8 years ago 
Task 3 fixed:
TcpListener removed, instead of him used sockets and TcpClient

Original comment by kostje...@gmail.com on 29 Jun 2008 at 4:00