xlwt save dlg

wxPython Users
1

I’m at the point where I need some help on this.

My dialog save as txt or xls. However the reader

can’t recognize the delimiter. (parces all to first column)

I have the file object that the reader needs!!!

def checkBtnClick(self, event):
wildcard = “(.txt)|.txt|”
“(.xls)|.xls|”
dlg = wx.FileDialog(
self, message=“Save file as …”,
defaultDir=".",
defaultFile="", wildcard=wildcard, style=wx.SAVE | wx.OVERWRITE_PROMPT
)
if dlg.ShowModal() == wx.ID_OK:
bg = self.newlist
self.filename=dlg.GetFilename()
self.dirname=dlg.GetDirectory()
filehandle=open(os.path.join(self.dirname, self.filename),‘w’)
filehandle.write(bg)

f = StringIO(self.newlist)
print ‘f is %s’ % f.read()
for filename in glob.glob(f.read()):
wb = xlwt.Workbook()
sheet = wb.add_sheet(‘sheet 1’)
newName = filename
print filename
spamReader = csv.reader(open(filename, ‘rb’), delimiter=’,’)
for rowx, row in enumerate(spamReader):
for colx, value in enumerate(row):
sheet.write(rowx, colx, value)
wb.save(newName + “.xls”)
print “Done”

2

George McCown wrote:

I'm at the point where I need some help on this.
My dialog save as txt or xls.

Not really. The only thing you save into that file is the list of
patterns from self.newlist. That is NOT Excel data, so you should not
pretend that you can create an .xls file.

However the reader can't recognize the delimiter. (parces all to
first column)
I have the file object that the reader needs!!!

Are you quite sure that each of the files in self.newlist actually
contains comma-separated data? Perhaps you should show us one of those
files.

···

--
Tim Roberts, timr@probo.com
Providenza & Boekelheide, Inc.

3

George McCown wrote:

I’m at the point where I need some help on this.

My dialog save as txt or xls.

Not really. The only thing you save into that file is the list of

patterns from self.newlist. That is NOT Excel data, so you should not

pretend that you can create an .xls file.

However the reader can’t recognize the delimiter. (parces all to

first column)
I have the file object that the reader needs!!!

Are you quite sure that each of the files in self.newlist actually

contains comma-separated data? Perhaps you should show us one of those

files.

–
Tim Roberts, ti...@probo.com

Providenza & Boekelheide, Inc.

self.newlist re-patterned by a regex method:

OBJECTID,FULL_ADDRESS
8759,1006 CARDINAL PL
8724,1006 CANARY LN
17745,1005 WESTCHESTER DR
9774,1005 W WABASH AVE

That is NOT Excel data, so you should not

pretend that you can create an .xls file.

Tim, please clarify this statement.

···

On Monday, December 16, 2013 5:12:41 PM UTC-6, Tim Roberts wrote:

4

George McCown wrote:

self.newlist re-patterned by a regex method:

OBJECTID,FULL_ADDRESS
8759,1006 CARDINAL PL
8724,1006 CANARY LN
17745,1005 WESTCHESTER DR
9774,1005 W WABASH AVE

Well, now, hold on. The code you showed us expects self.newlist to
contain a list of filenames (or filename glob patterns). Each of the
files in that list is then expected to contain comma-separated data,
like that. Are you saying that data above is what self.newlist
contains? If so, then your code is incorrect. You shouldn't be using
glob.glob at all.

Actually, there's a more important problem that I just noticed. You
have this:
            f = StringIO(self.newlist)
            print 'f is %s' % f.read()
            for filename in glob.glob(f.read()):

That "print" statement is going to use up all of the data from the
list. When that's done, "f" will be at end-of-file. Thus, the f.read()
in the next line will read nothing at all.

So, what, EXACTLY, is in self.newlist? Is it one file name? Is it
several file names? Is it the comma-separated data above? If you can
answer that, then we may be able to fix your code.

>>That is NOT Excel data, so you should not
>>pretend that you can create an .xls file.

Tim, please clarify this statement.

Well, look at what the code does. You call wx.FileDialog. If that
returns OK, then you fetch the filename and directory that the user
specified. You then create that file, and write the contents of
"self.newlist" to it. So, what you have written is NOT an Excel .xls
spreadsheet. It's just plain text. If it really is comma-separated
data, then you could use the ".csv" extension (which Excel knows how to
open), but it's definitely not a .xls file. .xls files are a special
binary format known only to Excel.

