SlideShare uma empresa Scribd logo
1 de 91
golintを使おう
golint
Copyright © 2009-2015 eureka, inc. All rights reserved.
もくじ
・じこしょうかい
・golintについて
・golintをみてみる
・わたしとgolint
じこしょうかい
- なまえ: Takuma Morikawa
- しょぞく:
- たんとう:
じこしょうかい
- なまえ: Takuma Morikawa
- しょぞく:
- たんとう:
(Go, JS, たまにインフラ)
じこしょうかい
- なまえ: Takuma Morikawa
- しょぞく:
- たんとう:
(Go, JS, たまにインフラ)
aws-sdk-goのラッパーとか
https://github.com/evalphobia/aws-sdk-go-wrapper
じこしょうかい
- なまえ: Takuma Morikawa
- しょぞく:
- たんとう:
(Go, JS, たまにインフラ)
aws-sdk-goのラッパーとか
https://github.com/evalphobia/aws-sdk-go-wrapper
golintについて
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
みなさんつかってますか?
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
みなさんつかってますか?
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
みなさんつかってます
か?
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
もちろん私は
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
設定していませんでした…orz
Copyright © 2009-2015 eureka, inc. All rights reserved.
golintについて
$ golint ./…
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
2) <XXX>でテストコードを書く
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
2) <XXX>でテストコードを書く
3) ʕº̫͡ºʔo00(git push origin dev [ターン!])
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
2) <XXX>でテストコードを書く
3) ʕº̫͡ºʔo00(git push origin dev [ターン!])
4) CircleCIがエラーを検知
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
2) <XXX>でテストコードを書く
3) ʕº̫͡ºʔo00(git push origin dev [ターン!])
4) CircleCIがエラーを検知
5) (Slackへのエラー通知でざわつく)
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
2) <XXX>でテストコードを書く
3) ʕº̫͡ºʔo00(git push origin dev [ターン!])
4) CircleCIがエラーを検知
5) (Slackへのエラー通知でざわつく)
6) (*ノェノ)キャー m9(^Д^)プギャー
Copyright © 2009-2015 eureka, inc. All rights reserved.
あのころのわたしたち
1) <XXX>でコードを書く
2) <XXX>でテストコードを書く
3) ʕº̫͡ºʔo00(git push origin dev [ターン!])
4) CircleCIがエラーを検知
5) (Slackへのエラー通知でざわつく)
6) (*ノェノ)キャー m9(^Д^)プギャー
< おっけー!
Copyright © 2009-2015 eureka, inc. All rights reserved.
ある日
ʕº̫͡ºʔo00(あれ、、、)
Copyright © 2009-2015 eureka, inc. All rights reserved.
ある日
ʕº̫͡ºʔo00(あれ、、、)
ʕº̫͡ºʔo00(pre-commitがうまく動いてないぞ…)
$ golint ./… 2>&1 | wc -l
Copyright © 2009-2015 eureka, inc. All rights reserved.
ある日
ʕº̫͡ºʔo00(ちゃんと動作を確認しましょう…)
https://gist.github.com/evalphobia/cf79c1911df7a5ae3f00
golintをみてみる
golintをみてみる
https://github.com/golang/lint/blob/master/lint.go
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lint()
← こ の 1 7 の関数内でファイルごと
にチェックをかけている
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintPackageComment()
← パ ッ ケ ー ジ の コ メ ン ト を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintPackageComment()
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintPackageComment()
↑ パ ッ ケ ー ジ コ メ ン ト が 離 れ て い る
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintPackageComment()
↑ パ ッ ケ ー ジ コ メ ン ト が な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintPackageComment()
↑ パ ッ ケ ー ジ コ メ ン ト の 前 に ス ペ ー ス が あ る
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintPackageComment()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintImports()
← i m p o r t を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintImports()
↑ ド ッ ト イ ン ポ ー ト 、 ダ メ 、 ゼ ッ タ イ
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintBlankImports()
← 無 名 i m p o r t 「 _ 」 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintBlankImports()
↑ ド ッ ト イ ン ポ ー ト 、 ダ メ 、 ゼ ッ タ イ↑ 無 名 イ ン ポ ー ト を テ ス ト や m a i n ( ) 以 外 で 使 っ て い る
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintExported()
← 大 文 字 で 始 ま る 変 数 や 関 数 を チ ェ ッ ク
( 公 開 状 態 )
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintExported()
↑ ド ッ ト イ ン ポ ー ト 、 ダ メ 、 ゼ ッ タ イ
↑ パ ッ ケ ー ジ 名 と 同 じ p r e f i x を 持 っ て い る
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintExported()
↑ 公 開 変 数 ( p u b l i c ) は 、 個 別 に 定 義 す る
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintExported()
↑ 公 開 し て い る も の に は コ メ ン ト を つ け ま し ょ う
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintExported()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
← 命 名 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ パ ッ ケ ー ジ 名 に ア ン ス コ 「 _ 」 は 使 わ な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ 名 前 に 全 部 大 文 字 を 使 っ ち ゃ あ か ん
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ k X X X の よ う な 化 石 命 名 も あ か ん
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ 名 前 に も ア ン ス コ は 使 っ ち ゃ あ か ん
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↓ ※ i d と か h t t p と か は 大 文 字 小 文 字 を 統 一 す る
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintNames()
← 変 数 の 宣 言 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintVarDecls()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ 0 値 で 初 期 化 し な く て も い い ん だ よ
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintVarDecls()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ 右 辺 が 類 推 可 能 な 値 の 場 合 は 型 宣 言 不 要 だ よ
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintElses()
← e l s e ブ ロ ッ ク を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintElses()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ i f 文 最 後 の e l s e は r e t u r n 返 す な ら い ら な い よ !
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintRanges()
← r a n g e を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintRanges()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ f o r r a n g e 文 で k e y し か 使 わ な い 場 合 は 、
2 番 目 の 変 数 を 「 _ 」 で 捨 て ず に 省 略 し た ほ う が え え
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorf()
← e r r o r . N e w を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorf()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑ e r r o r s . N e w ( f m t . S p r i n t f ( ) ) だ っ て ?
f m t . E r r o r f ( ) っ て い う の が あ っ て だ な …
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrors()
← e r r o r 変 数 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrors()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ グ ロ ー バ ル な エ ラ ー 変 数 の p r e f i x は " e r r " に し ま し ょ う
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorStrings()
← エ ラ ー 文 言 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorStrings()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ エ ラ ー 文 言 を 大 文 字 で 始 め る の は 非 推 奨 ※
↓ ※ し き い 値 は 0 . 6 な の で 通 常 は 表 示 さ れ な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorStrings()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ エ ラ ー 文 言 に 句 読 点 を つ け て は い け な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintReceiverNames()
← メ ソ ッ ド の レ シ ー バ 名 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintReceiverNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ レ シ ー バ 変 数 を ア ン ス コ 「 _ 」 に し ち ゃ あ か ん
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintReceiverNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ 「 m e 」 「 t h i s 」 「 s e l f 」 も あ か ん よ な あ
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintReceiverNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ 同 じ 構 造 体 で レ シ ー バ 変 数 名 は 統 一 し よ う
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintIntDec()
← イ ン ク リ メ ン ト ・ デ ク リ メ ン ト
を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintIntDec()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ 「 i + = 1 」 は 「 i + + 」 に し た ほ う が い い よ
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintMake()
← m a k e ( ) を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintMake()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ 空 の 配 列 は m a k e し な い で 宣 言 す る だ け の 方 が い い よ
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorReturn()
← r e t u r n 時 の e r r o r 型 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintErrorReturn()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ 関 数 で 複 値 を 返 す 場 合 、 e r r o r 型 は 最 後 に し ま し ょ う
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintUnexportedReturn()
← 非 公 開 型 の r e t u r n を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintUnexportedReturn()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
↑ 公 開 関 数 で 非 公 開 の 構 造 体 を 返 す の は あ か ん !
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintTimeNames()
← t i m e の 命 名 を チ ェ ッ ク
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go lintTimeNames()
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑↑ t i m e . D u r a t i o n 型 に 時 間 の 単 位 の 名 前 は つ け て は い け な い
Copyright © 2009-2015 eureka, inc. All rights reserved.
続きコードはWebで
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
[golintのエラーサンプル]
https://github.com/evalphobia/golint-sample
Copyright © 2009-2015 eureka, inc. All rights reserved.
続きコードはWebで
↑ コ メ ン ト の 形 式 に 添 っ て い な い
↑
[golintのエラーサンプル]
https://github.com/evalphobia/golint-sample
[公式にあった…orz]
https://github.com/golang/lint/tree/master/testdata
Copyright © 2009-2015 eureka, inc. All rights reserved.
lint.go しきい値
↑ 「 - m i n _ c o n f i d e n c e 」
Copyright © 2009-2015 eureka, inc. All rights reserved.
0.8以下のエラー
↑ パ ッ ケ ー ジ コ メ ン ト が な い
↑ e r r o r s . N e w ( s t r i n g ) の 文 字 列 が 大 文 字 で 始 ま っ て い る
わたしとgolint
わたしとgolint
もし、できることなら
わたしとgolint
あなただけの専用lintを作ってみたいと思いませんか?
わたしとgolint
…
わたしとgolint
間に合いませんでした…
thank you : )
Copyright © 2009-2015 eureka, inc. All rights reserved.

