Передать во второй запрос значение id, сгенерированное при выполнении первого запроса insert

@yafidik · Регистрация: 13.10.2014

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

две таблицы ... 1 таблицаLOGIN имеет логины и пароли + Id с типом uniqueidentifier и имеет(newid).
id вторая TECHER имена фамилии итд... итп... (и так как соединена с первой таблицей) login_id...
и через форму есть регистрация но при регистрации 1 учителя с логином паролем фамилией имением итд.. нужно чтобы в login_id(в таблице TECHER) добавлялось id (из таблице LOGIN) чтобы соединялось...

C#

   public partial class reg : Form
    {
        SqlConnection con = new SqlConnection(@"Data Source=(LocalDB)\v11.0;AttachDbFilename=D:\Program Files\test c#\1\WindowsFormsApplication1\WindowsFormsApplication1\Data.mdf;Integrated Security=True;Connect Timeout=30;");
        public reg()
        {
            InitializeComponent();
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            con.Open();
            SqlDataAdapter sda = new SqlDataAdapter("insert into LOGIN (login,pasword)values('" + textBox1.Text + "','" + textBox2.Text + "')", con);
            sda.SelectCommand.ExecuteNonQuery();
            con.Close();
            
            con.Open();
            SqlDataAdapter sdaa = new SqlDataAdapter("insert into TECHER (Id,f_t,n_t)values('?????????','" + textBox3.Text + "','" + textBox4.Text + "')", con);
            sdaa.SelectCommand.ExecuteNonQuery();
            con.Close();
 
            MessageBox.Show("Регестрация прошла успешно");
            this.Hide();
            authentication aa = new authentication();
            aa.Show();
        }
       public static void main(string[] args)
       {
           Application.Run(new reg());
       }
 
       private void reg_Load(object sender, EventArgs e)
       {
 
       }
    }

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

может заодно скажите как уменьшить код

C#

     con.Open();
            SqlDataAdapter sda = new SqlDataAdapter("insert into LOGIN (login,pasword)values('" + textBox1.Text + "','" + textBox2.Text + "')", con);
            sda.SelectCommand.ExecuteNonQuery();
            con.Close();
            
            con.Open();
            SqlDataAdapter sdaa = new SqlDataAdapter("insert into TECHER (Id,f_t,n_t)values('into LOGIN (Id)','" + textBox3.Text + "','" + textBox4.Text + "')", con);
            sdaa.SelectCommand.ExecuteNonQuery();
            con.Close();

надеюсь всё правильно ...

Добавлено через 18 минут
скорее всего сокращать в этом случае не надо между ними будет присваиваться переменной id .. но если не трудно всё равно напишите интересно как можно сократить ...

Добавлено через 1 минуту
и простите за название не забыл дописать ....

@Даценд · 19.08.2016, 22:48

yafidik,
При помощи промежуточного запроса select получайте значение id новой записи и передавайте во второй запрос.
А лучше объедините две таблицы в одну, тогда таких проблем не возникнет.
Это если я правильно понял вопрос.

@yafidik · 19.08.2016, 23:25 **[ТС]**

Даценд, проблема в том что я только начал изучать как с# так и sql ... как запрос выглядеть должен можете помочь ?!

@Даценд · 19.08.2016, 23:46

yafidik,
как-то так:

C#

con.Open();
SqlCommand com = new SqlCommand("select id from LOGIN where login=@login", con);
com.Parameters.AddWithValue("@login", textBox1.Text);
int id = (int)com.ExecuteScalar();
con.Close();

@yafidik · 19.08.2016, 23:59 **[ТС]**

Даценд, я правильно понял ?

C#

 con.Open();
 
 
SqlDataAdapter sda = new SqlDataAdapter("insert into LOGIN (login,pasword)values('" + textBox1.Text + "','" + textBox2.Text + "')", con);
sda.SelectCommand.ExecuteNonQuery();
con.Close();
 
SqlCommand com = new SqlCommand("select id from LOGIN where login=@login", con);
com.Parameters.AddWithValue("@login", textBox1.Text);
int id = (int)com.ExecuteScalar();
 
