What kind of manager are you?

Few weeks ago I have become a true manager in the sense I have 3 people I have to take care of. My responsibility is managing all aspects of software ‘construction’ (meaning coding). For those of you who don’t know me, I spent all my career (something like 6 years) trying to avoid any management roles. And now I had to step into this role of manager. First thing I’ve done was asking myself what kind of manager I am going to be.

The answer is complex because it is based on several factors. One of them is my personal preference. The other is my capabilities. And so on and so forth. Somewhere in the middle lies what’s achievable and feasible.

I believe it is very important to think about it for a while. We are not created equal and there is certainly not only one good management style. If you want to blindly mimic someone else’s style (no matter how good it is) you are going to fail because you are different. You have different strong (and weak) points. Something that works for your colleague may not work for you and vice versa.

Take a look at me as an example. As I said, I have quite strong software engineering background. Technical excellence gives me professional self confidence and courage to say ‘no’ whenever I feel it’s necessary. Because of my activity in the community (both on-line and off-line) I have quite a few friends in the field. I also enjoy teaching. Last but not least I think that if you want to do something you better do it well or don’t touch it at all.

All of these define the style of management I am going to use. I am more reluctant to cut corners and thus compromise quality so I am not a good candidate to manage a short project with critical deadline. On the other hand I may be the right guy to run a long term project with quickly changing requirements (good code quality makes changing easy). I will object whenever quality of our product is in danger and you better have good arguments when you try to hurry my team. I will say ‘no’ and won’t be afraid of consequences because you’re not the only software house in town, right?

Because I am a bit shy and from time to time I even think about myself as a sociopath, I am not going to entertain my team. Probably working on my team won’t be the best social experience you can get and my people won’t be super-integrated. On the other hand, they can get technical guidance of much higher quality than from an average manager. They may be sure that if I have a cool piece of work I won’t do it myself but instead I will give it to the guy who can benefit most from doing it.

That’s the kind of manager I want to be. Long project, small team, focus on quality and on growing people. There are only a few things I can offer to my team

  • no hurrying or overtime
  • technical advice (code review, pair programming)
  • tasks that correspond to your interests
  • no obstacles (I will take care of shitty CI server and a annoying business analyst)

but I believe it is enough for them to grow and become independent very quickly and for the project to maintain high quality level.

And what kind of manager are you? Or what kind of manager you would like to have?

VN:F [1.9.13_1145]
Rating: 4.0/5 (1 vote cast)

Classic DDD example, renewed

I’ve spent some time recently renewing my old DDDSample.NET project. I understand that it is the projection of my subjective and uninformed view of Domain-Driven Design, but still I see a value in having some implementation compared to having nothing — it provokes discussion. That’s why I will continue to invest my time in it. If you happen to read this and want to take a look at the code, please remember that is is a sample (as the name suggests), not a guidance. The difference is following: a sample shows how I am doing things while a guidance would show how you are supposed to do things.

Journey through the stack trace

Classic version of DDDSample is a web application with simple UI created with ASP.NET MVC. This is where all starts. I would like to take you on a journey through the stack trace to explain various architectural decisions. Let’s register a new handling event.

NHibernateAmbientSessionManager

The very first class that gets a chance to process the request is NHibernateAmbientSessionManager. It is responsible for creating an ambient NHibernate session that will be bound to the request. After processing the request the session is released. It can be accessed anywhere in the code statically through CurrentSessionContext class. This is a built-in feature of NHibernate.

public void CreateAndBind()
{
    CurrentSessionContext.Bind(_sessionFactory.OpenSession());
}
public void UnbindAndDispose()
{
    var session = CurrentSessionContext.Unbind(_sessionFactory);
    if (session != null)
    {
        session.Dispose();
    }
}

HandlingController

Next is the HandlingController and it’s RegisterHandlingEvent method/action

[AcceptVerbs(HttpVerbs.Post)]
public ActionResult RegisterHandlingEvent(string trackingId, DateTime? completionTime, string location, HandlingEventType type)
{
    if (!completionTime.HasValue)
    {
        ViewData.ModelState.AddModelError("completionTime", @"Event completion date is required and must be a valid date.");
    }
    if (!ViewData.ModelState.IsValid)
    {
        AddHandlingLocations();
        AddHandlingEventTypes();
        return View();
    }
    _handlingEventFacade.RegisterHandlingEvent(completionTime.Value, trackingId, location, type);
    return RedirectToAction("Track", "Tracking", new { trackingId });
}

There’s nothing fancy here, but is shows in a nice way that in this design, the responsibility of a controller is data validation and parsing (mostly done by MVC internals). After ensuring that data conforms to the required format, it is passed to another class which is…

HandlingFacade