Mais conteúdo relacionado

Destaque

How Race, Age and Gender Shape Attitudes Towards Mental Health
How Race, Age and Gender Shape Attitudes Towards Mental HealthHow Race, Age and Gender Shape Attitudes Towards Mental Health
How Race, Age and Gender Shape Attitudes Towards Mental Health
ThinkNow
 
Social Media Marketing Trends 2024 // The Global Indie Insights
Social Media Marketing Trends 2024 // The Global Indie InsightsSocial Media Marketing Trends 2024 // The Global Indie Insights
Social Media Marketing Trends 2024 // The Global Indie Insights
Kurio // The Social Media Age(ncy)
 

Destaque (20)

2024 State of Marketing Report – by Hubspot
2024 State of Marketing Report – by Hubspot2024 State of Marketing Report – by Hubspot
2024 State of Marketing Report – by Hubspot
 
Everything You Need To Know About ChatGPT
Everything You Need To Know About ChatGPTEverything You Need To Know About ChatGPT
Everything You Need To Know About ChatGPT
 
Product Design Trends in 2024 | Teenage Engineerings
Product Design Trends in 2024 | Teenage EngineeringsProduct Design Trends in 2024 | Teenage Engineerings
Product Design Trends in 2024 | Teenage Engineerings
 
How Race, Age and Gender Shape Attitudes Towards Mental Health
How Race, Age and Gender Shape Attitudes Towards Mental HealthHow Race, Age and Gender Shape Attitudes Towards Mental Health
How Race, Age and Gender Shape Attitudes Towards Mental Health
 
