נראה טוב מאוד,
דבר ראשון ששמתי לב זה שיש דיאלוג ויש גם מניו, יכלנו לשלב בדיאלוג ולהוסיף את העניין של האדמינים כדי שיהיה יותר יפה.. אבל בסדר..
דבר שני שמתי לב כמו שתמיד Ohad וZxoR אומרים לי ;] ולמדתי כבר .. על הקטע של ה elseif, ב ON TEXT אתה עושה המון if וזה מאמץ את הקוד .. זה פשוט לא נכון לעשות את זה כי זה סתם .. עדיף לעשות ELSEIF ואז זה לא בודק בכלל את התנאים האחרים.
עוד משהו זה עצה, במקום לעשות בכל תנאי ב
ON TEXT - %sign $+ Sponsers
לדוגמא, אפשר פשוט לעשות בתחילת הקוד -
tokenize 32 $right($1-,-1)
וזה ה$1- יוגדר בעצם בלי האות הראשונה. ב ON TEXT תעשה פשוט
on *:text:$(%sign $+ *,2):%channel: {
ואז זה יהיה יותר חכם קצר ונוח. בIF אתה פשוט אחרי זה עושה
.. בלי הסימן כי הגדרת את ה $1- בלעדיו.
משהו חשוב לבאגים שתדע ותתקן. אם אתה כותב
if ($1- == %sign $+ ip) || ($1- == %sign $+ server) && (!%flooding) {
זה לא יעבוד טוב וזה יעשה באגים. אתה צריך לעשות
if (($1- == %sign $+ ip) || ($1- == %sign $+ server)) && (!%flooding) {
וככה ה || ייתייחס לשניהם בלבד .. אם לא זה חרא עושה באגים :]
עוד משהו קטן זה שכתבת בכל
.. עדיף לעשות פעם אחת ובתוכו לעשות את כל התנאים לפקודות ..
זהו בסך הכל בוט מצויין.