con.Open();
SqlDataAdapter sdaa = new SqlDataAdapter("insert into TECHER (Id,f_t,n_t)values('"+id+"','" + textBox3.Text + "','" + textBox4.Text + "')", con);
sdaa.SelectCommand.ExecuteNonQuery();
con.Close();

если да то вылезает такая ошибка
после

C#

1	int id = (int)com.ExecuteScalar();

Кликните здесь для просмотра всего текста

System.InvalidOperationException не обработано
HResult=-2146233079
Message=Для ExecuteScalar нужно открытое и доступное подключение Connection. Подключение закрыто.
Source=System.Data
StackTrace:
в System.Data.SqlClient.SqlCommand.ValidateCommand(String method, Boolean async)
в System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean asyncWrite)
в System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method)
в System.Data.SqlClient.SqlCommand.ExecuteScalar()
в WindowsFormsApplication1.reg.button1_Click(Object sender, EventArgs e) в d:\Program Files\test c#\1\WindowsFormsApplication1\WindowsFormsApplication1\reg.cs:строка 33
в System.Windows.Forms.Control.OnClick(EventArgs e)
в System.Windows.Forms.Button.OnClick(EventArgs e)
в System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
в System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
в System.Windows.Forms.Control.WndProc(Message& m)
в System.Windows.Forms.ButtonBase.WndProc(Message& m)
в System.Windows.Forms.Button.WndProc(Message& m)
в System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
в System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
в System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
в System.Windows.Forms.UnsafeNativeMethods.DispatchMessageW(MSG& msg)
в System.Windows.Forms.Application.ComponentManager.System.Windows.Forms.UnsafeNat iveMethods.IMsoComponentManager.FPushMessageLoop(IntPtr dwComponentID, Int32 reason, Int32 pvLoopData)
в System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context)
в System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context)
в System.Windows.Forms.Application.Run(Form mainForm)
в WindowsFormsApplication1.Program.Main() в d:\Program Files\test c#\1\WindowsFormsApplication1\WindowsFormsApplication1\Program.cs:строка 19
в System.AppDomain._nExecuteAssembly(RuntimeAssembly assembly, String[] args)
в System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence assemblySecurity, String[] args)
в Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly()
в System.Threading.ThreadHelper.ThreadStart_Context(Object state)
в System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
в System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
в System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
в System.Threading.ThreadHelper.ThreadStart()
InnerException:

@Даценд · 20.08.2016, 00:18

Не обратил внимания, что поле id здесь uniqueidentifier т.е. Guid.
Приводить результат ExecuteScalar нужно не к int, а к Guid и переменная id должна быть соответствующего типа.

@yafidik · 20.08.2016, 00:32 **[ТС]**

Даценд,ааххахаха проблема с переменной + Id с типом uniqueidentifier(63d68281-e438-4a4f-88ff-26c75e10085a) и имеет(newid). пробовал string bool неработал .... object сработал WTF
правильно :??? и можно ли сократить ?

C#

            con.Open();
            
 
            SqlDataAdapter sda = new SqlDataAdapter("insert into LOGIN (login,pasword)values('" + textBox1.Text + "','" + textBox2.Text + "')", con);
            sda.SelectCommand.ExecuteNonQuery();
            SqlCommand comr = new SqlCommand("select id from LOGIN where login=@login", con);
            comr.Parameters.AddWithValue("@login", textBox1.Text);
            object id = (object)comr.ExecuteScalar();
            
            con.Close();
 
           
            con.Open();
            SqlDataAdapter sdaa = new SqlDataAdapter("insert into TACHER (logo_id,f_t,n_t)values('" +id + "','" + textBox3.Text + "','" + textBox4.Text + "')", con);
            sdaa.SelectCommand.ExecuteNonQuery();
            con.Close();
 
            MessageBox.Show("Регестрация прошла успешно");
            this.Hide();
            authentication aa = new authentication();
            aa.Show();

Guid тоже подходит спасибо !!!

@pincet · 22.08.2016, 08:50

во-первых этот "код"

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

