挙動が怪しい

[t4t5u0]

processing-slackbot/plugins/func.py

Line 65 in a2df105

else : # デフォルトではpngでupload

において、else ということは、適当なオプション例えば --svg などを指定した場合に、pngを返す。これは仕様なのか

[t4t5u0]

processing-slackbot/plugins/func.py

Line 44 in a2df105

else : # デフォルトではpngで返す

ここもそう。多分デフォルトの保証がされていない

[kota-shiokara]

おっしゃる通り。現在のコマンドが!outputの後にスペース入れて文字列というのを正規表現使ってとってきてるので、--gif以外の文字列は全部pngで返す仕様になっている。
しかも@listen_to('!output (.*)')でなので、!outputの後にスペース入れて文字列を入力しないとoutputメソッドが動かない。ここは要改善かと考えている。

[t4t5u0]

気が向いたら直すわ。一応正規表現も使ったことあるので

[t4t5u0]

.* じゃなくて *じゃだめなの？

(|[--(png|gif)]) みたいな感じで取ってくるとか

[kota-shiokara]

参考にしたサイトが.*使ってただけなので特にこだわりはない
(|[--(png|gif)]) これでちょっと挙動確認してみる

[t4t5u0]

* でとって、.split() した方がよさそう。--gif のあとに int を受け取るのが楽になるので

[kota-shiokara]

splitでとることにした。それと.じゃなくての場合取ってくることができなかった。

[t4t5u0]

.*じゃなくての場合

とは

[kota-shiokara]

うわ、記号のせいで変な感じになってる
「.*」じゃなくて「.」の場合ってことです

[t4t5u0]

.の場合うまく取れないのは自明(任意の一文字を表す正規表現なので)

正規表現ちょっと勉強したほうがいいかも

ちなみに (|--[png|gif]) はうまくいった？

[kota-shiokara]

あ、ごめん「*」の時でもあった
確かに正規表現は勉強しといた方がいいですよね...
(|--[png|gif])は上手くいかなかった、「--」しか取れなかった

[t4t5u0]

((--(png|gif))| )
これでいけんか。多分なおした

[kota-shiokara]

--pngもしくは付け足して実行した場合

failed to handle message !output --png
void draw(){
background(255);
} with plugin "output"
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/slackbot/dispatcher.py", line 55, in _dispatch_msg_handler
    func(Message(self._client, msg), *args)
TypeError: output() takes 2 positional arguments but 4 were given

と出た。ちなみにオプションなしや、「!output 」ってスペースつけた場合は無視された

[t4t5u0]

エラーだけ見せられてもコード見てないからわかんない…

関数に引数を多く渡していると怒られてるのでなんとかしてあげてください(エラー文からわかる情報)

無難に全部取るのが楽かも

[kota-shiokara]

そだね、多分これが一番手っ取り早い気がします

processing-slackbot/plugins/func.py

Lines 23 to 25 in 46b438f

	@listen_to('!output (.*)')
	def output(message, arg): # argはオプション
	args = arg.split() # オプションをリストで保持