AI Trends in Creative Operations 2024 by Artwork Flow.pdf
AI Trends in Creative Operations 2024 by Artwork Flow.pdfAI Trends in Creative Operations 2024 by Artwork Flow.pdf
AI Trends in Creative Operations 2024 by Artwork Flow.pdf
 
Skeleton Culture Code
Skeleton Culture CodeSkeleton Culture Code
Skeleton Culture Code
 
PEPSICO Presentation to CAGNY Conference Feb 2024
PEPSICO Presentation to CAGNY Conference Feb 2024PEPSICO Presentation to CAGNY Conference Feb 2024
PEPSICO Presentation to CAGNY Conference Feb 2024
 
Content Methodology: A Best Practices Report (Webinar)
Content Methodology: A Best Practices Report (Webinar)Content Methodology: A Best Practices Report (Webinar)
Content Methodology: A Best Practices Report (Webinar)
 
How to Prepare For a Successful Job Search for 2024
How to Prepare For a Successful Job Search for 2024How to Prepare For a Successful Job Search for 2024
How to Prepare For a Successful Job Search for 2024
 
Social Media Marketing Trends 2024 // The Global Indie Insights
Social Media Marketing Trends 2024 // The Global Indie InsightsSocial Media Marketing Trends 2024 // The Global Indie Insights
Social Media Marketing Trends 2024 // The Global Indie Insights
 
