Click here to Skip to main content

C#

Re: Need Help Understanding Error Handling

PIEBALDconsult6-Dec-10 12:21

6-Dec-10 12:21

This doesn't answer the question, but....

if ( correctNamespace.Length == 0 ) ...

how to convert dataset to excell ?

Gali19786-Dec-10 9:04

6-Dec-10 9:04

hi
i have 2 dataset's in my C# program
dataset1 holds table A and dataset2 holds table B
i need to create excell (xls file - not csv file) with 2 sheets
one for table A and one for table B

how to do it in C# ?

thank's in advance

Re: how to convert dataset to excell ?

Pawel Gielmuda6-Dec-10 22:50

6-Dec-10 22:50

Google is your friend Wink | ;)

Wink | ;)

http://www.davidhayden.com/blog/dave/archive/2006/05/26/2973.aspx[^]

Re: how to convert dataset to excell ?

RaviRanjanKr17-Dec-10 18:02

17-Dec-10 18:02

Hope this DataSet-to-Excel [^] link may work for you.
You can also navigate the given link (Note Article is in VB) for better response
How to transform a DataSet to spreadsheet XML for Excel [^]

Sending attachments with SMTP Client

musefan6-Dec-10 3:39

6-Dec-10 3:39

Hi All,

My requirement is... Generate a file, send that file as attachment using SMTP client and then delete the generated file.

I have generated the file fine.
I have implemented SMTP client usage fine.

first attempt error - file in use when trying to delete after sending email...

message.Attachments.Add(new System.Net.Mail.Attachment("FILEPATH"));
smtp.Send(message);

...OK, so SEND is for some reason still using the file. I have just removed another load of almost posted text as I have figured out the answer. Which is basically...

Collection<Stream> streams = new Collection<Stream>();

foreach(string filename in filenames)
{
   System.IO.FileInfo fileInfo = new System.IO.FileInfo(filename);
   Stream stream = new MemoryStream(File.ReadAllBytes(filename));
   message.Attachments.Add(new System.Net.Mail.Attachment(stream, fileInfo.Name, null));
   streams.Add(stream);
}

smtp.Send(message);

foreach(Stream stream in streams)
   stream.Close();

So I have solved my problem, but have still posted as may be useful for others but also, does anybody know why adding an attachment via a file path keeps the file locked after sending?

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

OriginalGriff6-Dec-10 3:42

6-Dec-10 3:42

WQithout seeing the code you used to create the file, it's diificult, but yes.

When you created the file, did you close and dispose the file before you tried to send it? If not, and it just went out of scope, then the file is in use until the garbage collector disposes of it. That could be next Tuesday...

Real men don't use instructions. They are only the manufacturers opinion on how to put the thing together.

Re: Sending attachments with SMTP Client

musefan6-Dec-10 3:52

6-Dec-10 3:52

The generated file is handled correctly. Hence the clean up code (deleting the generated file) works when I use a stream to add an attachment (as in my second code sample).

The only time the clean up fails is if I use my first code sample - if I comment the sole line of smtp.Send then the clean up does not fail. for some reason the Send function is still using the file. I cannot even manually delete the file after the failed clean up for the same reason. Only when I kill the program (which is a windows service) does the file become free again

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

musefan6-Dec-10 4:15

6-Dec-10 4:15

After a bit more playing the answer to send using a file path is as follows...

using (System.Net.Mail.Attachment attachment = new System.Net.Mail.Attachment(filename))
{
   message.Attachments.Add(attachment);
}
smtp.Send(message);

...I guess I was holding a handle on an System.Net.Mail.Attachment instance in my first code sample (somehow!). Though I would have liked to think this would not have keep a file open anyway!

this however does not work...

using(Stream stream = new MemoryStream(File.ReadAllBytes(filename)))
{
   message.Attachments.Add(stream, "myfile", null);
}
smtp.Send(message);

...because the Send() call is out of scope of the stream. Why this doesn't work but the above does I do not know.

EDIT: Actually I forgot to recompile - neither work (which I would suspect not to)

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client [modified]

PIEBALDconsult6-Dec-10 4:16

6-Dec-10 4:16

Why are you opening the file? Didn't you read my response to ~~your~~ (whoops it wasn't yours, sorry) the earlier post? Confused | :confused:

Confused | :confused:

modified on Monday, December 6, 2010 10:37 AM

Re: Sending attachments with SMTP Client

musefan6-Dec-10 4:20

6-Dec-10 4:20

PIEBALDconsult wrote:
Why are you opening the file? Didn't you read my response to your earlier post?

I am not opening the file. What in my posts gave you that idea?

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 4:24

6-Dec-10 4:24

new MemoryStream(File.ReadAllBytes(filename));

