มีวิธีที่ชาญฉลาดกว่านี้นอกเหนือจากการใช้คำสั่งหรือการสลับสายยาว ๆ หรือไม่?

ฉันกำลังใช้บอต IRC ที่ได้รับข้อความและฉันกำลังตรวจสอบข้อความนั้นเพื่อพิจารณาว่าจะเรียกใช้ฟังก์ชันใด มีวิธีที่ฉลาดกว่านี้ในการทำเช่นนี้? ดูเหมือนว่ามันจะออกจากมือได้อย่างรวดเร็วหลังจากที่ฉันได้รับคำสั่งถึง 20 คำสั่ง

บางทีอาจมีวิธีที่ดีกว่าที่จะสรุปสิ่งนี้ใช่ไหม

 public void onMessage(String channel, String sender, String login, String hostname, String message){

        if (message.equalsIgnoreCase(".np")){
//            TODO: Use Last.fm API to find the now playing
        } else if (message.toLowerCase().startsWith(".register")) {
                cmd.registerLastNick(channel, sender, message);
        } else if (message.toLowerCase().startsWith("give us a countdown")) {
                cmd.countdown(channel, message);
        } else if (message.toLowerCase().startsWith("remember am routine")) {
                cmd.updateAmRoutine(channel, message, sender);
        }
    }

ใช้ตารางการจัดส่ง นี่คือตารางที่มีคู่ ("ส่วนของข้อความ", pointer-to-function) โปรแกรมเลือกจ่ายงานจะมีลักษณะเช่นนี้ (ในรหัสหลอก):

for each (row in dispatchTable)
{
    if(message.toLowerCase().startsWith(row.messagePart))
    {
         row.theFunction(message);
         break;
    }
}

( equalsIgnoreCaseสามารถจัดการเป็นกรณีพิเศษที่ไหนสักแห่งก่อนหรือถ้าคุณมีการทดสอบเหล่านั้นจำนวนมากพร้อมกับตารางการจัดส่งที่สอง)

แน่นอนสิ่งที่pointer-to-functionต้องมีลักษณะขึ้นอยู่กับภาษาโปรแกรมของคุณ นี่คือตัวอย่างใน C หรือ C ++ ใน Java หรือ C # คุณอาจใช้แลมบ์ดานิพจน์เพื่อวัตถุประสงค์นั้นหรือจำลอง "ตัวชี้ไปยังฟังก์ชัน" โดยใช้รูปแบบคำสั่ง หนังสือออนไลน์ฟรี " Higher Order Perl " มีบทที่สมบูรณ์เกี่ยวกับตารางการจัดส่งโดยใช้ Perl

ฉันอาจทำสิ่งนี้:

public interface Command {
  boolean matches(String message);

  void execute(String channel, String sender, String login,
               String hostname, String message);
}

จากนั้นคุณสามารถให้ทุกคำสั่งใช้อินเทอร์เฟซนี้และคืนค่าจริงเมื่อตรงกับข้อความ

List<Command> activeCommands = new ArrayList<>();
activeCommands.add(new LastFMCommand());
activeCommands.add(new RegisterLastNickCommand());
// etc.

for (Command command : activeCommands) {
    if (command.matches(message)) {
        command.execute(channel, sender, login, hostname, message);
        break; // handle the first matching command only
    }
}

คุณกำลังใช้ Java - ทำให้มันสวยงาม ;-)

ฉันอาจใช้คำอธิบายประกอบ:

1. สร้างคำอธิบายประกอบวิธีที่กำหนดเอง

   @IRCCommand( String command, boolean perfectmatch = false )

2. เพิ่มคำอธิบายประกอบให้กับวิธีการที่เกี่ยวข้องทั้งหมดในคลาสเช่น

   @IRCCommand( command = ".np", perfectmatch = true )
   doNP( ... )

