c# problem kod querija

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

ovu gresku izbaci

ne razumem zasto kada ima 4 querija i 4 destination(2 combo-a i 2 date pickera)

[code]
izbaci mi ovu gresku


ali 4 unosa je (2 combo boxa i 2 data pickera) za 4 kolone u tabeli.

U cemu je problem?
Evo celog koda

[code]using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Data.OleDb;
using System.Windows.Forms;

namespace zadatak2
{
public partial class Form1 : Form
{
private OleDbConnection connection=new OleDbConnection();
public Form1()
{
connection.ConnectionString = @"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=D:\VS2010 projects\zadatak2\zadatak2\fedek.accdb;Jet OLEDB:Database Password=fedek;";
InitializeComponent();
}

private void Form1_Load(object sender, EventArgs e)
{
try
{

connection.Open();
OleDbCommand command=new OleDbCommand();
command.Connection=connection;
string query="select * from konf";
command.CommandText=query;

OleDbDataReader reader=command.ExecuteReader();
while(reader.Read())
{
comboBox1.Items.Add(reader["korisnik"].ToString());
comboBox2.Items.Add(reader["dogadjaj"].ToString());
}

}
catch(Exception ex)
{
MessageBox.Show("error" + ex);
}

connection.Close();
}

private void btnSave_Click(object sender, EventArgs e)
{
string query = "insert into Tomislav (Korisnik, Dogadjaj, DatumOd, DatumDo) values('" + this.comboBox1.SelectedItem + "','" + this.comboBox2.SelectedItem + "''" + this.dtpOd.Text + "''" + this.dtpDo.Text + "') ;";




OleDbCommand command1 = new OleDbCommand();
command1.Connection = connection;
command1.CommandText = query;

OleDbDataReader reader;


try
{
connection.Open();

reader = command1.ExecuteReader();
MessageBox.Show("saved");
while (reader.Read())
{
}

}

catch (Exception ex1)
{
MessageBox.Show("error" + ex1);
}

connection.Close();
}






}
}
[/code]

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Koliko vidim, imaš grešku u upitu, baš kao što kaže poruka:
 string query = "insert into Tomislav (Korisnik, Dogadjaj, DatumOd, DatumDo) values('" + this.comboBox1.SelectedItem + "','" + this.comboBox2.SelectedItem + "''" + this.dtpOd.Text + "''" + this.dtpDo.Text + "') ;" Obrati pažnju na to da nemaš zapete između para atributa 2-3 i 3-4.

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

a to sam ispravio ali mi izbaci kao data missmatch of types, ali u bazi mi se cini sve ok

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Tu je verovatno do datuma problem. Moraš da proveriš kog je formata datum u propertiju Text za DTP, i da li se format poklapa sa onim koji zahteva Access. Ako ti nije bitno da datum bude zabeležen kao datum (ne koristiš samo delove datuma u upitima) najjednostavnije je da u bazi tipove atributa DatumOd i DatumDo postaviš na Text. Pa čak i u tom slučaju možeš da izvučeš delove datuma ukoliko zatreba.

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

string query = "insert into Tomislav (Korisnik, Dogadjaj, DatumOd, DatumDo) values('" + this.comboBox1.SelectedItem + "','" + this.comboBox2.SelectedItem + "','" + this.dtpOd.Value.Date + "','" + this.dtpDo.Value.Date + "') ;";

Ovako sam resio.

• 2Ovo se svidja korisnicima: vasa.93, Fil
  
  Registruj se da bi pohvalio/la poruku!

Sklapanje queryja tako što samo u stringu spojiš sve promenljive nije dobra praksa. Zamisli da neki korisnik sebi da ime: Korisnik) values (''); drop table Tomislav;
Kad budeš pokrenuo query, upisaće se jedan prazan red, a zatim ode tabela.

Da bi izbegao ovo (sql injection) koristi parametre. Svaka sql baza podržava parametre u queryjima, i u velikom broju slučajeva će ponovno poretanje tog queryja sa drugačijim parametrima biti brže nego da sam sklopiš nov query.

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

ne poznajem mnogo parametre....

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Napisano: 04 Okt 2014 15:33

Pa upoznaj ih. Veoma je bitno to. Evo jednog kratkog primera iz kog možeš da vidiš kako to radi:



Dopuna: 04 Okt 2014 15:34

Istina, ovde se radi sa Oracle DBMS-om, ali to i nije toliko bitno.