Ruby Regex

I am new too using regex on files and I am not quite getting the
result I expect. The script currently doesn’t error but also doesn’t
update the file either.

Trying to remove leading numbers and ’ - 'from the start of a filename
given a directory.

require ‘fileutils’

def cleanFiles()
dir = ‘C:\Users\RenshawFamily\maven\Music\Foo Fighters\Live At
Wembley Stadium’

add files to array unless a directoy is found

myFiles = Array.new << File.split(dir) unless File.file?("")

Todo when files process step into directory and redo.

myFiles.each do|file|
# remove leading numbers a “-”
file.replace(/dd-/, ‘’)
Fileutils.mv(myFile, file)
end
end

I think you mean file.replace(/\d\d-/,’’) rather than
file.replace(/dd-/,’’)

/dd-/ will only match the character ‘d’ twice followed by a ‘-’

instead of dd, try \d\d
file.replace(/\d\d-/, ‘’)

in irb:

“12-test”.match(/\d\d-/)
=> #<MatchData “12-”>

On Mon, 9 May 2011 22:45:29 +0900

On Mon, May 9, 2011 at 3:45 PM, flebber [email protected] wrote:

dir = ‘C:\Users\RenshawFamily\maven\Music\Foo Fighters\Live At
Wembley Stadium’

Better uses forward slashes to avoid potential issues with quoting.

add files to array unless a directoy is found

myFiles = Array.new << File.split(dir) unless File.file?("")

You are adding two elements (dirname and basename) to a new Array. What
for?

Also, what is the test at the end of the line supposed to do?

Todo when files process step into directory and redo.

myFiles.each do|file|

Here you are iterating through an Array which you know to always only
contain 1 element (a nested Array with two elements, see above). It
does not make any sense to use an Array.

remove leading numbers a “-”

file.replace(/dd-/, ‘’)

You rather want

all numbers yield one dash

new_file = file.sub /\A\d+/, ‘-’

or

each number gets its own dash

new_file = file.sub(/\A\d+/) {|m| ‘-’ * m.length}

Fileutils.mv(myFile, file)

As far as I can see myFile is undefined here.

end
end

Overall the logic is quite dark to me…

Kind regards

robert

File.split() doesn’t do what you think it does, namely it does not
retrieve the files from the dir you specify. I’m not sure why you think
a method named split() will retrieve all the files from a directory. A
method named split() will typically ‘split’ a string into pieces based
on some criteria.

On Mon, May 9, 2011 at 3:45 PM, flebber [email protected] wrote:

require ‘fileutils’

def cleanFiles()
dir = ‘C:\Users\RenshawFamily\maven\Music\Foo Fighters\Live At
Wembley Stadium’

add files to array unless a directoy is found

myFiles = Array.new << File.split(dir) unless File.file?("")

You can simplify this enormously by using Dir#glob, like so:

files = Dir.glob(“C:/Users/RenshawFamily/maven/Music/Foo Fighters/Live
At Wembley Stadium/*.mp3”) #change for the format you use

This way, you get only MP3 files (or whatever ending you specify) in
an Array. Together with the regex advice, you’ll have less of a
headache with the script. :wink:


Phillip G.

Though the folk I have met,
(Ah, how soon!) they forget
When I’ve moved on to some other place,
There may be one or two,
When I’ve played and passed through,
Who’ll remember my song or my face.

On May 10, 5:34am, 7stud – [email protected] wrote:

File.split() doesn’t do what you think it does, namely it does not
retrieve the files from the dir you specify. I’m not sure why you think
a method named split() would retrieve all the files from a directory.


Posted viahttp://www.ruby-forum.com/.

@all good feedback much appreciated will work on it further and get
back with some results.

sayth

On May 10, 5:34am, 7stud – [email protected] wrote:

File.split() doesn’t do what you think it does, namely it does not
retrieve the files from the dir you specify. I’m not sure why you think
a method named split() would retrieve all the files from a directory.


Posted viahttp://www.ruby-forum.com/.

@Robert

Also, what is the test at the end of the line supposed to do?

Todo when files process step into directory and redo.

myFiles.each do|file|

Well in this first draft this check was just to ensure that no
directory were processed, but I wanted to develop it into a nested
test so the program would automatically process files in
subdirectories.

I need to review file.split for a suitable replacement.

so speaking it out, and I no there is errors but I am off reading RDOC
at moment :slight_smile: but this was the basic concept of where I was going.

myFiles = Array.new << File.split(dir) unless File.file?("") then
mydirs << Array.new
for myFiles in mfdirs.each do |file|
yadda yadda.

Sayth

On Tue, May 10, 2011 at 5:35 AM, flebber [email protected] wrote:

On May 10, 5:34am, 7stud – [email protected] wrote:

File.split() doesn’t do what you think it does, namely it does not
retrieve the files from the dir you specify. I’m not sure why you think
a method named split() would retrieve all the files from a directory.

Also, what is the test at the end of the line supposed to do?

Todo when files process step into directory and redo.

myFiles.each do|file|

OK, if you want to process multiple files you basically have these
options:

Dir[]
Dir.glob
Find.find

The advantage of the first two being that you can use a glob pattern
to select files and do not have to do it manually. Try this in IRB

Dir.glob(’**/[0-9]*’) {|f| p f, f.sub(/\A\d+/, ‘-’)}

myFiles = Array.new << File.split(dir) unless File.file?("") then
mydirs << Array.new
for myFiles in mfdirs.each do |file|
yadda yadda.

I am sorry, this is not a concept but few lines of source code. As
long as your are not yet familiar with functionality of the language
it’s probably easier to write down an algorithm in English words.
This also helps us understand what you are attempting to do. Only
then we be helpful.

Kind regards

robert

This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.

| Privacy Policy | Terms of Service | Remote Ruby Jobs