3. ใน Constructor ของคุณให้ใช้ Reflections เพื่อสร้าง HashMap of Methods จากวิธีการทำหมายเหตุประกอบทั้งหมดในคลาสของคุณ:

   ...
   for (Method m : getDeclaredMethods()) {
   if ( isAnnotationPresent... ) {
       commandList.put(m.getAnnotation(...), m);
       ...

4. ในonMessageวิธีการของคุณเพียงแค่วนซ้ำcommandListพยายามที่จะจับคู่สตริงในแต่ละคนและเรียกmethod.invoke()ว่ามันเหมาะกับ

   for ( @IRCCommand a : commanMap.keyList() ) {
       if ( cmd.equalsIgnoreCase( a.command )
            || ( cmd.startsWith( a.command ) && !a.perfectMatch ) {
           commandMap.get( a ).invoke( this, cmd );

ถ้าคุณนิยามอินเตอร์เฟสให้บอกว่าIChatBehaviourมีวิธีใดที่เรียกว่าExecuteซึ่งใช้ใน a messageและcmdวัตถุ:

public Interface IChatBehaviour
{
    public void execute(String message, CMD cmd);
}

ในรหัสของคุณคุณจะใช้อินเทอร์เฟซนี้และกำหนดพฤติกรรมที่คุณต้องการ:

public class RegisterLastNick implements IChatBehaviour
{
    public void execute(String message, CMD cmd)
    {
        if (message.toLowerCase().startsWith(".register"))
        {
            cmd.registerLastNick(channel, sender, message);
        }
    }
}

และอื่น ๆ สำหรับส่วนที่เหลือ

ในชั้นเรียนหลักของคุณคุณจะมีรายการพฤติกรรม ( List<IChatBehaviour>) ซึ่งบอต IRC ของคุณใช้ จากนั้นคุณสามารถแทนที่ifงบของคุณเป็นดังนี้:

for(IChatBehaviour behaviour : this.behaviours)
{
    behaviour.execute(message, cmd);
}

ด้านบนควรลดจำนวนรหัสที่คุณมี วิธีการข้างต้นจะช่วยให้คุณสามารถจัดหาพฤติกรรมเพิ่มเติมให้กับคลาสบอตของคุณโดยไม่ต้องแก้ไขคลาสบอทเอง (ตามที่Strategy Design Pattern)

หากคุณต้องการให้มีพฤติกรรมที่จะทำการยิงเพียงครั้งเดียวในแต่ละครั้งคุณสามารถเปลี่ยนลายเซ็นของexecuteวิธีการที่จะให้ผลtrue(พฤติกรรมที่มีการยิง) หรือfalse(พฤติกรรมที่ไม่ได้ยิง) และแทนที่ห่วงด้านบนด้วยสิ่งนี้:

for(IChatBehaviour behaviour : this.behaviours)
{
    if(behaviour.execute(message, cmd))
    { 
         break;
    }
}

ข้างต้นจะน่าเบื่อมากขึ้นที่จะใช้และเริ่มต้นเนื่องจากคุณต้องการสร้างและส่งคลาสพิเศษทั้งหมดอย่างไรก็ตามมันควรทำให้บอทของคุณสามารถขยายและแก้ไขได้อย่างง่ายดายเนื่องจากคลาสพฤติกรรมทั้งหมดของคุณจะถูกห่อหุ้มและหวังว่าจะเป็นอิสระจากกันและกัน

"ฉลาด" สามารถ (อย่างน้อย) สามสิ่ง:

ประสิทธิภาพที่สูงขึ้น

ข้อเสนอแนะตารางการจัดส่ง (และรายการเทียบเท่า) เป็นสิ่งที่ดี ตารางดังกล่าวถูกเรียกว่า "CADET" ในหลายปีที่ผ่านมาสำหรับ "ไม่สามารถเพิ่มได้หรือไม่ลอง" อย่างไรก็ตามพิจารณาความคิดเห็นเพื่อช่วยผู้ดูแลมือใหม่ในการจัดการตารางดังกล่าว

การบำรุงรักษา

"ทำให้สวย" ไม่ใช่คำเตือนที่ไม่ใช้งาน

และมักถูกมองข้าม ...

ความยืดหยุ่น

การใช้ toLowerCase มีข้อผิดพลาดในการที่ข้อความบางภาษาในบางภาษาจะต้องได้รับการปรับโครงสร้างที่เจ็บปวดเมื่อมีการเปลี่ยนแปลงระหว่าง magiscule และ miniscule น่าเสียดายที่มีข้อผิดพลาดเดียวกันสำหรับ toUpperCase เพิ่งทราบ

คุณสามารถให้ทุกคำสั่งใช้อินเตอร์เฟสเดียวกันได้ จากนั้นโปรแกรมแยกวิเคราะห์ข้อความสามารถส่งคืนคำสั่งที่เหมาะสมซึ่งคุณจะดำเนินการเท่านั้น

public interface Command {
    public void execute(String channel, String message, String sender) throws Exception;
}

public class MessageParser {
    public Command parseCommandFromMessage(String message) {
        // TODO Put your if/switch or something more clever here
        // e.g. return new CountdownCommand();
    }
}

public class Whatever {
    public void onMessage(String channel, String sender, String login, String hostname, String message) {
        Command c = new MessageParser().parseCommandFromMessage(message);
        c.execute(channel, message, sender);
    }
}

ดูเหมือนรหัสมากขึ้น ใช่คุณยังคงต้องแยกวิเคราะห์ข้อความเพื่อที่จะรู้ว่าคำสั่งใดที่จะดำเนินการ แต่ตอนนี้มันอยู่ในจุดที่กำหนดไว้อย่างถูกต้อง มันอาจถูกนำมาใช้ซ้ำในที่อื่น (คุณอาจต้องการฉีด MessageParser แต่นั่นเป็นเรื่องอื่นนอกจากนี้รูปแบบ Flyweightอาจเป็นแนวคิดที่ดีสำหรับคำสั่งขึ้นอยู่กับจำนวนที่คุณคาดว่าจะสร้าง)

สิ่งที่ฉันจะทำคือ:

1. จัดกลุ่มคำสั่งที่คุณมีไว้เป็นกลุ่ม (คุณมีอย่างน้อย 20 ตอนนี้)
2. ในระดับแรกจัดหมวดหมู่ตามกลุ่มดังนั้นคุณมีคำสั่งที่เกี่ยวข้องกับชื่อผู้ใช้คำสั่งเพลงคำสั่งนับเป็นต้น
3. จากนั้นคุณไปในวิธีการของแต่ละกลุ่มคราวนี้คุณจะได้รับคำสั่งดั้งเดิม

สิ่งนี้จะทำให้จัดการได้ง่ายขึ้น ได้ประโยชน์มากขึ้นเมื่อจำนวน 'อื่นถ้า' เติบโตมากเกินไป

แน่นอนว่าบางครั้งการมี 'ถ้าอย่างอื่น' เหล่านี้จะไม่ใช่ปัญหาใหญ่ ฉันไม่คิดว่า 20 เป็นสิ่งที่ไม่ดี