新コマンドの実装についての提案 #25

New Issue

konkon-fox · 2023-10-12T23:36:58+09:00

konkon-fox commented

2023-10-12 23:36:58 +09:00

新コマンドの追加案が複数あるようですが、実装方法についての提案です。
投稿時の処理の多くを行う/test/bbs-main.phpに直接書くのではなく、各コマンドごとでファイルを分けて作成し/test/bbs-main.phpでincludeするという形式で統一するのはどうでしょうか。
各コマンドごとの仕様確認・修正が楽になり、致命的なバグが存在した場合等でも容易にメインの投稿プログラムから切り離せるメリットがあります。

新コマンドの追加案が複数あるようですが、実装方法についての提案です。投稿時の処理の多くを行う`/test/bbs-main.php`に直接書くのではなく、各コマンドごとでファイルを分けて作成し`/test/bbs-main.php`で`include`するという形式で統一するのはどうでしょうか。各コマンドごとの仕様確認・修正が楽になり、致命的なバグが存在した場合等でも容易にメインの投稿プログラムから切り離せるメリットがあります。

apple commented

2023-10-13 00:03:59 +09:00

First-time contributor

大賛成です
むしろそういう拡張性をずっと望んでいたので

大賛成ですむしろそういう拡張性をずっと望んでいたので

haruhi commented

2023-10-13 02:37:22 +09:00

First-time contributor

文字の装飾とか末尾もコメントアウトでオンオフできますが本来はbbs-mainから切り離したいですよねー
元々v2はnoreさんが突貫で作ったものなのでそういうところは多いですがー

文字の装飾とか末尾もコメントアウトでオンオフできますが本来はbbs-mainから切り離したいですよねー元々v2はnoreさんが突貫で作ったものなのでそういうところは多いですがー

haruhi commented

2023-10-13 02:44:02 +09:00

First-time contributor

末尾はID9桁目の事でしたー

konkon-fox commented

2023-10-13 22:10:42 +09:00

ブラウザ版の文字装飾とIDの9桁目は管理画面でオンオフ出来る方が望ましいかもしれませんね。
その場合、現在のコードではsetting.jsonに既に存在している項目しか変更出来ませんので、無理やりsetting.jsonに追加するコードを書くか、予め板作成時のテンプレートのsetting.jsonに項目を追加する必要がありそうです。

ブラウザ版の文字装飾とIDの9桁目は管理画面でオンオフ出来る方が望ましいかもしれませんね。その場合、現在のコードではsetting.jsonに既に存在している項目しか変更出来ませんので、無理やりsetting.jsonに追加するコードを書くか、予め板作成時のテンプレートのsetting.jsonに項目を追加する必要がありそうです。

apple commented

2023-10-13 22:55:16 +09:00

First-time contributor

明示的にutf-8で文字列を渡して貰えれば一部のプラグインはFFIなどを利用することでゼロチャンネルのプラグインとほぼ共通のコードで開発できそうですね。
ラッパーこそ必要ですがコア部分はいけるはず。

明示的にutf-8で文字列を渡して貰えれば一部のプラグインはFFIなどを利用することでゼロチャンネルのプラグインとほぼ共通のコードで開発できそうですね。ラッパーこそ必要ですがコア部分はいけるはず。

konkon-fox commented

2023-10-13 23:59:01 +09:00

0ちゃんねるにもコマンドがあるので流用したいところですが私はperlが読めないので少し難しそうですね。phpで1から書いたほうが早そうです。
もちろん私個人の問題なのでperlとphpの橋渡しが出来る方は上手く利用していただきたいですね。
ちなみに!xDyと!chttの草案はほとんど出来ていますので1から作るのはとても大変って訳でもないです。

0ちゃんねるにもコマンドがあるので流用したいところですが私はperlが読めないので少し難しそうですね。phpで1から書いたほうが早そうです。もちろん私個人の問題なのでperlとphpの橋渡しが出来る方は上手く利用していただきたいですね。ちなみに!xDyと!chttの草案はほとんど出来ていますので1から作るのはとても大変って訳でもないです。

konkon-fox commented

2023-10-27 20:55:21 +09:00

2f3bd763fa により新コマンド実装の方針を示せたかと思うのでクローズします。
また、はるひさんがご提案された機能のオプション化については 171bc1e921 にて達成されています。

https://git.3chan.cc/stat2/delightly-v2fork/commit/2f3bd763faeda86bc6f1675442bf54dd71812cdf により新コマンド実装の方針を示せたかと思うのでクローズします。また、はるひさんがご提案された機能のオプション化については https://git.3chan.cc/stat2/delightly-v2fork/commit/171bc1e921a2f3b38e8eb9433584953a9a3963fc にて達成されています。

konkon-fox closed this issue

2023-10-27 20:55:21 +09:00

Sign in to join this conversation.