13 votes
Accepted

Simple IRC Bot in C#

you should separate the IRCBot from the Main into its own class with it's own fields/properties you should specify all required ...
t3chb0t's user avatar
  • 44.3k
12 votes

Simple IRC Bot in C#

In addition to t3schb0t's answer I'd also change your parsing logic. Your code is just few lines and it's already too complicate to read. After you moved code into a separate class then it's time for ...
Adriano Repetti's user avatar
12 votes
Accepted

Checking some rules before a Telegram bot replies to a message

If you want to get rid of your nested code then you should make a function and use guard statements. If we simplify your code a bit, we can have: ...
Peilonrayz's user avatar
  • 43.3k
11 votes
Accepted

Stack Exchange Chat Caret Pathfinder

There is still code-golf Here you have some golfed code: function(e){e.style.background = "";} ungolf it: ...
Downgoat's user avatar
  • 902
7 votes

Stack Exchange Chat Caret Pathfinder

One line variable declaration Please avoid declaring variables on the same line: var elements = {}, messages; It's also a great way to royally mess up scope ...
Quill's user avatar
  • 11.9k
7 votes
Accepted

Unix chat server making communication between terminals possible

Portability The code uses pthread which is a POSIX (Portable Operating System Interface) structure. If the code uses POSIX for one thing, it might be better to use <...
pacmaninbw's user avatar
  • 23.8k
6 votes
Accepted

Making Discord great again - TrumpBot

using System; using Discord; using Discord.API; using Discord.ETF; using Discord.Net; using Discord.Legacy; using Discord.Logging; using Discord.Commands; That's ...
Vogel612's user avatar
  • 25.2k
6 votes

Terminal Chat Room

Your code of course can be improved, here are some quick wins: Order imports alphabetically, this helps readability/maintainability, exclude use warnings; and <...
Ashraf Bashir's user avatar
6 votes

Terminal Chat Room

After a quick glance, I think there are a few things you can improve. This is by far not a complete list. Are there any redundancies I can remove? There are no real redundancies that I can see, ...
simbabque's user avatar
  • 993
6 votes
Accepted

Cardshifter chat lobby using vanilla JS

Quick skimming let addChatMessage; Why is this outside your lobbyController? ...
Vogel612's user avatar
  • 25.2k
5 votes
Accepted

Simple chatbot written in Python

Clean and straightforward, very nice. Minor suggestions: If you do from collections import defaultdict ... ...
pjz's user avatar
  • 2,296
5 votes
Accepted

Simple slack bot in python

In general this code looks pretty good. I do have a couple of small pythonism comments: dict.get() returns None if key not ...
Stephen Rauch's user avatar
4 votes
Accepted

TCP/IP multithread chat

Welcome in C#-World :) You chose a nice use-case for the start. So here are my remarks (in random order): 1) Why do you need the count-Variable? It's the same as <...
urmeli's user avatar
  • 156
4 votes
Accepted

Conversational bot architecture

So, I think that this would be a good opportunity for a generalized service pattern. In general fat controllers are a fairly bad idea, in fact fat methods anywhere are generally a fairly bad idea. ...
photoionized's user avatar
4 votes

Text messaging app that allows you to interact with AI

In terms of staying organized, one thing you could keep in your mind is something called the Single Responsibility Principle. You may have heard about this before, but if not, it basically says that ...
Langecrew's user avatar
4 votes

Async Slack bot

It seems okay at first glance. One thing that is kind of bothering me is how you use the beautiful async/await format for some of it then switch back to callback hell for other pieces. Perhaps it's ...
Eric Uldall's user avatar
4 votes

Checking some rules before a Telegram bot replies to a message

Interesting question. We can solve the problem (at least partially) by changing the order of the if statements. I'm assuming you'll want to add more conditions in ...
Daniel's user avatar
  • 4,490
4 votes
Accepted

Interview Take Home Challenge: Chat server

The specs given to you read a little strangely. I don't know have having no external dependencies makes something a *single implementation* or for that matter what ...
Bailey Parker's user avatar
4 votes

Unix chat server making communication between terminals possible

regarding: if(n<0){ printf("Buffer: %s", buffer); error("Reading Failed"); } the call to printf() will ...
user3629249's user avatar
  • 2,798
4 votes
Accepted

Simple Console Chat in Java

Very good implementation, commented and easy to understand. There is no much to improve but I am providing you my suggestions. Configuration Viewer and ...
Marc's user avatar
  • 5,532
3 votes

Primitive Twitch.tv IRC Chat Bot

I'm guessing that Read, Socket, and Initialize are modules you wrote. The functions defined ...
zondo's user avatar
  • 3,787
3 votes
Accepted

Text messaging app that allows you to interact with AI

Be modular, apply best practices So far, this is reasonably good, minus the numerous consecutive empty lines. You have related code in understandable chunks so that's one positive that really lends to ...
Legato's user avatar
  • 9,829
3 votes
Accepted

Internet Relay Chat bot core with plugins system

I have the following points to make on the code: - No logging - No external config file for the settings (Open/Close Principal violation) - Creating resources in the init instead of in a separate ...
C. Harley's user avatar
  • 1,523
3 votes

Java Multi-threaded Chat Room

First of all, it's nice to see at least some comments. It makes understanding your code a lot easier. To the suggestions: Yes, you are right, inheriting from JFrame isn't good. Inheriting from ...
Shryne's user avatar
  • 189
3 votes

Simple multithreaded chat server in Rust

I made some changes to your code (see below). You can check the code for what I've done so far: I added some comments on it so you can compare it with your version. I'll note some important points in ...
Ömer Erden's user avatar
3 votes
Accepted

Chat Viewer working with files

I have some suggestions. MessageViewerProgram Instead of brute forcing all the messages in the directory, I suggest that you list them and iterate on them instead. You can use the method ...
Doi9t's user avatar
  • 3,316
3 votes

A simple chatterbot using Tkinter in Python

cmd3 is not a very good function name. Consider instead something like on_enter. ...
Reinderien's user avatar
2 votes

Badge Oneboxer for Chat.SE

I apologize, but my review isn't going to be very vigorous. Don't check all elements Every time a new chat message is added, you are searching through all the chat messages on the page for messages ...
SirPython's user avatar
  • 13.3k
2 votes

Badge Oneboxer for Chat.SE

N.b. the current GitHub source already has some of these fixed. Firstly, all lines are indented one level more than I would expect (the first comment is also indented in the GitHub source); that ...
ferada's user avatar
  • 11.1k

Only top scored, non community-wiki answers of a minimum length are eligible