Discussion :

[takinelinfo]

Bonjour,

je me pose actuellement une petite question.
Vaut il mieux caster un objet ou en créer un nouveau en terme de performance?

Voici un petit bout de code:

Code c# :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public class DatabaseConnection
{
	...
	...
 
	public IEnumerable<NpgsqlDataReaderEvoluate> Read(NpgsqlDataReader result)
	{
		while (result.Read()){
			yield return new NpgsqlDataReaderEvoluate(result);
			//Ou vaut il mieux faire yield return new (NpgsqlDataReaderEvoluate) result;
		}
	}
}

Et ma classe

Code c# :

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
public class NpgsqlDataReaderEvoluate
{
	private NpgsqlDataReader _Reader { get; set; }
	public NpgsqlDataReaderEvoluate(NpgsqlDataReader reader)
	{
		_Reader = reader;
	}
 
	public int GetInt(string fieldValue)
	{
		return _Reader.GetInt32(_Reader.GetOrdinal(fieldValue));
	}
 
	public string GetString(string fieldValue)
	{
		return _Reader.GetString(_Reader.GetOrdinal(fieldValue));
	}
 
	....
	....
	....
}

Le problème c'est que je n'arrive pas à hériter de NpgsqlDataReader ("Cannot access internal constructor 'NpgsqlDataReader' here").

Bref vaut il mieux hériter (trouver une solution) ou créer un nouvel objet sera aussi performant?

[DonQuiche]

Bonsoir.

Un cast est évidemment plus rapide qu'une instanciation. Mais si la classe dont tu voudrais hériter a rendu tous ses constructeurs inaccessibles, cela n'est pas possible.

Cela ne te laisserait donc que la solution que tu as utilisée et qui équivaut grosso modo au pattern "adaptateur" (adapter).

[brachior]

Si ta classe englobante ne diffère que de quelques méthodes n'utilisant rien d'autre que l'objet englobé,
Tu peux voir à utiliser des méthodes d'extensions.

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
 
public static class NpgsqlDataReaderEvoluate
{
  public static int GetInt(this NpgsqlDataReader reader, string fieldValue)
  {
    return reader.GetInt32(reader.GetOrdinal(fieldValue));
  }
 
  public static string GetString(this NpgsqlDataReader reader, string fieldValue)
 {
    return reader.GetString(reader.GetOrdinal(fieldValue));
  }
 
  ....
  ....
  ....
}

[Guulh]

Tu peux voir à utiliser des méthodes d'extensions.

Absolument

J'ajoute que l'on a jamais intérêt à dériver les classes d'ADO.Net, parce que la bonne pratique est plutôt de se servir des factories (comme détaillé ici), et que l'on instancie donc aucune classe directement via new.

J'ajoute aussi que ces méthode d'extension ont intérêt à prendre les interface (ou les classes abstraites) telles que IDataReader, IDataParameter en paramètre, leur code ne dépendant pas du provider utilisé (ici Npgsql, mais ça pourrait être Oracle, SQL server, ...)