Use let chains for readability

Author: jplatte

joke_bot/src/main.rs

@@ -161,37 +161,30 @@ async fn create_matrix_session(
 async fn handle_message(
     http_client: &HttpClient,
     matrix_client: &MatrixClient,
-    e: &Raw<AnySyncTimelineEvent>,
+    ev: &Raw<AnySyncTimelineEvent>,
     room_id: OwnedRoomId,
     bot_user_id: &UserId,
 ) -> Result<(), Box<dyn Error>> {
     if let Ok(AnySyncTimelineEvent::MessageLike(AnySyncMessageLikeEvent::RoomMessage(
         SyncMessageLikeEvent::Original(m),
-    ))) = e.deserialize()
+    ))) = ev.deserialize()
+        // workaround because Conduit does not implement filtering
+        && m.sender != bot_user_id
+        && let MessageType::Text(t) = m.content.msgtype
     {
-        // workaround because Conduit does not implement filtering.
-        if m.sender == bot_user_id {
-            return Ok(());
-        }
-
-        if let MessageType::Text(t) = m.content.msgtype {
-            println!("{}:\t{}", m.sender, t.body);
-            if t.body.to_ascii_lowercase().contains("joke") {
-                let joke = match get_joke(http_client).await {
-                    Ok(joke) => joke,
-                    Err(_) => "I thought of a joke... but I just forgot it.".to_owned(),
-                };
-                let joke_content = RoomMessageEventContent::text_plain(joke);
+        println!("{}:\t{}", m.sender, t.body);
+        if t.body.to_ascii_lowercase().contains("joke") {
+            let joke = match get_joke(http_client).await {
+                Ok(joke) => joke,
+                Err(_) => "I thought of a joke... but I just forgot it.".to_owned(),
+            };
+            let joke_content = RoomMessageEventContent::text_plain(joke);
 
-                let txn_id = TransactionId::new();
-                let req = send_message_event::v3::Request::new(
-                    room_id.to_owned(),
-                    txn_id,
-                    &joke_content,
-                )?;
-                // Do nothing if we can't send the message.
-                let _ = matrix_client.send_request(req).await;
-            }
+            let txn_id = TransactionId::new();
+            let req =
+                send_message_event::v3::Request::new(room_id.to_owned(), txn_id, &joke_content)?;
+            // Do nothing if we can't send the message.
+            let _ = matrix_client.send_request(req).await;
         }
     }
 

message_log/src/main.rs

@@ -4,9 +4,8 @@ use ruma::{
     api::client::{filter::FilterDefinition, sync::sync_events},
     assign,
     events::{
-        AnySyncMessageLikeEvent, AnySyncTimelineEvent, OriginalSyncMessageLikeEvent,
-        SyncMessageLikeEvent,
-        room::message::{MessageType, RoomMessageEventContent, TextMessageEventContent},
+        AnySyncMessageLikeEvent, AnySyncTimelineEvent, SyncMessageLikeEvent,
+        room::message::MessageType,
     },
     presence::PresenceState,
 };
@@ -51,21 +50,15 @@ async fn log_messages(
             {
                 // Filter out the text messages
                 if let AnySyncTimelineEvent::MessageLike(AnySyncMessageLikeEvent::RoomMessage(
-                    SyncMessageLikeEvent::Original(OriginalSyncMessageLikeEvent {
-                        content:
-                            RoomMessageEventContent {
-                                msgtype:
-                                    MessageType::Text(TextMessageEventContent {
-                                        body: msg_body, ..
-                                    }),
-                                ..
-                            },
-                        sender,
-                        ..
-                    }),
+                    SyncMessageLikeEvent::Original(m),
                 )) = event
+                    && let MessageType::Text(t) = m.content.msgtype
                 {
-                    println!("{sender} in {room_id}: {msg_body}");
+                    println!(
+                        "{sender} in {room_id}: {msg_body}",
+                        sender = m.sender,
+                        msg_body = t.body
+                    );
                 }
             }
         }