Comments on: Hey, I Think I’ve Invented a New Pattern http://georgemauer.net/blog/hey-i-think-ive-invented-a-new-pattern/ George Mauer is on the net Sun, 16 May 2010 17:22:49 -0400 http://wordpress.org/?v=2.8.2 hourly 1 By: togakangaroo http://georgemauer.net/blog/hey-i-think-ive-invented-a-new-pattern/comment-page-1/#comment-77 togakangaroo Tue, 19 Jan 2010 20:27:52 +0000 http://georgemauer.net/blog/?p=119#comment-77 Put this out there and then realize I made a mistake. While the pattern is still good, the example is off. It seems like the CrystalReports ReportDocument object occasionally needs to be exposed of explicitly (god I hate CrystalReports). So the modified example would be: using(var doc = _dataProvider .GetOrderDataFor(customerId) .ComposeIntoReportUsing(_reportGenerator) ) { return doc.ToStreamUsing(_pdfReportExporter); } Not nearly as slick but when you hit up against framework limitations what can you do. Put this out there and then realize I made a mistake. While the pattern is still good, the example is off. It seems like the CrystalReports ReportDocument object occasionally needs to be exposed of explicitly (god I hate CrystalReports). So the modified example would be:
using(var doc =
_dataProvider
.GetOrderDataFor(customerId)
.ComposeIntoReportUsing(_reportGenerator) ) {
return doc.ToStreamUsing(_pdfReportExporter);
}

Not nearly as slick but when you hit up against framework limitations what can you do.

]]> By: togakangaroo http://georgemauer.net/blog/hey-i-think-ive-invented-a-new-pattern/comment-page-1/#comment-76 togakangaroo Tue, 19 Jan 2010 14:37:17 +0000 http://georgemauer.net/blog/?p=119#comment-76 @Laura And I do appreciate it. @Fritz Now you know I love my decorators but not really. Decorators might looks something like this at the point of configuration, this is at the point of usage - and that makes a world of difference. Another post perhaps, but I find that adapters and factories can play a part: return _dataProvider .GetOrderDataFor(customerId) .ComposeIntoReportUsing(_reportGenerator) .ToStreamUsing(_pdfReportExporter) .AttachToMailMessage(new MailMessage("fred.flintstone@brock.com", "BRubble@brock.com")) .SendUsing(_smtpAdapter); // Where _smtpAdapter simply wraps the SmtpMail.Send call return _dataProvider .GetOrderDataFor(customerId) .ComposeIntoReportUsing(_reportGeneratorFactory.Get(reportParameter1, reportParameter2)) .ToStreamUsing(_pdfReportExporter); // When you also need to inject parameters into the report in addition to the dataset @Laura
And I do appreciate it.

@Fritz
Now you know I love my decorators but not really. Decorators might looks something like this at the point of configuration, this is at the point of usage – and that makes a world of difference.

Another post perhaps, but I find that adapters and factories can play a part:
return _dataProvider
.GetOrderDataFor(customerId)
.ComposeIntoReportUsing(_reportGenerator)
.ToStreamUsing(_pdfReportExporter)
.AttachToMailMessage(new MailMessage(”fred.flintstone@brock.com”, “BRubble@brock.com”))
.SendUsing(_smtpAdapter);
// Where _smtpAdapter simply wraps the SmtpMail.Send call

return _dataProvider
.GetOrderDataFor(customerId)
.ComposeIntoReportUsing(_reportGeneratorFactory.Get(reportParameter1, reportParameter2))
.ToStreamUsing(_pdfReportExporter);
// When you also need to inject parameters into the report in addition to the dataset

]]> By: Fritz http://georgemauer.net/blog/hey-i-think-ive-invented-a-new-pattern/comment-page-1/#comment-75 Fritz Mon, 18 Jan 2010 23:29:50 +0000 http://georgemauer.net/blog/?p=119#comment-75 It has a vaguely decorator-ish flavor to it. It has a vaguely decorator-ish flavor to it.

]]> By: Laura Veazey http://georgemauer.net/blog/hey-i-think-ive-invented-a-new-pattern/comment-page-1/#comment-74 Laura Veazey Mon, 18 Jan 2010 21:36:46 +0000 http://georgemauer.net/blog/?p=119#comment-74 Wow. I don't understand any of this. But at least I commented. Wow. I don’t understand any of this. But at least I commented.

]]>