SqlDataAdapter sda = new SqlDataAdapter("insert into LOGIN (login,pasword)values('" + textBox1.Text + "','" + textBox2.Text + "')", con);

выкинь на помойку помойку сразу и изучи параметры
а во вторых, изучи предложение OUTPUT для insert

T-SQL

1	insert LOGIN (login,pasword) output inserted.id values .....

и будет счастие

@yafidik · 22.08.2016, 14:15 **[ТС]**

pincet, а что именно не так ? прости я же сказал я новичёк ....а точнее только начал изучать ...

@pincet · 22.08.2016, 14:28

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

выкинь на помойку помойку сразу и изучи параметры

что не ясно?

@Lutd · 22.08.2016, 14:43

Не по теме:

"РегИстрация прошла успешно"

@Даценд · 22.08.2016, 14:52

yafidik,
в моем сообщении выше есть строки:

C#

1 2	SqlCommand com = new SqlCommand("select id from LOGIN where login=@login", con); com.Parameters.AddWithValue("@login", textBox1.Text);

В первой строке @login - имя параметра, вместо которого при выполнении запроса будет подставлено значение. Знак '@' обязателен.
Во второй строке к коллекции параметров команды добавляется новый параметр, ему присваивается значение, взятое из textBox1.Text.
При использовании параметров нет необходимости заботиться о кавычках, решетках и других символах, которые нужны при формировании строки запроса вашим способом, т.к. тип параметра определяется автоматически.

@yafidik · 22.08.2016, 18:02 **[ТС]**

pincet, сорян сам запутал себя .... я кароче думал что тебе не понравился

C#

1 2	SqlCommand com = new SqlCommand("select id from LOGIN where login=@login", con); com.Parameters.AddWithValue("@login", textBox1.Text);

... моя невнимательность не прочёл ... а так спасибо !!! буду изучать

@pincet · 22.08.2016, 18:53

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

select id from LOGIN where login=@login", con

часть ответа, касающуюся этого, ты тоже не прочел. Зачем лишний SELECT, если есть OUTPUT

Передать во второй запрос значение id, сгенерированное при выполнении первого запроса insert

Решение

Решение

@Даценд 5871 / 4748 / 2940 Регистрация: 20.04.2015 Сообщений: 8,361
	19.08.2016, 22:48	2
	yafidik, При помощи промежуточного запроса select получайте значение id новой записи и передавайте во второй запрос. А лучше объедините две таблицы в одну, тогда таких проблем не возникнет. Это если я правильно понял вопрос. 1

@yafidik 0 / 0 / 0 Регистрация: 13.10.2014 Сообщений: 13
	19.08.2016, 23:25 [ТС]	3
	Даценд, проблема в том что я только начал изучать как с# так и sql ... как запрос выглядеть должен можете помочь ?! 0

@Даценд 5871 / 4748 / 2940 Регистрация: 20.04.2015 Сообщений: 8,361
	20.08.2016, 00:18	6
	Сообщение было отмечено yafidik как решение Решение Не обратил внимания, что поле id здесь uniqueidentifier т.е. Guid. Приводить результат ExecuteScalar нужно не к int, а к Guid и переменная id должна быть соответствующего типа. 1

@yafidik 0 / 0 / 0 Регистрация: 13.10.2014 Сообщений: 13
	22.08.2016, 14:15 [ТС]	9
	pincet, а что именно не так ? прости я же сказал я новичёк ....а точнее только начал изучать ... 0

@pincet 1643 / 1144 / 171 Регистрация: 23.07.2010 Сообщений: 6,794
	22.08.2016, 14:28	10
	Сообщение от pincet выкинь на помойку помойку сразу и изучи параметры что не ясно? 0

@Lutd
	22.08.2016, 14:43 #11
	Не по теме: "РегИстрация прошла успешно" 0

@pincet 1643 / 1144 / 171 Регистрация: 23.07.2010 Сообщений: 6,794
	22.08.2016, 18:53	14
	Сообщение от yafidik select id from LOGIN where login=@login", con часть ответа, касающуюся этого, ты тоже не прочел. Зачем лишний SELECT, если есть OUTPUT 0