Comments on: Ruby String#Nameize Revised

By: Patrick Tulskie

Patrick Tulskie — Sun, 20 Sep 2009 23:42:39 +0000

Thanks for that raggi. Great set of tweaks. Yeah I’ve come to learn a great deal since I originally wrote this dirty code. Case has been one of those things I’ve stopped abusing such as you saw before. I’m glad that people are still finding this useful and improving upon it. As you’ll probably notice, I’ve stopped writing on my blog as much and have been reading through the code of others’ as of late.

By: raggi

raggi — Thu, 10 Sep 2009 16:14:05 +0000

I can see where you’re trying to apply “performance optimizations” but they’re introducing bugs, that you don’t seem to have benchmark, profile, or test coverage for.

You don’t need gsub with a regex of /^ … /

You could do the entire sub line operation in a single sub, and probably should do. Sub isn’t delete, afterall. Oh, and the two sub cases are equivalent, with a different first group.

Stop using String#match without utilising the match data, and for that matter, use case properly, rather than this new way you’ve learned.

Oh screw it… here comes codes…

class String SPACE = ' ' APOS = "'"

# Extension of the string class to properly handle camel names def nameize case self when / / # If the name has a space in it, we gotta run the parts through the nameizer. split(SPACE).each { |part| part.nameize! }.join(SPACE) when /^[A-Z]/ # If they took the time to capitalize their name then let's just jump out. self when /^(mac|mc)(\w)(.*)$/i "#{$1.capitalize}#{$2.capitalize}#{$3}" when /^o\'/i split(APOS).each{ |piece| piece.capitalize! }.join(APOS) else capitalize # Basically if the name is a first name or it's not Irish then capitalize it. end end def nameize! replace nameize # BANG! end end

require 'rubygems' require 'minitest/unit' include MiniTest::Assertions assert_equal("Martin O'Donald", "martin o'donald".nameize) assert_equal("Randy McFarley", "Randy mcfarley".nameize) assert_equal("Sue MacDonald", "sue macdonald".nameize)

Have fun, and play safe.

By: Brookr

Brookr — Fri, 28 Aug 2009 06:46:23 +0000

Thanks for sharing this, just what I needed!

By: Oz

Oz — Sat, 07 Feb 2009 07:46:36 +0000

Hey Patrick, sorry that this comment isn’t really about the post, but I just saw this site and I really like it! Nice layout and everything. I enjoy your writing style too

By: Clinton Skakun

Clinton Skakun — Wed, 21 Jan 2009 23:39:01 +0000

Can’t wait to get into Ruby! Looks awesome.

Currently, I use PHP for application development. Instead of using Ruby for that I’m going to use it for building programs with GUIs. I’m looking into building in GTK and Qt. GTK because it’s entirely free and Qt because it’s the best, even though the license costs money. Never know I might buy it, but not until I’m made some real money on software development.

Anyways I think it’s great that you’re into Ruby.

Have a great one!
Clinton Skakun