Journal InTime

_ Schwartzian Transform

ってのを試してみたら、

ary.collect {|i| [i.key1, i.key2, i]}.sort.collect {|j| j.last}

みたいな時に、最初のブロックのiが<=>を持ってないと、key1/key2が同じ値の要素があった時に、

test.rb:24:in `<=>': undefined method `<=>' for #<Foo:0x401b794c> (NoMethodError)
        from test.rb:24:in `sort'
        from test.rb:24

のようなエラーになってしまう。(当たり前か。)

<=>を定義せずに済ませることってできないかなあ。 たとえば、いくつかの属性を持つオブジェクトがあったとして、どの属性も重み(?)が同じだとすると、 デフォルトの<=>って定義したくないような気がするんだけど。 それにデフォルトの<=>があったとしても、上の例ではほんとは呼ぶ必要はない(配列に入れてるのは 比較に利用するためじゃなくて後で値を取り出すためだから)わけだし。

class STArray < Array
  attr_reader :value

  def initialize(value, *args)
    @value = value
    replace(args)
  end
end
...
p ary.collect {|i| STArray.new(i, i.key1, i.key2)}.sort.collect {|i| i.value}

くらいかなあ。ちょっとおおげさ?

_ t-code for skkinput3

t-code for skkinput3(<URL:http://openlab.jp/tcode/tc2/index.html#develop>) を導入。ナイスだ。

でもちょっと動きが遅いみたい。 skkinput3のlisp interpreterのせいだろうか。

_ encoding support for MySQL adapter

もりきゅうさんの記事を参考に MySQLアダプタにencodingオプションを追加するパッチを作って、tracに投げてみた。

development:
  adapter: mysql
  database: depot_development
  socket: /var/run/mysqld/mysqld.sock
  username: shugo
  password:
  encoding: cp932

みたいな感じで設定する。

"表"のquoteもOKで、binary_test.rbもちゃんと通ったけど、なぜかencoding: sjisだとquote のテストでコケる。 "\\"が"\201_"になるみたいだけど、MySQLの変換テーブルのせい?

quoteまわりはPostgreSQLアダプタもあやしい気がするけど、Mysql.quoteじゃなくてMysql#quoteを使うべきだということになかなか気付かずハマったのと、MySQLのバイナリのquote方法をしらべるので力尽きたので、今日はこれまで。

_ コンビニメソッド

コンビニメソッド、ということでいかがですか。

[丁稚な日々より引用]

文体と合わないので却下の方向で。

ちなみにコンビニ関数 というのはすでに定着してるようですね(ほんとか)。

_ encoding support for MySQL adapter(2)

MySQL#real_connect は self が返るので、そのパッチだと mysql == conn ですね。 つまり @connection.quote でいいわけです。

[3 日坊主日記 - encoding support for MySQL adapter , 高速バスの予約より引用]

がーん、そうだったんですか。 と思ったらもうcommitされてるし。

reconnect!も考えると、こんな感じですかね。

Index: lib/active_record/connection_adapters/mysql_adapter.rb
===================================================================
--- lib/active_record/connection_adapters/mysql_adapter.rb	(revision 3155)
+++ lib/active_record/connection_adapters/mysql_adapter.rb	(working copy)
@@ -38,17 +38,7 @@

       mysql = Mysql.init
       mysql.ssl_set(config[:sslkey], config[:sslcert], config[:sslca], config[:sslcapath], config[:sslcipher]) if config[:sslkey]
-      if config[:encoding]
-        begin
-          mysql.options(Mysql::SET_CHARSET_NAME, config[:encoding])
-        rescue
-          raise ActiveRecord::ConnectionFailed, 'The :encoding option is only available for MySQL 4.1 and later with the mysql-ruby driver.  Again, this does not work with the ruby-mysql driver or MySQL < 4.1.'
-        end
-      end
-
-      conn = mysql.real_connect(host, username, password, database, port, socket)
-      conn.query("SET NAMES '#{config[:encoding]}'") if config[:encoding]
-      ConnectionAdapters::MysqlAdapter.new(conn, logger, [host, username, password, database, port, socket], mysql)
+      ConnectionAdapters::MysqlAdapter.new(mysql, logger, [host, username, password, database, port, socket], config[:encoding])
     end
   end

@@ -97,10 +87,11 @@
         "MySQL server has gone away"
       ]

-      def initialize(connection, logger, connection_options=nil, mysql=Mysql)
+      def initialize(connection, logger, connection_options=nil, encoding=nil)
         super(connection, logger)
         @connection_options = connection_options
-        @mysql = mysql
+        @encoding = encoding
+        connect
       end

       def adapter_name #:nodoc:
@@ -144,7 +135,7 @@
       end

       def quote_string(string) #:nodoc:
-        @mysql.quote(string)
+        @connection.quote(string)
       end

       def quoted_true
@@ -170,7 +161,7 @@
           @connection.ping
         else
           @connection.close rescue nil
-          @connection.real_connect(*@connection_options)
+          connect
         end
       end

@@ -318,6 +309,18 @@


       private
+        def connect
+          if @encoding
+            begin
+              @connection.options(Mysql::SET_CHARSET_NAME, @encoding)
+            rescue
+              raise ActiveRecord::ConnectionFailed, 'The :encoding option is only available for MySQL 4.1 and later with the mysql-ruby driver.  Again, this does not work with the ruby-mysql driver or MySQL < 4.1.'
+            end
+          end
+          @connection.real_connect(*@connection_options)
+          @connection.query("SET NAMES '#{@encoding}'") if @encoding
+        end
+
         def select(sql, name = nil)
           @connection.query_with_result = true
           result = execute(sql, name)

追記:

PostgreSQLの方も修正してパッチをtracにつっこんどきました。