Where do you draw the line when moving functions which operate on data into the class which contains that data? For example, imagine you have a simple class which stores a description of the weather, with variables for temperature, humidity, wind speed and direction, and the time at which the measurement was taken. Now imagine you have an object of this class and you want to transmit it to someone else – another process, another machine, whatever. Do you put the code to transmit the object into the object itself – for example, by adding a Send(destination-type) method to the simple data class? Or do you keep this kind of feature in separate classes that can send and receive anything over the medium – whether it’s networking, file i/o, interprocess comms, or anything similar?
My gut instinct is to keep my data classes simple and wrap them up when I want to transmit them – in classes which serialise them and present the sender and receiver classes with a simple interface they understand. The alternative seems to be to put everything including the kitchen sink into the simple data classes – every function which might ever operate on that data, however indirectly. In short, network error handling code doesn’t seem to me to belong in a simple data class.
This seems obvious to me, but I keep seeing developers put Send() methods on their classes. They even tell message classes to Send() themselves, which seems highly counterintuitive to me; if I write a letter on a piece of paper, I don’t tell the paper to send itself. I wrap the letter in an envelope and hand it to the postman, because he has a van and a map. What do people think?
There’s logic to do with the payload item itself: what’s the wind speed now?
There’s logic to do with interpreting that data: can we dock this ship now?
There’s logic to do with deciding to send the payload somewhere: oh here’s a new weather value, someobody over there cares.
Then the actual network stuff.
The payload probably needs to be able to serialize and deserialise itself. I don’t see that any of the rest is the payload’s concern. There must be a better place for Send(). Not least because you might choose to send several payload objects at once, and they can’t all send each other.