emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* how the $@%& do you add a property to a plist?
@ 2015-07-18 18:36 Matt Price
  2015-07-18 20:01 ` Thomas S. Dye
  0 siblings, 1 reply; 2+ messages in thread
From: Matt Price @ 2015-07-18 18:36 UTC (permalink / raw)
  To: Org Mode

[-- Attachment #1: Type: text/plain, Size: 1891 bytes --]

Hi,

I am continuing to work on my marking stuff; it's working really well,
actually.

I am currently importing a list of student names and emails from a csv file
-- this is normally how I get the list of enrolled students.  I have
modified the defauly settings from our Universities CSV export so I get
these files in the form:

Student Name,Email
Student One, em.address@utoronto.ca
Student Two, em.address2@utoronto.ca

I then parse this into a list, each element of which is itself a list
("Student One" "em.address@utoronto.ca"), etc.

This is OK, but not very robust. I would like instead to set *properties*,
in case the CSV file has some other ordering of fields.  So I've tried this:

(defun parse-plist-csv-file (file)
  (interactive
   (list (read-file-name "CSV file: ")))
  (let ((buf (find-file-noselect file))
        (result nil))
    (with-current-buffer buf
      (goto-char (point-min))
      (let ((header (split-string  (buffer-substring-no-properties
                                    (line-beginning-position)
(line-end-position)) ","))
            )
        (while (not (eobp))
          (let ((line  (split-string (buffer-substring-no-properties
                                      (line-beginning-position)
(line-end-position))))
                (count 0)
                (new-plist '()))
            (while (< count (length line))
              (setq plist-new (plist-put new-plist  '(nth count header)
(nth count line)))
              (setq count (1+ count)))
            (push  new-plist result)
            (forward-line 1))))
      (reverse result))))

The line that's not working here is:

              (setq plist-new (plist-put new-plist  '(nth count header)
(nth count line)))

which just inserts nil nil into 'new-plist. Obviously I'm making some
elementary elsip mistake, but my elisp is too weak to figure it out.  Can
you guys help?  thanks,
Matt

[-- Attachment #2: Type: text/html, Size: 2603 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: how the $@%& do you add a property to a plist?
  2015-07-18 18:36 how the $@%& do you add a property to a plist? Matt Price
@ 2015-07-18 20:01 ` Thomas S. Dye
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas S. Dye @ 2015-07-18 20:01 UTC (permalink / raw)
  To: Matt Price; +Cc: Org Mode


Matt Price <moptop99@gmail.com> writes:

> Hi,
>
> I am continuing to work on my marking stuff; it's working really well,
> actually.
>
> I am currently importing a list of student names and emails from a csv file
> -- this is normally how I get the list of enrolled students.  I have
> modified the defauly settings from our Universities CSV export so I get
> these files in the form:
>
> Student Name,Email
> Student One, em.address@utoronto.ca
> Student Two, em.address2@utoronto.ca
>
> I then parse this into a list, each element of which is itself a list
> ("Student One" "em.address@utoronto.ca"), etc.
>
> This is OK, but not very robust. I would like instead to set *properties*,
> in case the CSV file has some other ordering of fields.  So I've tried this:
>
> (defun parse-plist-csv-file (file)
>   (interactive
>    (list (read-file-name "CSV file: ")))
>   (let ((buf (find-file-noselect file))
>         (result nil))
>     (with-current-buffer buf
>       (goto-char (point-min))
>       (let ((header (split-string  (buffer-substring-no-properties
>                                     (line-beginning-position)
> (line-end-position)) ","))
>             )
>         (while (not (eobp))
>           (let ((line  (split-string (buffer-substring-no-properties
>                                       (line-beginning-position)
> (line-end-position))))
>                 (count 0)
>                 (new-plist '()))
>             (while (< count (length line))
>               (setq plist-new (plist-put new-plist  '(nth count header)
> (nth count line)))
>               (setq count (1+ count)))
>             (push  new-plist result)
>             (forward-line 1))))
>       (reverse result))))
>
> The line that's not working here is:
>
>               (setq plist-new (plist-put new-plist  '(nth count header)
> (nth count line)))
>
> which just inserts nil nil into 'new-plist. Obviously I'm making some
> elementary elsip mistake, but my elisp is too weak to figure it out.  Can
> you guys help?  thanks,
> Matt

According to plist-put's documentation:

(setq x (plist-put x prop val))

(setq new-plist (plist-put new-plist  (nth count header) (nth count line)))
      ^^^^^^^^^                       ^

So, you don't want plist-new, I think.  Also, quoting the first (nth
...)  list means the function nth is interpreted as a value of the list,
rather than a function.  I think you want it to return a property.

I didn't test any of this, so your mmv.

hth,
Tom

-- 
Thomas S. Dye
http://www.tsdye.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-07-18 20:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-18 18:36 how the $@%& do you add a property to a plist? Matt Price
2015-07-18 20:01 ` Thomas S. Dye

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).