Trends In Paid Search: Navigating The Digital Landscape In 2024
Trends In Paid Search: Navigating The Digital Landscape In 2024Trends In Paid Search: Navigating The Digital Landscape In 2024
Trends In Paid Search: Navigating The Digital Landscape In 2024
 
5 Public speaking tips from TED - Visualized summary
5 Public speaking tips from TED - Visualized summary5 Public speaking tips from TED - Visualized summary
5 Public speaking tips from TED - Visualized summary
 
ChatGPT and the Future of Work - Clark Boyd
ChatGPT and the Future of Work - Clark Boyd ChatGPT and the Future of Work - Clark Boyd
ChatGPT and the Future of Work - Clark Boyd
 
Getting into the tech field. what next
Getting into the tech field. what next Getting into the tech field. what next
Getting into the tech field. what next
 
Google's Just Not That Into You: Understanding Core Updates & Search Intent
Google's Just Not That Into You: Understanding Core Updates & Search IntentGoogle's Just Not That Into You: Understanding Core Updates & Search Intent
Google's Just Not That Into You: Understanding Core Updates & Search Intent
 
How to have difficult conversations
How to have difficult conversations How to have difficult conversations
How to have difficult conversations
 
Introduction to Data Science
Introduction to Data ScienceIntroduction to Data Science
Introduction to Data Science
 
Time Management & Productivity - Best Practices
Time Management & Productivity -  Best PracticesTime Management & Productivity -  Best Practices
Time Management & Productivity - Best Practices
 
The six step guide to practical project management
The six step guide to practical project managementThe six step guide to practical project management
The six step guide to practical project management
 
Beginners Guide to TikTok for Search - Rachel Pearson - We are Tilt __ Bright...
Beginners Guide to TikTok for Search - Rachel Pearson - We are Tilt __ Bright...Beginners Guide to TikTok for Search - Rachel Pearson - We are Tilt __ Bright...
Beginners Guide to TikTok for Search - Rachel Pearson - We are Tilt __ Bright...
 