Now, you do have code later on that tries to create a genuine Excel .xls
files, but you're not using the filename from the wx.FileDialog. You're
using the filenames from self.newlist.

So, what were you really trying to do here? Were you trying to say "If
the user specified a .txt name, then save self.newlist in
comma-separated form. If the user specified a .xls name, then create an
Excel spreadsheet."? If that was your goal, then I think we can help
you do that, but it's quite different from the code you have. Something
like this:

        if dlg.ShowModal() == wx.ID_OK:
            ext = os.path.splitext(dlg.GetFilename())[1]
            if ext == '.txt':
                # They asked for a .txt file.
                f = open(dlg.GetPath(),'w')
                f.write(self.newlist)
            else:
                # They asked for a .xls file.
                f = StringIO(self.newlist)
                wb = xlwt.Workbook()
                sheet = wb.add_sheet('sheet 1')
                reader = csv.reader(f, 'rb'), delimiter=',')
                for rowx, row in enumerate(reader):
                    for colx, value in enumerate(row):
                        sheet.write(rowx, colx, value)
                wb.save(dlg.GetPath())
            print "Done"

···

--
Tim Roberts, timr@probo.com
Providenza & Boekelheide, Inc.

5

Thanks Tim, I implimented your if method and changed the reader to say: reader = csv.reader(f)

I believe the csv mod uses comma delimited by default.

···

On Monday, December 16, 2013 8:23:18 PM UTC-6, Tim Roberts wrote:

George McCown wrote:

self.newlist re-patterned by a regex method:

OBJECTID,FULL_ADDRESS

8759,1006 CARDINAL PL

8724,1006 CANARY LN

17745,1005 WESTCHESTER DR

9774,1005 W WABASH AVE

Well, now, hold on. The code you showed us expects self.newlist to

contain a list of filenames (or filename glob patterns). Each of the

files in that list is then expected to contain comma-separated data,

like that. Are you saying that data above is what self.newlist

contains? If so, then your code is incorrect. You shouldn’t be using

glob.glob at all.

Actually, there’s a more important problem that I just noticed. You

have this:

        f = StringIO(self.newlist)

        print 'f is %s' % f.read()

        for filename in glob.glob(f.read()):

That “print” statement is going to use up all of the data from the

list. When that’s done, “f” will be at end-of-file. Thus, the f.read()

in the next line will read nothing at all.

So, what, EXACTLY, is in self.newlist? Is it one file name? Is it

several file names? Is it the comma-separated data above? If you can

answer that, then we may be able to fix your code.

That is NOT Excel data, so you should not

pretend that you can create an .xls file.

Tim, please clarify this statement.

Well, look at what the code does. You call wx.FileDialog. If that

returns OK, then you fetch the filename and directory that the user

specified. You then create that file, and write the contents of

“self.newlist” to it. So, what you have written is NOT an Excel .xls

spreadsheet. It’s just plain text. If it really is comma-separated

data, then you could use the “.csv” extension (which Excel knows how to

open), but it’s definitely not a .xls file. .xls files are a special

binary format known only to Excel.

Now, you do have code later on that tries to create a genuine Excel .xls

files, but you’re not using the filename from the wx.FileDialog. You’re

using the filenames from self.newlist.

So, what were you really trying to do here? Were you trying to say "If

the user specified a .txt name, then save self.newlist in

comma-separated form. If the user specified a .xls name, then create an

Excel spreadsheet."? If that was your goal, then I think we can help

you do that, but it’s quite different from the code you have. Something

like this:

    if dlg.ShowModal() == wx.ID_OK:

        ext = os.path.splitext(dlg.GetFilename())[1]

        if ext == '.txt':

            # They asked for a .txt file.

            f = open(dlg.GetPath(),'w')

            f.write(self.newlist)

        else:

            # They asked for a .xls file.

            f = StringIO(self.newlist)

            wb = xlwt.Workbook()

            sheet = wb.add_sheet('sheet 1')

            reader = csv.reader(f, 'rb'), delimiter=',')

            for rowx, row in enumerate(reader):

                for colx, value in enumerate(row):

                    sheet.write(rowx, colx, value)

            wb.save(dlg.GetPath())

        print "Done"

–
Tim Roberts, ti...@probo.com

Providenza & Boekelheide, Inc.