Как организовать бизнес-логику на сервере?

Question

[Clifford]

На самом деле мой вопрос уже был, но там не дали внятного ответа. Архитектура асинхронного сервера C#
Меня скорей интересует, как организовать прослойку между application server и логикой, ну и реализация самой логики.
Особенно интересно узнать, как обработка данных происходит, допустим, в больших мморпг. Ведь в таких играх нужно перерабатывать кучи данных(координаты, предметы итд) Как сервер справляется с нагрузкой?

Ладно, вернусь к своим "баранам". Сейчас у меня реализована прослойка и логика, мягко говоря, не очень.
Если в краце, то клиент присылает json-сообщение на сервер, сервер выдергивает тип(что совершил пользователь) из этого сообщения, этот тип летит в функцию, где происходит поиск по словарю, ну а когда ключ будет найден выполняется нужная функция. Объяснил не очень, проще показать будет в коде.

// key - ключ\тип действия пользователя;   
// player - экземпляр игрока 
private void messageHnd(String key, Client player)
        {
            foreach (String i in Settings.types.Keys) // types - это словарь вида <String, delegate> 
            {
                if (i == key)
                {
                    Settings.types[key](data, player);
                }
            }
        }

Честно говоря, меня не очень радует процедурное программирование на C# + этот способ больше подходит для Python'а, нежели шарпа. Поэтому надеюсь, что вы направите меня на правильный путь.

PS. Никто не знает, как проверять занят ли SocketAsyncEventArgs или нет. Бывали случаи, что сообщение не отправлялось, так как процесс был занят.

Comments

[Алексей Кулаков]

Зачем вы делаете перебор словаря? сейчас поясню как ваш код работает. foreach принимает объект реализующий IEnumerable. Этот объект не может ничего делать кроме как переходить и переходить к следующему элементу пока есть следующий. таким образом вы перебераете все элемены. и потом найдя его выполняете поиск в словаре по ключу. и потом продолжаете перебор значений пока перечисление не кончится тем самым вы убили всю производительность поиска по словарю.

не делайте так больше. делайте вот так. предположим есть словарь Dictionary тогда

int result:
dictionary.TryGetValue(key, out result)
if (result!=null) {
  /*выполняем нужные действия*/
}

таким способом вы улучшите производительность конкретно этого куска кода и аналогичных ему если они есть.

если нужно узнать есть ли ключ в словаре то используйте метод ContainsKey.

Также не следует перебирать элементы коллекции HashSet. Вообще воздержитесь от лишних переборов. и запомните, что Linq-методы такие как First, FirstOrDefault, Last, LastOrDefault, Count(не путать со свойством), и некоторые другие тоже делают перебор значений коллекции и проверяют на совпадение с неким предикатом, так как source у них также IEnumerable

[Алексей Кулаков]

Поправочка, в том коде что я привел ошибка - int не может быт null, так что там должен быть некий ссылочный тип в виде значения. но суть та же. не нужно перебирать словарь чтобы узнать есть ли по данному ключу значение.

[Clifford]

Большое спасибо за пояснение, на таких мелких деталях и потону...

[Алексей Кулаков]

Если не читали, то прочитайте Рихтера "C# via CLR". там многие тонкости описаны, которые могут повлиять на производительность. да и вообще рассказывают про то как та или иная конструкция работает. Сам много почерпнул из этой книги.

Answer 1

[Кирилл]

почитайте, мб чтонить вам покажется полезным:
habrahabr.ru/company/mailru/blog/220359
habrahabr.ru/company/mailru/blog/182088