SQLite Проверка на выполнение запроса

@GENDALF_ISTARI · Регистрация: 20.08.2013

Author24 — интернет-сервис помощи студентам

C#

 //создание таблицы запрос таблицы command_SQL
        public int Sqlite_Create_Table(string command_SQL,SQLiteConnection sqliteconn)
        {
            int flagcreat = 0;
            try
            {
                //подключаемся
                SQLiteCommand command = new SQLiteCommand(sqliteconn);
                //передача запроса
                command.CommandText = command_SQL;
                //выполнение запроса
                command.CommandType = CommandType.Text;
                //получение ошибок
                flagcreat = command.ExecuteNonQuery();
                
            }
            catch 
            {
 
                flagcreat = 0;
            }
 
            return flagcreat;
        }

C#

SQLiteConnection sqliteconn;
string query="create table highscores (name varchar(20), score int)";
 
int error=Sqlite_Create_Table(query, sqliteconn);
 
MessageBox.Show(error.ToString());

возвращает 0
хотя все правильно , должно вернуть 1

Ну и как проверить на правильность запроса
это ExecuteScalar()
или что то другое ?

@OwenGlendower · 02.08.2016, 02:27

GENDALF_ISTARI, метод ExecuteNonQuery возвращает количество строк в таблице которые были затронуты запросом. В приведенном коде нет работы со строками - есть только создание таблицы. Признаком успешного выполнения такого запроса является отсутствие исключения.

P.S. Ловить исключения блоком catch без сохранения информации об исключении плохая практика. В данном случае лучше вообще убрать try/catch.

@GENDALF_ISTARI · 02.08.2016, 12:00 **[ТС]**

ну если уберу
вернет же 0 , а не 1

C#

//создание таблицы запрос таблицы command_SQL
        public int Sqlite_Create_Table(string command_SQL,SQLiteConnection sqliteconn)
        {
            int flagcreat = 0;
                //подключаемся
                SQLiteCommand command = new SQLiteCommand(sqliteconn);
                //передача запроса
                command.CommandText = command_SQL;
                //выполнение запроса
                command.CommandType = CommandType.Text;
                //получение ошибок
                flagcreat = command.ExecuteNonQuery();
                           
 
            return flagcreat;
        }

C#

SQLiteConnection sqliteconn;
string query="create table highscores (name varchar(20), score int)";
 
int error=Sqlite_Create_Table(query, sqliteconn);
 
MessageBox.Show(error.ToString());

@OwenGlendower · 02.08.2016, 12:08

GENDALF_ISTARI, метод и так всегда будет возвращать 0 - с try/catch или без него потому что для запроса CREATE TABLE ... метод ExecuteNonQuery возвращает 0. Он вернет осмысленное число, только для INSERT/UPDATE/DELETE запросов. То есть возвращаемое значение никак не характирует успех или неудачу запроса. Если с запросом что-то не так, то будет сгенерировано исключение. Соответственно метод нужно переписать как-то так:

C#

public bool Sqlite_Create_Table(string command_SQL, SQLiteConnection sqliteconn)
{
    try
    {
        SQLiteCommand command = new SQLiteCommand(command_SQL, sqliteconn);
        command.ExecuteNonQuery();
        return true;
    }
    catch
    {
        return false;
    }
}

@GENDALF_ISTARI · 03.08.2016, 20:05 **[ТС]**

command.ExecuteNonQuery();
так нафик он вообще сдался ?

C#

public bool Sqlite_Create_Table(string command_SQL, SQLiteConnection sqliteconn)
{
    try
    {
        SQLiteCommand command = new SQLiteCommand(command_SQL, sqliteconn);
        //command.ExecuteNonQuery(); //пока бесполезный метод 
        return true;
    }
    catch
    {
        return false;
    }
}

@OwenGlendower · 04.08.2016, 01:18

GENDALF_ISTARI, он выполняет команду.

@GENDALF_ISTARI · 04.08.2016, 14:19 **[ТС]**

он что сохраняет исключение
command.ExecuteNonQuery(); //в command пишет
чтоб потом реагировал
catch
{

}

@OwenGlendower · 04.08.2016, 14:23

GENDALF_ISTARI, он не сохраняет исключение, а генерирует его. Наличие исключения говорит об ошибке, отсутствие - об успехе. Поэтому после ExecuteNonQuery написано return true; - эта строка будет выполнена только если не было исключения. Возьми книгу по C# и изучи главу посвященную обработке исключений.

@pincet · 04.08.2016, 14:25

Сообщение от GENDALF_ISTARI

это ExecuteScalar()

вернет значение первого поля первой строки выборки (со всеми вытекающими, ибо выборка может быть и пустая)

Не по теме:

а вообще, пора бы уже и почитать чего, а не постить "не очень красивый код"™ в надежде, что его весь просмотрят и напишут новый

@GENDALF_ISTARI · 04.08.2016, 17:03 **[ТС]**

Понял

@GENDALF_ISTARI 16 / 33 / 19 Регистрация: 20.08.2013 Сообщений: 740
	04.08.2016, 17:03 [ТС]	10
	Понял 0

@OwenGlendower Администратор 16994 / 13352 / 5211 Регистрация: 17.03.2014 Сообщений: 27,303 Записей в блоге: 1
	02.08.2016, 02:27	2
	GENDALF_ISTARI, метод ExecuteNonQuery возвращает количество строк в таблице которые были затронуты запросом. В приведенном коде нет работы со строками - есть только создание таблицы. Признаком успешного выполнения такого запроса является отсутствие исключения. P.S. Ловить исключения блоком catch без сохранения информации об исключении плохая практика. В данном случае лучше вообще убрать try/catch. 1

@OwenGlendower Администратор 16994 / 13352 / 5211 Регистрация: 17.03.2014 Сообщений: 27,303 Записей в блоге: 1
	04.08.2016, 01:18	6
	GENDALF_ISTARI, он выполняет команду. 1

@GENDALF_ISTARI 16 / 33 / 19 Регистрация: 20.08.2013 Сообщений: 740
	04.08.2016, 14:19 [ТС]	7
	он что сохраняет исключение command.ExecuteNonQuery(); //в command пишет чтоб потом реагировал catch { } 0

@OwenGlendower Администратор 16994 / 13352 / 5211 Регистрация: 17.03.2014 Сообщений: 27,303 Записей в блоге: 1
	04.08.2016, 14:23	8
	GENDALF_ISTARI, он не сохраняет исключение, а генерирует его. Наличие исключения говорит об ошибке, отсутствие - об успехе. Поэтому после ExecuteNonQuery написано `return true;` - эта строка будет выполнена только если не было исключения. Возьми книгу по C# и изучи главу посвященную обработке исключений. 1

@pincet 1643 / 1144 / 171 Регистрация: 23.07.2010 Сообщений: 6,793
	04.08.2016, 14:25	9
	Сообщение от GENDALF_ISTARI это ExecuteScalar() вернет значение первого поля первой строки выборки (со всеми вытекающими, ибо выборка может быть и пустая) Не по теме: а вообще, пора бы уже и почитать чего, а не постить "не очень красивый код"™ в надежде, что его весь просмотрят и напишут новый 1

Опции темы