golintを使おう

  • 2. Copyright © 2009-2015 eureka, inc. All rights reserved. もくじ ・じこしょうかい ・golintについて ・golintをみてみる ・わたしとgolint
  • 3. じこしょうかい - なまえ: Takuma Morikawa - しょぞく: - たんとう:
  • 4. じこしょうかい - なまえ: Takuma Morikawa - しょぞく: - たんとう: (Go, JS, たまにインフラ)
  • 5. じこしょうかい - なまえ: Takuma Morikawa - しょぞく: - たんとう: (Go, JS, たまにインフラ) aws-sdk-goのラッパーとか https://github.com/evalphobia/aws-sdk-go-wrapper
  • 6. じこしょうかい - なまえ: Takuma Morikawa - しょぞく: - たんとう: (Go, JS, たまにインフラ) aws-sdk-goのラッパーとか https://github.com/evalphobia/aws-sdk-go-wrapper
  • 8. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて みなさんつかってますか?
  • 9. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて みなさんつかってますか?
  • 10. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて みなさんつかってます か?
  • 11. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて
  • 12. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて もちろん私は
  • 13. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて 設定していませんでした…orz
  • 14. Copyright © 2009-2015 eureka, inc. All rights reserved. golintについて $ golint ./…
  • 15.
  • 16.
  • 17. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く
  • 18. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く 2) <XXX>でテストコードを書く
  • 19. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く 2) <XXX>でテストコードを書く 3) ʕº̫͡ºʔo00(git push origin dev [ターン!])
  • 20. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く 2) <XXX>でテストコードを書く 3) ʕº̫͡ºʔo00(git push origin dev [ターン!]) 4) CircleCIがエラーを検知
  • 21. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く 2) <XXX>でテストコードを書く 3) ʕº̫͡ºʔo00(git push origin dev [ターン!]) 4) CircleCIがエラーを検知 5) (Slackへのエラー通知でざわつく)
  • 22. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く 2) <XXX>でテストコードを書く 3) ʕº̫͡ºʔo00(git push origin dev [ターン!]) 4) CircleCIがエラーを検知 5) (Slackへのエラー通知でざわつく) 6) (*ノェノ)キャー m9(^Д^)プギャー
  • 23. Copyright © 2009-2015 eureka, inc. All rights reserved. あのころのわたしたち 1) <XXX>でコードを書く 2) <XXX>でテストコードを書く 3) ʕº̫͡ºʔo00(git push origin dev [ターン!]) 4) CircleCIがエラーを検知 5) (Slackへのエラー通知でざわつく) 6) (*ノェノ)キャー m9(^Д^)プギャー < おっけー!
  • 24. Copyright © 2009-2015 eureka, inc. All rights reserved. ある日 ʕº̫͡ºʔo00(あれ、、、)
  • 25. Copyright © 2009-2015 eureka, inc. All rights reserved. ある日 ʕº̫͡ºʔo00(あれ、、、) ʕº̫͡ºʔo00(pre-commitがうまく動いてないぞ…)
  • 26. $ golint ./… 2>&1 | wc -l
  • 27.
  • 28. Copyright © 2009-2015 eureka, inc. All rights reserved. ある日 ʕº̫͡ºʔo00(ちゃんと動作を確認しましょう…) https://gist.github.com/evalphobia/cf79c1911df7a5ae3f00
  • 31. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lint() ← こ の 1 7 の関数内でファイルごと にチェックをかけている
  • 32. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintPackageComment() ← パ ッ ケ ー ジ の コ メ ン ト を チ ェ ッ ク
  • 33. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintPackageComment()
  • 34. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintPackageComment() ↑ パ ッ ケ ー ジ コ メ ン ト が 離 れ て い る
  • 35. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintPackageComment() ↑ パ ッ ケ ー ジ コ メ ン ト が な い
  • 36. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintPackageComment() ↑ パ ッ ケ ー ジ コ メ ン ト の 前 に ス ペ ー ス が あ る
  • 37. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintPackageComment() ↑ コ メ ン ト の 形 式 に 添 っ て い な い
  • 38. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintImports() ← i m p o r t を チ ェ ッ ク
  • 39. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintImports() ↑ ド ッ ト イ ン ポ ー ト 、 ダ メ 、 ゼ ッ タ イ
  • 40. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintBlankImports() ← 無 名 i m p o r t 「 _ 」 を チ ェ ッ ク
  • 41. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintBlankImports() ↑ ド ッ ト イ ン ポ ー ト 、 ダ メ 、 ゼ ッ タ イ↑ 無 名 イ ン ポ ー ト を テ ス ト や m a i n ( ) 以 外 で 使 っ て い る
  • 42. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintExported() ← 大 文 字 で 始 ま る 変 数 や 関 数 を チ ェ ッ ク ( 公 開 状 態 )
  • 43. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintExported() ↑ ド ッ ト イ ン ポ ー ト 、 ダ メ 、 ゼ ッ タ イ ↑ パ ッ ケ ー ジ 名 と 同 じ p r e f i x を 持 っ て い る
  • 44. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintExported() ↑ 公 開 変 数 ( p u b l i c ) は 、 個 別 に 定 義 す る
  • 45. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintExported() ↑ 公 開 し て い る も の に は コ メ ン ト を つ け ま し ょ う
  • 46. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintExported() ↑ コ メ ン ト の 形 式 に 添 っ て い な い
  • 47. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ← 命 名 を チ ェ ッ ク
  • 48. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ パ ッ ケ ー ジ 名 に ア ン ス コ 「 _ 」 は 使 わ な い
  • 49. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ 名 前 に 全 部 大 文 字 を 使 っ ち ゃ あ か ん
  • 50. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ k X X X の よ う な 化 石 命 名 も あ か ん
  • 51. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ 名 前 に も ア ン ス コ は 使 っ ち ゃ あ か ん
  • 52. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↓ ※ i d と か h t t p と か は 大 文 字 小 文 字 を 統 一 す る
  • 53. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintNames() ← 変 数 の 宣 言 を チ ェ ッ ク
  • 54. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintVarDecls() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ 0 値 で 初 期 化 し な く て も い い ん だ よ
  • 55. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintVarDecls() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ 右 辺 が 類 推 可 能 な 値 の 場 合 は 型 宣 言 不 要 だ よ
  • 56. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintElses() ← e l s e ブ ロ ッ ク を チ ェ ッ ク
  • 57. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintElses() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ i f 文 最 後 の e l s e は r e t u r n 返 す な ら い ら な い よ !
  • 58. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintRanges() ← r a n g e を チ ェ ッ ク
  • 59. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintRanges() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ f o r r a n g e 文 で k e y し か 使 わ な い 場 合 は 、 2 番 目 の 変 数 を 「 _ 」 で 捨 て ず に 省 略 し た ほ う が え え
  • 60. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorf() ← e r r o r . N e w を チ ェ ッ ク
  • 61. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorf() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ e r r o r s . N e w ( f m t . S p r i n t f ( ) ) だ っ て ? f m t . E r r o r f ( ) っ て い う の が あ っ て だ な …
  • 62. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrors() ← e r r o r 変 数 を チ ェ ッ ク
  • 63. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrors() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ グ ロ ー バ ル な エ ラ ー 変 数 の p r e f i x は " e r r " に し ま し ょ う
  • 64. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorStrings() ← エ ラ ー 文 言 を チ ェ ッ ク
  • 65. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorStrings() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ エ ラ ー 文 言 を 大 文 字 で 始 め る の は 非 推 奨 ※ ↓ ※ し き い 値 は 0 . 6 な の で 通 常 は 表 示 さ れ な い
  • 66. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorStrings() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ エ ラ ー 文 言 に 句 読 点 を つ け て は い け な い
  • 67. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintReceiverNames() ← メ ソ ッ ド の レ シ ー バ 名 を チ ェ ッ ク
  • 68. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintReceiverNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ レ シ ー バ 変 数 を ア ン ス コ 「 _ 」 に し ち ゃ あ か ん
  • 69. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintReceiverNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ 「 m e 」 「 t h i s 」 「 s e l f 」 も あ か ん よ な あ
  • 70. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintReceiverNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ 同 じ 構 造 体 で レ シ ー バ 変 数 名 は 統 一 し よ う
  • 71. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintIntDec() ← イ ン ク リ メ ン ト ・ デ ク リ メ ン ト を チ ェ ッ ク
  • 72. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintIntDec() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ 「 i + = 1 」 は 「 i + + 」 に し た ほ う が い い よ
  • 73. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintMake() ← m a k e ( ) を チ ェ ッ ク
  • 74. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintMake() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ 空 の 配 列 は m a k e し な い で 宣 言 す る だ け の 方 が い い よ
  • 75. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorReturn() ← r e t u r n 時 の e r r o r 型 を チ ェ ッ ク
  • 76. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintErrorReturn() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ 関 数 で 複 値 を 返 す 場 合 、 e r r o r 型 は 最 後 に し ま し ょ う
  • 77. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintUnexportedReturn() ← 非 公 開 型 の r e t u r n を チ ェ ッ ク
  • 78. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintUnexportedReturn() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ ↑ 公 開 関 数 で 非 公 開 の 構 造 体 を 返 す の は あ か ん !
  • 79. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintTimeNames() ← t i m e の 命 名 を チ ェ ッ ク
  • 80. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go lintTimeNames() ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑↑ t i m e . D u r a t i o n 型 に 時 間 の 単 位 の 名 前 は つ け て は い け な い
  • 81. Copyright © 2009-2015 eureka, inc. All rights reserved. 続きコードはWebで ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ [golintのエラーサンプル] https://github.com/evalphobia/golint-sample
  • 82. Copyright © 2009-2015 eureka, inc. All rights reserved. 続きコードはWebで ↑ コ メ ン ト の 形 式 に 添 っ て い な い ↑ [golintのエラーサンプル] https://github.com/evalphobia/golint-sample [公式にあった…orz] https://github.com/golang/lint/tree/master/testdata
  • 83. Copyright © 2009-2015 eureka, inc. All rights reserved. lint.go しきい値
  • 84. ↑ 「 - m i n _ c o n f i d e n c e 」
  • 85. Copyright © 2009-2015 eureka, inc. All rights reserved. 0.8以下のエラー ↑ パ ッ ケ ー ジ コ メ ン ト が な い ↑ e r r o r s . N e w ( s t r i n g ) の 文 字 列 が 大 文 字 で 始 ま っ て い る
  • 91. thank you : ) Copyright © 2009-2015 eureka, inc. All rights reserved.

