SakuraProject / sakura-bot

SakuraBotです。
MIT License
9 stars 3 forks source link

ログが分かりづらい #99

Closed kousakirai closed 1 year ago

kousakirai commented 1 year ago

起動時に流れてるけどprintを使うのかloggingを使うのかどっちなのかわからんから統一してもろて

kousakirai commented 1 year ago

あと各Cogの読み込み完了のログをすべて消して All cogs loaded!なりのログにまとめてもいいんじゃないだろうか

yaakiyu commented 1 year ago

デバッガー側のこと全然考えずに開発進めてたからなぁ...( 僕が作るときはloggingの方にしてるけど、printが残ってるのは古い残骸だな ただ僕logging詳しくないのであんまりやる気が...(( 各Cogでログ出してるのはなんかあった時のためだけど よく考えると要らんかもな どうにかしないと

あと、mysqlのTable already existsみたいなエラーが大量に流れるんだけど 何とかその表示消せないかな

kousakirai commented 1 year ago

Loggingに関しては昔触ってたからQRの問題解決次第やる。 mysqlも、独自のツールをやめてSQLAlchemyを採用してぇなぁって個人的にちょっと思ってたからyaakiyuがGOサイン出してくれるなら作業するよ https://qiita.com/arkuchy/items/75799665acd09520bed2 ←SQLAlchemyについてのqiita

yaakiyu commented 1 year ago

SQLAlchemyね... 触ったことないからあんまわからんけど できるのであればやっていいよ

kousakirai commented 1 year ago

SQLAlchemyの移行作業はぼちぼちやるわ。 とりあえずlogging周りの作業は終わったよ。

tuna2134 commented 1 year ago

SQLAlchemyは細かいところができないので反対する

tuna2134 commented 1 year ago

SQL言語学んでくださいな

kousakirai commented 1 year ago

SQLAlchemyは細かいところができないので反対する

細かいこととは?抽象的な言い方じゃなくて具体的な言い方で反論してくださいな。 仮に生のSQL文を実行できないことを指してるなら https://www.sukerou.com/2019/04/sqlalchemysqlsql.html この記事に方法が乗ってるのでSQLAlchemyを使用することを反対する理由にはならないが。

tuna2134 commented 1 year ago

一つに統一したい無理

tuna2134 commented 1 year ago

さらにif文とか向こうでランダム数字列を作成することもできない

tuna2134 commented 1 year ago

しかも大体の人は今になれているから変える必要がない

tuna2134 commented 1 year ago

あとここで話すことではない

kousakirai commented 1 year ago

さらにif文とか向こうでランダム数字列を作成することもできない

人の文章ぐらいちゃんと読めよ。SQL文実行可能って書いてんだろうが。sql.alchemy.sql.text関数にSQL文流し込んでexecute関数に投げれば実行できる。

しかも大体の人は今になれているから変える必要がない

慣れてる=使い易いではないだろ。はっきり言うけど今のこの状態まじで使いづらい。どこでどんなテーブルが発行されていてどんな処理をしているのかの把握が死ぬほど面倒。SQLAlchemyを使えばmodelを定義してそれに従ってテーブルが作成されるようにできる。

気に入らんのならまずSQLAlchemyでできないことを根拠ありきで否定しろよ。自分の思い込みや正しくない知識で物語るなや。〇〇だから~って言いてぇなら人が納得するような根拠を出せや。議論をする前にシャットアウトするのはただの害悪やぞ。あと人の質問にまともに答えろ。こっちは「細かいことって何?具体的にあげて」って聞いてんのに「一つに統合したい無理」って答えになってねぇんだよ。疑問詞で聞いてんのにYesかNoで答える日本人かよ。

あとここで話すことではない

それはそうやな。issueを別で建てておくことにする。

tuna2134 commented 1 year ago

それはお前だけだろ

tuna2134 commented 1 year ago

使いずらいと思ったのは、全員にアンケート取ってから言えよ

tuna2134 commented 1 year ago

しかもあれ非同期がない時点で終わっている。導入しない方がいい

tuna2134 commented 1 year ago

少なくとも今の状態をわざわざ手間暇かけてかくひつようがない

tuna2134 commented 1 year ago

気に入らんのならまずSQLAlchemyでできないことを根拠ありきで否定しろよ。自分の思い込みや正しくない知識で物語るなや。〇〇だから~って言いてぇなら人が納得するような根拠を出せや。議論をする前にシャットアウトするのはただの害悪やぞ。あと人の質問にまともに答えろ。こっちは「細かいことって何?具体的にあげて」って聞いてんのに「一つに統合したい無理」って答えになってねぇんだよ。疑問詞で聞いてんのにYesかNoで答える日本人かよ。

暴言吐くぐらいならやめたら?

tuna2134 commented 1 year ago

少なくともその発言に不快感しかなかった

tuna2134 commented 1 year ago

別の言い方があるはず

kousakirai commented 1 year ago

使いずらいと思ったのは、全員にアンケート取ってから言えよ

はいはいwわーったから出してやるよ。

しかもあれ非同期がない時点で終わっている。導入しない方がいい

まじでロクに調べてねぇのな。aiomysqlをエンジンに使えば非同期使えるんだよ。

別の言い方があるはず

いい加減てめぇのその態度に苛つきが限界なんだわ。

tuna2134 commented 1 year ago

限界だと思うなら、じゃあやめろよ

kousakirai commented 1 year ago

そうかい。ならやめてやるわ。くそったれが。じゃあな。 yaakiyu俺のコントリまとめて解除しといて。俺がやることはもうないみたいや。

tuna2134 commented 1 year ago

僕の言い方が少し悪かった。 で、僕の考えを聞いて欲しい。

SQLAlchemyに対する考え: ORMLibraryはたしかに楽なのは認める。だけどさらに高度なことをする際にexecuteとか混じって、途中でこれ何のために作ったのかがわからなくなって、最終的にダメなコードができるのが恐れている。しかも従来のやり方から異なるやり方にすると僕としてもすぐには適応できない それと尖った言い方はやめてほしい

何か反論があれば、お願いします。

yaakiyu commented 1 year ago

うーん なんで自分がいない間にこんなことになるんだ... 一方的にやめる話されても残されて困るのこっちなんだが... SQLAlchemyの話は確かにここだとトピ違いだな

104 行こう