Re: Sending attachments with SMTP Client

musefan6-Dec-10 4:30

6-Dec-10 4:30

File.ReadAllBytes() closes the file after creating a byte[] - this is not the issue

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 4:40

6-Dec-10 4:40

It is to me -- I send large XML and EDI files.

Oh, and PDFs generated by Crystal. ::shudder::

Re: Sending attachments with SMTP Client

musefan6-Dec-10 4:57

6-Dec-10 4:57

So then you mean the issue is loading file in memory? I agree with that, its not ideal but I am working with relatively small files and they should stay that way

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 5:32

6-Dec-10 5:32

You don't want your code crash the email server if you can help it. (Been there, done that. Blush | :O

Blush | :O

)

I prefer a more general solution whenever I can find one. I don't follow YAGNI -- I always stop to think, "How can this come back to bite me on the backside? I can't foresee all possible problems, but what steps can I take now to alleviate the future problems that I can foresee?"

Also, its easier to allow for very large files today than it will be in a few months.

And if the files are very small, why attach them? Why not use them as the message text?

On the other hand... why not avoid the files entirely? Generate the "file" in memory, store it in the database, read it from there, attach it, and then delete the record. Maybe have the delete postponed until after a database backup.

Re: Sending attachments with SMTP Client

musefan6-Dec-10 5:46

6-Dec-10 5:46

PIEBALDconsult wrote:
You don't want your code crash the email server if you can help it.

If file sizes start to get to large for emailing then we will prevent sending by email and force users to use the FTP option.

PIEBALDconsult wrote:
I don't follow YAGNI

I agree with regards to potential problems down the line. But I am not going to waste time implementing functionality that is not needed.

PIEBALDconsult wrote:
And if the files are very small, why attach them? Why not use them as the message text?

They are data exports. XML files, MDB files - not very usable as just email body content

PIEBALDconsult wrote:
On the other hand... why not avoid the files entirely?

Because one of the file types if MDB then a file must be created on disk

PIEBALDconsult wrote:
Generate the "file" in memory, store it in the database, read it from there, attach it, and then delete the record

Why would I want to create a file in memory, store in DB and then read it straight back into memory?

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 5:57

6-Dec-10 5:57

musefan wrote:
Why would I want to create a file in memory, store in DB and then read it straight back into memory?

Separation of responsibilities: one (or several) process(es) creates, another sends. Maybe another archives. Who knows?

Re: Sending attachments with SMTP Client

musefan6-Dec-10 6:08

6-Dec-10 6:08

PIEBALDconsult wrote:
Separation of responsibilities: one (or several) process(es) creates, another sends. Maybe another archives. Who knows?

In some requirements, yes maybe so. But mine is much more just 'Create and Send' - its wasted effort to split the process

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 6:22

6-Dec-10 6:22

I'll check back in a year. Big Grin | :-D

Big Grin | :-D

Re: Sending attachments with SMTP Client

musefan6-Dec-10 6:39

6-Dec-10 6:39

don't forget now... maybe I will come crawling before then Laugh | :laugh:

Laugh | :laugh:

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 7:21

6-Dec-10 7:21

musefan wrote:
maybe I will come crawling before then

I don't think so... I'm sure you'll be able to handle it on your own. Thumbs Up | :thumbsup:

Thumbs Up | :thumbsup:

Re: Sending attachments with SMTP Client

Luc Pattyn6-Dec-10 4:16

6-Dec-10 4:16

I had the same experience; my guess is the attachment files remain locked by the mail system until it is done sending them. Your code is not to blame for any of that. Solutions would be to create:
- an attachment from the file contents;
- an attachment form a stream, as you did;
- a copy of the file (leaving even more garbage on the file system).

Smile | :)

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use <PRE> tags for code snippets, they preserve indentation, improve readability, and make me actually look at the code.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 4:26

6-Dec-10 4:26

Or don't delete the file -- archive it at a later date. You never know when you may need a paper trail to CYA. Or to resend for some reason.

Re: Sending attachments with SMTP Client

musefan6-Dec-10 4:41

6-Dec-10 4:41

Files are created purely for sending to users. There is no need for keeping the file - besides generated files could end up frequent and many so want to keep the file system tidy

Life goes very fast. Tomorrow, today is already yesterday.

Re: Sending attachments with SMTP Client

PIEBALDconsult6-Dec-10 4:58

6-Dec-10 4:58

musefan wrote:
Files are created purely for sending to users

Aren't they all?

musefan wrote:
There is no need for keeping the file

"There will be." -- Murphy

musefan wrote:
keep the file system tidy

Create them in a directory specifically for them.

General News Suggestion Question Bug Answer Joke Praise Rant Admin

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.