例：Br6, Qh1 アルファベットと数字の組み合わせで表現された通し番号をインクリメントするナイスな方法を教えてください

Question

■条件
・通し番号は3文字

・左側の文字について
・A~Zまでのアルファベット大文字
・Zまで進むとその次はAに戻る
・真ん中のアルファベットがzからaに進むタイミングでインクリメントする

・真ん中の文字について
・アルファベット小文字
・zまで進むとその次はaに戻る
・右側の文字が7から1に進むタイミングでインクリメントする

・右側の文字について
・1～7までの数字
・7まで進むとその次は1に戻る
・通し番号の最小の位

・処理を開始する通し番号は今回は決め打ちでBr6

def test_method(first_year,second_year)
in_group = "Br6"

  groups = []
  groups[0] = in_group
  a1 = in_group.slice(0)
  a2 = in_group.slice(1)
  i  = in_group.slice(2)
  51.times do |count|
    if i == "7"
      i = "1"
      if a2 == "z"
        a2 = "a"
        if a1 == "Z"
          a1 = "A"
        else
          a1.succ!
        end
      else
        a2.succ!
      end
    else
      i.succ!
    end
    groups[count+1] = a1+a2+i
  end
  return groups
end

凄い変なコードになってしまったので、かっこいい感じにできないでしょうか。
※ここに投稿するにあたって色々と削っており、うまく動作しないかもしれません。その際はご指摘いただけますと幸いです。

Answer 1

簡単な処理なので似たような感じになってしまいますが、配列の初期化処理を無くしてみました。

in_group = 'Br6'
units    = 52

groups =
  units.times.each_with_object([]) do |_, sum|
    sum.push(in_group)
    string, number = in_group[0..1], in_group[2]
    in_group = number == '7' ? string.next + '1' : string + number.next
  end

p groups
#=> ["Br6", "Br7", "Bs1", "Bs2", "Bs3", "Bs4", "Bs5", "Bs6", "Bs7", "Bt1", "Bt2", "Bt3", "Bt4", "Bt5", "Bt6", "Bt7", "Bu1", "Bu2", "Bu3", "Bu4", "Bu5", "Bu6", "Bu7", "Bv1", "Bv2", "Bv3", "Bv4", "Bv5", "Bv6", "Bv7", "Bw1", "Bw2", "Bw3", "Bw4", "Bw5", "Bw6", "Bw7", "Bx1", "Bx2", "Bx3", "Bx4", "Bx5", "Bx6", "Bx7", "By1", "By2", "By3", "By4", "By5", "By6", "By7", "Bz1"]

Answer 2

そんなあなたに String#next

in_group = 'Br6'
units    = 52

_, string, number = in_group.split(/(..)/)
groups = []

units.times do
  groups << string + number
  if number.next! == '8'
    string.next! 
    number = '1'
  end
end

p groups

# => [
 "Br6",
 "Br7",
 "Bs1",
 "Bs2",
 "Bs3",
 "Bs4",
 :
 "By5",
 "By6",
 "By7",
 "Bz1"
]

Comments

[hana-da]

あ、succ使ってたんですね。。。

'Bz'.succ # => 'Ca'

なので。。。あとは大丈夫ですね。

[退会済みユーザー]

ありがとうございます。string.nextを調べてみます。

[hana-da]

一緒です。nextもsuccも一緒です。ごめんなさい。質問よくみてませんでした。

[退会済みユーザー]

でもここの部分はだいぶかっこよくしていただいたので参考になります。
if number.next! == '8'
string.next!
number = '1'
end