Notas do Editor

  1. ・自己紹介
  2. ・自己紹介
  3. ・自己紹介
  4. ・自己紹介
  5. ・自己紹介
  6. ・自己紹介
  7. ・自己紹介
  8. ・自己紹介
  9. ・自己紹介
  10. ・自己紹介
  11. ・自己紹介
  12. ・自己紹介
  13. ・自己紹介
  14. ・自己紹介
  15. ・自己紹介
  16. ・自己紹介
  17. ・自己紹介
  18. ・自己紹介
  19. ・自己紹介
  20. ・自己紹介
  21. ・自己紹介
  22. ・自己紹介
  23. ・自己紹介
  24. ・自己紹介
  25. ・自己紹介
  26. ・自己紹介
  27. ・自己紹介
  28. ・自己紹介
  29. ・自己紹介
  30. ・自己紹介
  31. ・自己紹介
  32. ・自己紹介
  33. ・自己紹介
  34. ・自己紹介
  35. ・自己紹介
  36. ・自己紹介
  37. ・自己紹介
  38. ・自己紹介
  39. ・自己紹介
  40. ・自己紹介
  41. ・自己紹介
  42. ・自己紹介
  43. ・自己紹介
  44. ・自己紹介
  45. ・自己紹介
  46. ・自己紹介
  47. ・自己紹介
  48. ・自己紹介
  49. ・自己紹介
  50. ・自己紹介
  51. ・自己紹介
  52. ・自己紹介
  53. ・自己紹介
  54. ・自己紹介
  55. ・自己紹介
  56. ・自己紹介
  57. ・自己紹介
  58. ・自己紹介
  59. ・自己紹介
  60. ・自己紹介
  61. ・自己紹介
  62. ・自己紹介
  63. ・自己紹介
  64. ・自己紹介
  65. ・自己紹介
  66. ・自己紹介
  67. ・自己紹介
  68. ・自己紹介
  69. ・自己紹介
  70. ・自己紹介
  71. ・自己紹介
  72. ・自己紹介