Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

コメントを書けるようにする #18

Closed
YuseiIto opened this issue Feb 25, 2021 · 9 comments · Fixed by #19
Closed

コメントを書けるようにする #18

YuseiIto opened this issue Feb 25, 2021 · 9 comments · Fixed by #19

Comments

@YuseiIto
Copy link
Member

YuseiIto commented Feb 25, 2021

#17 をやるためにコメントの書き方を決めたい。

私としては CのANTLR実装 を参考にして 

/* Block comment like this */

// This form of simple line comment

でコメントできるようにするのがわかりやすいと思う。
他に案はありますか

@frodo821
Copy link
Member

frodo821 commented Feb 25, 2021

@YuseiIto 私的には複行コメントなし、単行コメントは # 始まりが慣れてるので好き。
複行コメントなしの理由は2つあって、複行コメントを含むブロックをコメントアウトするのがだるいのと、コメントのネストは好きじゃない

@YuseiIto
Copy link
Member Author

そうかー私はデバッグしている時に雑にコメントアウトして開発できるので好きなんだけどな、block comment.

@frodo821
Copy link
Member

うーん、悩ましいなぁ……。実装の手間的な問題はまぁ置いておくとしても、ブロックコメントをありがたく思ったことがないのよね

@frodo821
Copy link
Member

frodo821 commented Feb 25, 2021

@YuseiIto

COMMENT: '#' ~[\n\r]*;

単行コメントはこれでいい? //にこだわりがあるならそれでもいいけど

@YuseiIto
Copy link
Member Author

実装の手間的な問題を置いておくのはどうなん?笑 使う言語にするならそこ大事な気が 👀

あと、複業コメントはライセンスとかに役立つというのもあると思っている。

@YuseiIto
Copy link
Member Author

YuseiIto commented Feb 25, 2021

単行コメントの記号に特にこだわりはないけど、複行コメントの記号次第かな...

まあcommitのコードに残ったらただのbad practiceなんだけど、自分でデバッグする時には これ を多用する(&& Dynastyでもブロックコメント入れるならしたい) ので、複行コメントの記号次第かな...と思っている。

@frodo821
Copy link
Member

frodo821 commented Feb 25, 2021

@YuseiIto うーん……、その目的ならコンパイル時評価のif文導入する方がよくない……?
申し訳ないんだけど、複行コメントで幸せになれたプロジェクトを私は知らない……

@frodo821
Copy link
Member

frodo821 commented Feb 25, 2021

@YuseiIto
とりあえずブロックコメントは仕様に入れることにします。

# 単行コメント
#-
複行コメント
-#

でどうですか?

この文字の組み合わせにした意図としては、/*が演算子で予約済みで、ユーザー定義演算子導入時に処理の分岐が面倒なので、演算子に使っていない文字にしたかったというのがあります。#が(将来的にも)演算子に使えない文字である以上、-#は構文解析的に演算子ではないのでこの点はクリア。

@YuseiIto
Copy link
Member Author

いいですね、それでいこう

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants