((Pile)state.piles.get(Integer.parseInt(carddata.getNamedItem("pile")

.getNodeValue()))).addCard(new Card(Integer.parseInt(carddata.getNamedItem("suit")

.getNodeValue()), Integer.parseInt(carddata.getNamedItem("value").getNodeValue())));

...is the most horrendous line of code that I've ever written. I hope it works, because I had forgotten what it meant by the time I reached the end of the line.

I'd also like to mention to whoever understands it that I've never liked the syntax for casting in Java. It looks untidy.