[Piercing Blow] Dapper Update

[Boris2105]

"accountId не существует в текущем тексте"

 
public Character GetUpdate(int weapon_primary)
        {
            try
            {
                using (var connection = Factory.conn)
                {
                    string updateQuery = @"UPDATE `characters` SET `WEAPON_PRIMARY` = @weapon_primary WHERE `AccountId` IN(@accountId)";
                    var update = SqlMapper.Execute(connection, updateQuery, new { WEAPON_PRIMARY = weapon_primary, AccountId = accountId });
                }
            }
        }
 
Code: C#

[PROGRAMMATOR]

Ну так откуда ему взяться? Ты только weapon_primary передаёшь.
Да и не понятна мне логика с условием AccountId. Если будет несколько персонажей, то эта пушка установится для всех персонажей учётной записи.

[Boris2105]

AccountId это ид аккаунта персонажа, разве его не надо указывать чтобы он сменил оружие только у одного человека, а не у всех?

[PROGRAMMATOR]

Это понятно, но персонажей может быть от 1 до 9 на одном аккаунте.

[Boris2105]

В игре теперь он меняет и ошибок от сервера нету, но в бд значение не меняется

 
        public Character GetUpdate(int weapon_primary, int id)
        {
            try
            {
                using (var connection = Factory.conn)
                {
                    string updateQuery = @"UPDATE `character` SET `WEAPON_PRIMARY` = @weapon_primary WHERE `ID` IN(@id)";
                    var update = SqlMapper.Execute(connection, updateQuery, new { WEAPON_PRIMARY = weapon_primary, ID = id });
                }
            }
            catch (Exception ex)
            {
                Log.Error(ex.ToString());
            }
            return null;
        }
 
Code: C#

[PROGRAMMATOR]

Название таблицы character или всё же characters?

var update = SqlMapper.Execute(connection, updateQuery, new { WEAPON_PRIMARY = weapon_primary, ID = id });
 
Code: C#

>>>

var count = connection.Execute(updateQuery, new { WEAPON_PRIMARY = weapon_primary, ID = id });
 
Code: C#

[Boris2105]

А использовать в пакете так?

 
Character update = CharacterManager.Instance.GetUpdate(character.WEAPON_PRIMARY, character.ID);
 
Code: C#

[PROGRAMMATOR]

Не знаю, ты же там проектируешь приложение.

Вот скажи мне, что вернётся в Character update и зачем оно вообще там? У тебя уже есть ссылка на character у которого установлено свойство WEAPON_PRIMARY.

И наверное, если юзаешь всё же Dapper а не EF, то наверное есть смысл поставить Dapper.SimpleCRUD раз проблемы с запросами и именованием методов.

[Boris2105]

А разве его не надо никак использовать в пакете PROTOCOL_CHAR_CHANGE_EQUIP_REQ?

[PROGRAMMATOR]

Всё так, но в данный момент метод Update выполняет сохранение основного оружия у персонажа. А должен сохранять все свойства объекта\модели\сущности Character. Как ты себе видишь обновление второстепенного оружия, холодного, метательного или же имени, класса и умений? Под каждую операцию будет свой Update?