This class hides the design of internals of the application from the UI. As far as controllers are concerned, business logic is implemented in façade’s methods. The truth is slightly different, however.

public void RegisterHandlingEvent(DateTime completionTime, string trackingId, string location, HandlingEventType type)
{
    var command = new RegisterHandlingEventCommand
                        {
                            CompletionTime = completionTime,
                            TrackingId = trackingId,
                            OccuranceLocation = location,
                            Type = type
                        };
    _pipelineFactory.Process(command);
}

All the façade is doing is packing the arguments into a command object and sending this command to the pipeline where it is processed. Why create a separate class instead of putting this method on the controller? I like my controllers to be concerned only by the UI and have only one reason to change.

NHibernateTransactionCommandFilter

Before the command is actually executed, a number of filters are invoked. I use a very simple command pattern implementation I’ve written a few weeks ago for the sole purpose of using in DDDSample. It allows me to define these filters in a nice (again, subjective view) way

public void OnHandling(object command)
{
    _ambientSession = CurrentSessionContext.Unbind(_sessionFactory);
    _session = _sessionFactory.OpenSession();
    _transaction = _session.BeginTransaction();
    CurrentSessionContext.Bind(_session);
}
public void OnHandled(object command, object result)
{
    CurrentSessionContext.Unbind(_sessionFactory);
    _transaction.Commit();
    _session.Close();
    CurrentSessionContext.Bind(_ambientSession);
}

Before handling a command an aforementioned ambient NHibernate session is unbound from the current context. We are not going to use it while processing the command because we need a clear workplace. We open a new session and a a new transaction. Whatever happens in the command handler, stays in the command handler. After a command succeeds we unbind the session it used and commit the transaction. Then we restore the ambient session so that UI can use it.

Why don’t we just use a transaction on this ambient session? There is a couple of reasons. First, there can be a transaction started before processing the command and we don’t want to take part in it. Second, we want to be absolutely sure we don’t have any entities in the session-level cache. Why? Because we might use some hints to fetch them more efficiently in context of command processing (like for example force eager load on some relations). And last but not least, if command fails we don’t want to break the session UI was using because it may need it for whatever reason after the command processing attempt failed.

The corollary of this is, we don’t want to put any persistent (tracked by NHibernate) into the command. If we allowed this, we could attach it by mistake to this other session and very strange things could happen. The side effect is we can safely serialize commands which allows us, if the need comes, to put our domain model on a separate tier then its client (the web application).

RegisterHandlingEventCommandHandler

When we have the new session and transaction in place it’s time to do the actual work. Here’s how a typical command handler look like

public object Handle(RegisterHandlingEventCommand command)
{
    var trackingId = new TrackingId(command.TrackingId);
    var cargo = _cargoRepository.Find(trackingId);
    var occuranceLocationUnLocode = new UnLocode(command.OccuranceLocation);
    var occuranceLocation = _locationRepository.Find(occuranceLocationUnLocode);
    var evnt = new HandlingEvent(command.Type, occuranceLocation, DateTime.Now, command.CompletionTime, cargo);
    _handlingEventRepository.Store(evnt);
    return null;
}

The idea here is to have only one method or constructor invocation on an Aggregate Root in one command. As you can see, command handler’s responsibility is to convert the raw data from the command to the proper Value Objects. The validation of data is performed when constructing VOs. In this case we create a new AR so we first instantiate a new object and then call repository to store it.

HandlingEventRepository

What happens in the repository?

public void Store(HandlingEvent handlingEvent)
{
    Session.Save(handlingEvent);
    _eventPublisher.Raise(handlingEvent);
}

Two things. First, we save the newly created AR to the command’s session. Then we use an Event Aggregator to publish this AR as an event. I based this approach on Eric Evans’ What I’ve learned about DDD since the book presentation where he explains that in many domains you encounter Aggregate Roots that are immutable. He calls them Events.

CargoWasHandlerEventHandler

The Event Aggregator publishes the event to all interested parties. In our case there is only one

public void Handle(HandlingEvent handlingEvent)
{
    handlingEvent.Cargo.DeriveDeliveryProgress(handlingEvent);
}

It calls a method on a Cargo Aggregate Root to notify it about the event. One thing worth notice here is we are doing it synchronously in the sample but our contract does not guarantee synchronicity. We write our event handlers in such a way that they can be executed asynchronously to the transaction that published the event. It is good to have such possibilities open in case we need to scale.

Cargo

This is how we reach the Cargo AR

public virtual void DeriveDeliveryProgress(HandlingEvent lastHandlingEvent)
{
    Delivery = Delivery.DerivedFrom(RouteSpecification, Itinerary, lastHandlingEvent);
}

