Discussion :

[zakotcha]

bonjour,
je suis entrain de développer un module de disponibilité de chambre
lorsque je choisie la date début et la date fin et je clique sur le bouton rechercher je dois afficher la liste de toutes les chambres dans un tableau et devant ceux disponibles je mets une colonne vert et devant ceux occupées je mets une colonne rouge

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
25
26
string cmdQuery = "select ID_ch, nom_ch date_debut, date_fin from calendrier, chambre where chambre.ID_ch=calendrier.ID_ch";
	OracleCommand cmd = new OracleCommand(cmdQuery);
	 cmd.Connection = con;
    cmd.CommandType = CommandType.Text;
	 OracleDataReader reader = cmd.ExecuteReader();
	  while (reader.Read())
    {
	TableRow tRow = new TableRow();
                tb.Rows.Add(tRow);
                TableCell tCell = new TableCell();
                TableCell tCel2 = new TableCell();
                if (Convert.ToDateTime(TextBox1.Text)>reader[1] || Convert.ToDateTime(TextBox1.Text)<reader[2] || Convert.ToDateTime(TextBox2.Text)>reader[1] || Convert.ToDateTime(TextBox2.Text)>reader[2])
                {
                    tCell.Text = reader[0];
                    tRow.Cells.Add(tCell);
                    tCel2.BackColor = Color.Green;
                    tRow.Cells.Add(tCel2);
                }
                else
                {
                    tCell.Text = reader[0];
                    tRow.Cells.Add(tCell);
                    tCel2.BackColor = Color.Red;
                    tRow.Cells.Add(tCel2);
                }
	}

avec ce code il m'affiche toujours les chambres occupées seulement
alors que je devrais afficher tous les chambres en indiquant les occupées par une couleur rouge et les disponibles par une couleur vert
sachant que j'ai deux tables
Calendrier: id_cal, date_debut,date_fin,ID_ch
Chambre: ID_ch, nom_ch, id_visiteur

[StringBuilder]

1/ La recherche de disponibilité devrait se faire directement dans la base de données et non dans ton code. Là si tu as un milliard de chambres, dont seulement 10 disponible, tu vas charger le milliard de ligne et planter à la fois la base de données, le réseau, et le programme.
2/ Pour saisir une date, n'utilise pas un TextBox, mais un DatePicker. Là tu n'as aucune certitude que la valeur saisie est une date valide. Et encore moins que son format est celui que C# attend.
3/ Du coup tu devrais modifier ton code et poser la question sur le forum "Langage SQL" dans la catégorie bases de données.

[jopopmk]

Salut,

en plus des remarques de StringBuilder, quelques une en sus sur le SQL :
- il te manque une virgule,
- que vient faire id_visiteur dans ta table Chambre ?
- dans ta requête où est le BETWEEN entre les dates ?
- enfin, si tu veux avoir toutes les lignes de ta table Chambre même si aucun tuple n'est associé côté Calendrier, il faut faire une "jointure à gauche" (la syntaxe d'une telle jointure peut dépendre du SGBD utilisé) mais attention, ça reste une "multiplication", si tu as plusieurs réservations pour une même chambre en considérant les contraintes appliquées (dates), celle-ci apparaîtra plusieurs fois.