Discussion :

[Rodie]

Bonjour à tous

J'ai un site Web où des utilisateurs peuvent créer des comptes perso en donnant leur pseudo.
Avant d'insérer leur pseudo, je dois vérifier qu'il n'existe pas déjà.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
string command "select id_member from member where pseudo='robert'";
SqlDataSource dataSource = new SqlDataSource(connectionString, command);
this.mygridView.DataSource = dataSource;
this.mygridView.DataBind();
if (mygridView.Rows.Count > 0)
{
  // pseudo already exists
}

C'est ce que j'ai l'habitude de faire, et je sais que c'est bien pour remplir un gridView mais nul pour tester juste l'existence d'un pseudo en base.
Savez-vous comment je peux simplifier mon programme ?
Mici bien

[StormimOn]

Tu peux modifier ta requête ainsi

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

select count(*) from member where pseudo='robert'

et l'exécuter avec la méthode ExecuteScalar afin de récupérer le nombre de lignes concernées, à savoir 0 ou 1 car ton pseudo doit être unique je suppose.

Pour plus d'infos, ExecuteScalar sur MSDN2

[totoche9]

raté !!

donc, pourquoi ne pas faire un count et tester si tu as > 0 avant de lier ta datagrid à ta datasource.

Ou bien passer par un datareader et un ExecuteReader

Tu fais une première lecture. Si elle échoue c'est que ton select n'a rien ramené sinon tu lie ta datasource.

Espérant t'avoir aidé

[Rodie]

Quoi raté ? ze savais que ç'était nul
Voila ce que j'ai fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
using (SqlConnection connection = new SqlConnection(connectionString))
{
  connection.Open();
  SqlCommand cmd = new SqlCommand(command, connection);
  result = cmd.ExecuteScalar();
  connection.Close();
}
 
if (result == null)
{
  // pseudo already exists
}

Merci bien à vous

[StormimOn]

Merci bien à vous

Je suppose donc que c'est résolu ? Si c'est le cas, clique sur le bouton résolu en bas de page

[Rodie]

Euh... c'est presque résolu
J'ai ajouté une transaction et j'ai le code suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
SqlTransaction transaction = null;
try
{
  using (SqlConnection connection = new SqlConnection(connectionString))
  {
    connection.Open();
    transaction = connection.BeginTransaction();
    SqlCommand sqlCommand = new SqlCommand(command, connection);
    sqlCommand.Transaction = transaction;
    throw new Exception("not working out");
    sqlCommand.ExecuteScalar();
    transaction.Commit();
    transaction.Dispose();
    connection.Close();
  }
}
catch (Exception ex)
{
  if (transaction != null)
  {
    transaction.Rollback();
  }
  throw ex;
}

Le ptit problème, c'est que lorsque je fais le throw new Exception, je fais bien dans le catch en bas mais la transaction est déjà consommée alors le transcration.Rollback() plante.

Ce Rollback() est-il complètement inutile du fait du transaction.Commit() au-dessus ?
Et mon code est-il du code de porc ?

Mici bien pour vos réponses en tout cas !!