The method looks very simple and it is not a coincidence. In classic DDD we have limited testing options since we can’t set our Entities to proper state easily (as easy in if we use Event Sourcing). That’s why we don’t want to test Entities. We want to test Value Objects because they are immutable and truly persistence ignorant. All we do in an Entity is pass the arguments to a VO and replace the current value with a new one. The probability that we make a bug in this one-liner is small, at least I hope so.

Delivery

Finally we arrived at our destination, the Value Object. Here I won’t include any code as it is quite complex and lengthy. I’ve already said probably the most important thing about VOs in this approach: they are immutable. This property allows us to test them easily. The compromise we are willing to accept in regards to Value Objects is, we expose their state via properties. Because we don’t have Command and Query Responsibility Separation, we use same objects (but remember, fetched via different session) to read and to write.

Summary

I hope you enjoyed this journey through the layers of the onion. Remember that this is just a sample. Most of its features were tested in the field but some combinations of them were not. I am eager to hear what you think about this approach and also to hear about your approach to DDD when size and/or complexity does not justify CQRS.

VN:F [1.9.13_1145]
Rating: 5.0/5 (2 votes cast)

68 spotkanie KGD.NET

Już za dwa dni, w środę 26 października odbędzie się 68. spotkanie KGD.NET. Dlaczego o tym tu pisze? Ponieważ spotkanie będzie, dzięki uprzejmości portalu VirtualStudy.pl, strumieniowane na żywo oraz nagrywane. Informację o spotkaniu na witrynie VS można znaleźć tutaj. Mieszkańców Krakowa i okolic oczywiście zachęcam na przyjścia osobiście (godzina 18.30, siedziba ABB przy Starowiślnej 13). Zarejestrować się można na stronie spotkania. Osoby, które się zarejestrują i przyjdą będą uczestniczyć w losowaniu nagród, m.in. voucherów Pluralsight, podręcznika .NET i innych.

Zapraszam serdecznie

PS. Po spotkaniu idziemy na piwo:)

VN:F [1.9.13_1145]
Rating: 0.0/5 (0 votes cast)

ABB Dev Day

W ubiegły piątek miałem przyjemność wystąpić na konferencji ABB Dev Day 2011 w Krakowie. Muszę przyznać, że była to jedna z najlepszych konferencji, na jakich byłem. Organizatorzy — Michał Śliwoń (@mihcall) i Rafał Legiędź (@rafek) – wykonali kawał świetnej roboty. Duże brawa dla nich.

Na tej konferencji pojawiłem się z prezentacją o (jakże by inaczej) CQRS. Więszość uczestników pewnie pamięta, ale na wszelki wypadek przypominam: rzuciłem wtedy wyzwanie zaimplementowania prostej aplikacji w ramach eksperymentalnej architektury CQREST (połączenie CQRS i REST, jak się łatwo domyśleć):

Nagrodą jest licencja na jeden z produktów JetBrains oraz oczywiście wieczna sława:) Jeśli ktoś się do tej pory wahał, czy warto, niech przestanie się wahać i weźmie się za kodowanie.

VN:F [1.9.13_1145]
Rating: 0.0/5 (0 votes cast)

Renewing DDDSample.Net, iteration 0

As I promised in one of the previous posts, I am now in process of renewing the DDDSample.Net source code. It’s been quite a while since I published the first version of it. Many technologies have changed and my take on Domain-Driven Design and software architecture also has changed slightly.

First of all, the repo was moved to github. Hurray! In the process of transforming TFS repository into a git one I lost the branch structure. Originally all versions were branched one from another which made updating all samples a little bit easier since (ideally) I had to change only one and then merge this change. With git I am thinking of a slightly different structure. I am pretty sure I can freeze some portions of the solution (like routing engine and most of the UI) and move them to separate repos. Then I can import them as submodules to each of the samples.

At first I just wanted to add new features to the existing samples but when I opened the vanilla version (a port of Java DDDSample) I found out that it is so outdated that it urgently needs to be renewed. The changes I made so far include:

  • Upgrading solution to VS2010 format
  • Removing the web setup project (I guess nobody used it, ever)
  • Switching to NuGet for resolving most of the packages (using don’t commit packages workflow described here by David Ebbo)
  • Upgrading some dependencies (like NHibernate) to the newest versions
  • Switching from using façade-like classes in application layer to commands and command handlers implemented using brand new tiny LeanCommandUnframework (a separate micro project hosted here on github)
  • Switching from Unity 1.2 to Autofac
  • Removing dependency on NServiceBus
  • Cleaning up the global.asax file

I am now quite happy with the solution. Everything seems to have its place in the architecture. There are some things yet to be clean up such as changing the way command handlers are created but these should be minor fixes. If you have some spare time, please clone the repo and check if you like what I did. And please note that only Vanilla version was updated.

    VN:F [1.9.13_1145]
    Rating: 0.0/5 (0 votes cast)