Ошибка при преобразовании типа данных varchar к int

@Яна44 · Регистрация: 23.01.2013

Есть 3 таблицы:

Покупатели - customer(id,name,city,phone,rating)
поставщики - supplier(id,name,city,phone)
договор - contract(id,cust_id,sup_id,date,sum)

Пытаюсь сделать приложение для работы с БД. На одной из форм для таблицы contract использую компонент DBLookupcomboboxEh. В нем появляется список покупателей, выбираю одну из фамилий, а в бд нужно занести его id. Аналогично с поставщиками. Проблема состоит в том, что при добавление новой записи(заключении нового договора) появляется такая ошибка: Ошибка при преобразовании типа данных varchar к int. Как я понимаю в DBLookupcomboboxEh отображается фамилия (тип nvarchar), а id (int). Подскажите пожалуйста как исправить эту ошибку.

Параметры DBLookupcomboboxEh вроде настроены правильно:
Для покупателей:
DataField-cust_id
DataSource-DataModule2.ContractDataSource
KeyField-id
ListField-name
ListSource-DataModule2.CustomerDataSource

C поставщиками аналогично.

Код (для добавления записи):

Delphi

procedure TForm5.btn1Click(Sender: TObject);
var cust_id, sup_id, date, sum: string;
 
begin
   cust_id:=QuotedStr(unit5.form5.dblkpcmbxhcust_id.text);
   sup_id:=QuotedStr(unit5.form5.dblkpcmbxhsup_id.text);
   dbdth2.Text:= DateToStr(dtp1.Date);
   date:=QuotedStr(unit5.form5.dbdth2.text);
   sum:=QuotedStr(unit5.form5.dbdth1.text);
 
   with Unit2.DataModule2.qry  do
   begin
     Close;
     SQL.Clear;
     SQL.Add('exec add_contract' +cust_id+','+sup_id+','+date+','+sum);
     Execsql;
     Unit2.DataModule2.ContractQuery.Active:=False;
     Unit2.DataModule2.ContractQuery.Active:=true;
   end;
end;

@rdama · 27.03.2013, 01:24

Вместо

Delphi

 
..
var cust_id, sup_id, date, sum: string;
..
   cust_id:=QuotedStr(unit5.form5.dblkpcmbxhcust_id.text);
   sup_id:=QuotedStr(unit5.form5.dblkpcmbxhsup_id.text);
..
SQL.Add('exec add_contract' +cust_id+','+sup_id+','+date+','+sum);
..

надо

Delphi

   
..
var date, sum: string;
     cust_id, sup_id : integer;
..   
   cust_id:=unit5.form5.dblkpcmbxhcust_id.KeyValue;
   sup_id:=unit5.form5.dblkpcmbxhsup_id.KeyValue;
..
SQL.Add('exec add_contract' +IntToStr(cust_id)+','+IntToStr(sup_id)+','+date+','+sum);
..

А так вообще делать нельзя

Delphi

1	dbdth2.Text:= DateToStr(dtp1.Date);

Меняем региональные настройки и к вашей программе приходит белая пушистая лисичка.
Да и вообще нужно использовать параметризированный запрос.
Вместо

Delphi

procedure TForm5.btn1Click(Sender: TObject);
var cust_id, sup_id, date, sum: string;
 
begin
   cust_id:=QuotedStr(unit5.form5.dblkpcmbxhcust_id.text);
   sup_id:=QuotedStr(unit5.form5.dblkpcmbxhsup_id.text);
   dbdth2.Text:= DateToStr(dtp1.Date);
   date:=QuotedStr(unit5.form5.dbdth2.text);
   sum:=QuotedStr(unit5.form5.dbdth1.text);
 
   with Unit2.DataModule2.qry  do
   begin
     Close;
     SQL.Clear;
     SQL.Add('exec add_contract' +cust_id+','+sup_id+','+date+','+sum);
     Execsql;
     Unit2.DataModule2.ContractQuery.Active:=False;
     Unit2.DataModule2.ContractQuery.Active:=true;
   end;
end;

Стоит написать так

Delphi

