Запрос комментариев: скрипт Ruby, который считает длину имени таблицы MySQL - PullRequest
0 голосов
/ 23 апреля 2010

Я новичок в ruby, и я хотел бы спросить вас, ребята, есть ли что-то, что может улучшить мой код Ruby. Вот мой сценарий:

#!/usr/bin/ruby -w

require 'mysql'
dbh = Mysql.real_connect('localhost', 'db_user', 'password', 'db_table')
tables = dbh.query('show tables')

tables.each do |table|
    puts "#{table}" + " (" + "#{table}".length.to_s + ")"
end

Мне бы очень хотелось услышать ваши комментарии. Заранее спасибо

Ответы [ 2 ]

1 голос
/ 23 апреля 2010

Выглядит хорошо, небольшие изменения, которые я бы сделал, это когда вы печатаете строку.Вместо объединения нескольких строк просто поместите все в одну строку.

Поэтому измените это:

puts "#{table}" + " (" + "#{table}".length.to_s + ")" на

puts "#{table} (#{table.length})".

1 голос
/ 23 апреля 2010

Мелкие детали, но любой из них выглядит чище, ИМХО, особенно первый, потому что он позволяет вам быстро визуализировать выходную раскладку:

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...