Slide 1
Slide 1 text
レガシーシステムが あらわれた！
diverse.tech ＃１
~ レガシーシステムを かたらう よる ~
Slide 2
Slide 2 text
これは レガシーな プログラムを
かいぜんしていくための はじめの
いっぽをふみだす おはなしです。
Slide 3
Slide 3 text
アバウト ミー
•かぶしきがいしゃ Diverse
•にゅうしゃ 3かげつめの かけだし サーバーサイドエンジニア
•Github
•https://github.com/k-motoyan
Slide 4
Slide 4 text
それは あるひの できごとでした…
Slide 5
Slide 5 text
「この むかしから ある システムに
あたらしく きのうを ついかしたいんだ。」
Slide 6
Slide 6 text
「この むかしから ある システムに
あたらしく きのうを ついかしたいんだ。」
はい
いいえ
Slide 7
Slide 7 text
「この むかしから ある システムに
あたらしく きのうを ついかしたいんだ。」
はい
→ いいえ
Slide 8
Slide 8 text
「げんじつ からは にげられない！」
Slide 9
Slide 9 text
「どうする？」
コードを みる
しようを かくにんする
Slide 10
Slide 10 text
「どうする？」
コードを みる
→ しようを かくにんする
Slide 11
Slide 11 text
「しようを しっているひとは
もういないよ。」
Slide 12
Slide 12 text
「かいはつチームの めのまえが
まっくらになった。」
Slide 13
Slide 13 text
No content
Slide 14
Slide 14 text
「どうする？」
コードを みる
しようを かくにんする
Slide 15
Slide 15 text
「どうする？」
→ コードを みる
しようを かくにんする
Slide 16
Slide 16 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 17
Slide 17 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
これは かくうの ソースコードです
Slide 18
Slide 18 text
「さわるだけで デグレードしそうな
ソースコードだ。」
Slide 19
Slide 19 text
「どうする？」
はらをくくって きのうを ついかする
テストを かく
Slide 20
Slide 20 text
「どうする？」
はらをくくって きのうを ついかする
→ テストを かく
Slide 21
Slide 21 text
「どの テストを かく？」
ユニット テスト
しようか テスト
Slide 22
Slide 22 text
「どの テストを かく？」
→ ユニット テスト
しようか テスト
Slide 23
Slide 23 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 24
Slide 24 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 25
Slide 25 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 26
Slide 26 text
「いぞんどの たかい
コードが うめこまれていて
ユニットテストを かくことが むずかしい。」
Slide 27
Slide 27 text
「どの テストを かく？」
ユニット テスト
しようか テスト
Slide 28
Slide 28 text
「どの テストを かく？」
ユニット テスト
→ しようか テスト
Slide 29
Slide 29 text
ʊਓਓਓਓਓਓਓਓਓਓʊ
ʼɹ͠Α͏͔ɹςετɹʻ
ʉ:?:?:?:?:?:?:?:?:ʉ
Slide 30
Slide 30 text
༷Խςετ
DIBSBDUFSJ[BUJPOUFTU
༷ԽςετͱɺίʔυͷݱࡏͷৼΔ͍Λ֬ೝ͢ΔͨΊ
ͷςετͰ͢ɻ
ݱࡏͷৼΔ͍Λςετͱͯ͠ॻ͖ى͜͢͜ͱͰɺͦͷίʔ
υʹରͯ͠मਖ਼ΛՃ͑ͨ࠶ʹ͏ΤϯόάɾσάϨʔυΛ
੍͢ΔޮՌ͕͋Γ·͢ɻ
͋͘·ͰݱࡏͷৼΔ͍Λ֬ೝ͢ΔͨΊͷςετͰ͋Γɺ
༷͕උ͞ΕΔλΠϛϯάͰ༷ԽςετࣺͯΔ͔मਖ਼
͢Δඞཁ͕ग़ͯ͘Δɻ
Slide 31
Slide 31 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 32
Slide 32 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 33
Slide 33 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
Slide 34
Slide 34 text
class UserController < ApplicationController
def index
@user = User.find_by!(id: params[:id], status: :active)
if [email protected]_debug_user?
AccessLog.create user_id: @user.id,
user_name: @user.name,
access_page: "user_index"
end
begin
@user.friends.each do |friend|
if !friend.enabled_friend_access_notification
@notification =
Notification.create user_id: @friend.id
friend_id: @user.id
message: "#{@user.name}͕ϚΠϖʔδʹΞΫηε͠·ͨ͠"
end
PushService.send(@notification)
end
rescue => e
@error = ErrorLog.create message: "ϑϨϯυͷ௨ʹࣦഊ͠·ͨ͠"
redirect_to error_page_path(@error)
end
case @user.score
when 0..10
render :score1_user
when 11..30
render :score2_user
when 31..50
render :score3_user
when 51..70
render :score4_user
when 81..100
render :score5_user
end
end
end
71 から 80 のケースがない！？
Slide 35
Slide 35 text
「バグなのか しようなのか わからない
コードに しようか テストを かいた。」
Slide 36
Slide 36 text
「こうかは バツグンだ！」
Slide 37
Slide 37 text
「かいはつチームは しようか テスト で、
きのうついかの かいはつが
おこなえるようになった！！」
Slide 38
Slide 38 text
こうして むかしからある
システムに きのうを ついかする
おしごとを こなすことが できました。
Slide 39
Slide 39 text
しかし むかしからある、
レガシーシステムがもつ もんだいは
これだけでは ありません。
Slide 40
Slide 40 text
かいはつチームの たたかいは
まだまだ つづく…
Slide 41
Slide 41 text
さんこう ぶんけん
ΫϦεɾόʔνϟϧʢஶʣɹ٢ɹʢ༁ʣ
ʰϨΨγʔιϑτΣΞվળΨΠυʱɹᠳӭࣾ
Slide 42
Slide 42 text
Special Thanks
υοτ෩ϑΥϯτ
SDFɿIUUQTXXXIPHFSBDPNQDCGPOU
ձઃӦΛͯ͘͠Εͨ%JWFSTFͷํʑ
ͦͯ͠ɺࠓ͝དྷ͍ͨօ༷