procedure TForm5.btn1Click(Sender: TObject);
begin
   with Unit2.DataModule2.qry  do
   begin
     Close;
     SQL.Clear;
     SQL.Add('exec add_contract( :cust_id , :sup_id , :с_date , :c_sum )';
     Parameters.ParamByName('cust_id').Value := unit5.form5.dblkpcmbxhcust_id.KeyValue;
     Parameters.ParamByName('sup_id').Value := unit5.form5.dblkpcmbxhsup.KeyValue;
     Parameters.ParamByName('с_date').Value := dtp1.Date;
     Parameters.ParamByName('c_sum ').Value := StrToFloat(unit5.form5.dbdth1.text);
     Execsql;
     Unit2.DataModule2.ContractQuery.Active:=False;
     Unit2.DataModule2.ContractQuery.Active:=true;
   end;
end;

здесь параметры date и sum заменены на c_date и c_sum т.к. date и sum это зарезервированные слова.
Избегайте названий полей и переменных зарезервированными словами т.к. это не минуемо приведет к ошибкам.
В этой строке не ясно в каком виде вы храните суммы но скорее всего необходимо сделать так как я написал.

Delphi

1	Parameters.ParamByName('c_sum ').Value := StrToFloat(unit5.form5.dbdth1.text);

Вряд ли вы храните суммы в текстовых полях. Хотя можете использовать целочисленные. Кто вас знает.
Так же перед выполнением

Delphi

1	StrToFloat(unit5.form5.dbdth1.text)

стоит проверить какой используется в системе разделитель целой и дробной частей. Т.к. ввод осуществляется пользователем и может возникнуть ситуация, что поставили точку вместо запятой и наоборот.
Можно конечно плеваться ошибками о невозможности преобразования. Но лучше сделать замену символа на тот что применяется в системе.

@Яна44 · 27.03.2013, 18:22 **[ТС]**

rdama, спасибо большое, сделала как вы сказали:

Delphi

procedure TForm5.btn1Click(Sender: TObject);
begin
   with Unit2.DataModule2.qry  do
   begin
     Close;
     SQL.Clear;
     SQL.Add('exec add_contract( :cust_id , :sup_id , :с_date , :c_sum )';
     Parameters.ParamByName('cust_id').Value := unit5.form5.dblkpcmbxhcust_id.KeyValue;
     Parameters.ParamByName('sup_id').Value := unit5.form5.dblkpcmbxhsup.KeyValue;
     Parameters.ParamByName('с_date').Value := dtp1.Date;
     Parameters.ParamByName('c_sum ').Value := StrToFloat(unit5.form5.dbdth1.text);
     Execsql;
     Unit2.DataModule2.ContractQuery.Active:=False;
     Unit2.DataModule2.ContractQuery.Active:=true;
   end;
end;

но теперь выскакивает ошибка: qry: Parameter 'c_sum' not found
Не знаю как исправить! У поля sum тип данных real.
В системе используется разделитель "." Как сделать замену символа, на тот что применяется в системе?

Пробовала и вот этот код:

Delphi

procedure TForm5.btn1Click(Sender: TObject);
var date, sum: string;
    cust_id, sup_id:Integer;
 
begin
   cust_id:=unit5.form5.dblkpcmbxhcust_id.KeyValue;
   sup_id:=unit5.form5.dblkpcmbxhsup_id.KeyValue;
   dbdth2.Text:= DateToStr(dtp1.Date);
   date:=QuotedStr(unit5.form5.dbdth2.text);
   sum:=QuotedStr(unit5.form5.dbdth1.text);
 
   with Unit2.DataModule2.qry  do
   begin
     Close;
     SQL.Clear;
     SQL.Add('exec add_contract' +IntToStr(cust_id)+','+IntToStr(sup_id)+','+date+','+sum);
     Execsql;
     Unit2.DataModule2.ContractQuery.Active:=False;
     Unit2.DataModule2.ContractQuery.Active:=true;
   end;
end;

Но там тоже ошибка: неправильный синтаксис около конструкции ","
Помогите пожалуйста!

@Яна44 · 28.03.2013, 22:52 **[ТС]**

Ну что...никто не поможет???

@kwinto · 28.03.2013, 23:20

Delphi

1	Parameters.ParamByName('c_sum ').Value

Явно лишний пробел в указании параметра 'c_sum '.

@Яна44 · 28.03.2013, 23:43 **[ТС]**

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

Delphi

1	Parameters.ParamByName('c_sum ').Value

Явно лишний пробел в указании параметра 'c_sum '.

Спасибо, помогло, но теперь новая ошибка: Ошибка при преобразовании типа данных nvarchar к date
Не подскажите,что не так?

Добавлено через 9 минут
Получилось! Заменила строку

Сообщение от Яна44

Parameters.ParamByName('с_date').Value := dtp1.Date;

на
Parameters.ParamByName('c_date').Value := DateToStr(unit5.Form5.dtp1.Date);
Вроде работает!

@kwinto · 28.03.2013, 23:46

замените переменную date: string; скажем на dt: string;
У Вас цепляется ф-ция

Delphi

1	function Date: TDateTime;

@Яна44 2 / 2 / 1 Регистрация: 23.01.2013 Сообщений: 52
	28.03.2013, 22:52 [ТС]	4
	Ну что...никто не поможет??? 0