WPF ComboBox et String Résolu/Fermé

Question

Bonjour à tous, 

Mon but est de faire un WPF qui contient une ComboBox {=CBox} (sans Items)
Au lancement je récupère dans un tableau , les Ports Série actifs du PC.
J'utilise ensuite ce tableau pour remplir ma CBox. Jusqu'ici aucun problème.
Il parvient quand je souhaite récupérer la sélection du COM de la CBox dans un string :
  MonString = CBox.SelectedValue.ToString(); 

J'ai une exception du type:  

'((ComboBoxItem)CBox.SelectedItem).Content' threw an exception of type 'System.InvalidCastException'

Cependant en essayant avec des Items pré-remplis, aucun problème..
J'ai essayé de convertir mon tableau en plusieurs type différents mais aucun résultat.

Merci d'avance pour vos réponses :D 
Je reste à votre dispo pour toutes questions 


Mon code:

public partial class MainWindow : Window
    {
        public  SerialPort myport;
        string MonString = "";
        public MainWindow()
        {
            InitializeComponent();
            myport = new SerialPort();
            myport.BaudRate = 9600;

string[] ports = SerialPort.GetPortNames();
            foreach (string port in ports)
            {
               // Console.WriteLine(port);
                CBox.Items.Add(port);
            }
        }

private void Connect_Click(object sender, RoutedEventArgs e)
        {
            try
            {
                MonString = CBox.SelectedValue.ToString(); // Le problème est ici!!
                myport.PortName = MonString;
                myport.Open();
            }
            catch { StatuBar.Content = "Echec de connection..."; }
        }
    }

Utilisateur anonyme · Answer

Bonsoir

ton code enfreint à la fois les principes de bases de la programmation objet et l'essence de WPF, à savoir la liaison de données (binding).

public partial class MainWindow : Window
    {
        public  SerialPort myport; // on ne met pas une variable interne public, et par convention le nom d'un membre public commence par une majuscule.
        string MonString = ""; //par convention le nom d'une variable interne commence par une minuscule
        public MainWindow()
        {
            InitializeComponent();
            myport = new SerialPort(); //pas d'objet modèle
            myport.BaudRate = 9600;

string[] ports = SerialPort.GetPortNames();
            foreach (string port in ports) // pas de liaison de données
            {
               // Console.WriteLine(port);
                CBox.Items.Add(port);
            }
        }

private void Connect_Click(object sender, RoutedEventArgs e)
        {
            try
            {
                MonString = CBox.SelectedValue.ToString(); // pas de liaison de données
                myport.PortName = MonString;
                myport.Open();
            }
            catch { StatuBar.Content = "Echec de connection..."; }
        }
    }
 

Une fenêtre est un objet comme un autre et doit respecter les mêmes règles.
Si tu veux que le port série soit accessible de l'extérieur, tu en fais une propriété.

Option 1

public SerialPort MyPort {get; set;}


Dans le code behind, tu pourras l'utiliser exactement comme un champ

toto.MyPort = bidule.MyPort;

Cependant, si tu dois changer quoique ce soit au comportement interne, tu passes à l'option 2

priavte SerialPort myPort;
public SerialPort MyPort
{
    get { return myPort;}
    set  {myPort = value;}
}

On voit que la convention de nommage a tout son sens ici.
Le champ privé est protégé par l'encapsulation dans une propriété, si par sécurité MyPort doit être en lecture seule, c'est impossible avec un champ public, alors qu'avec une propriété il suffit de ne pas écrire le setter.
De plus, la liaison de donnée utilise les propriétés et non les champs.

Pour bien coder en WPF, il faut d'abord, bien coder objet, et avoir compris le binding.
J'ai écrit 2 tutos qui présentent mon point de vue d'autodidacte, ils ne sont pas très académiques, mais justement trop académique pour un autodidacte ça ne marche pas forcément.
https://codes-sources.commentcamarche.net/faq/11239-la-programmation-objet-appliquee-a-net-par-l-exemple-partie-1-sur-3
et
https://codes-sources.commentcamarche.net/faq/11277-apercu-du-binding-en-wpf

Pour approfondir, je te conseille d'acquérir le livre WPF par la pratique de Thomas Lebrun.

Ce soir, je n'ai pas de ports série sous la main pour tester ton problème.
Je regarderai demain, si je peux.

Utilisateur anonyme · Answer

La liaison de données, c'est la traduction de Binding.

Je tache de voir ce qui ne va pas avec les ports dans la journée.

Utilisateur anonyme · Answer

J’ai pris ton code initial, sur un pc avec des ports com. 
J’ai juste viré le try catch.
J’ai une erreur quand je ne sélectionne rien (normal), mais sinon ça fonctionne.
Hier ce qui m’étonnait était le fait que l’erreur n’est pas en lien direct avec ton code.

As tu posté tout le code?

Peux tu virer le try catch, parfois un catch reçoit une erreur venue d’ailleurs dans le code en raison de la pile d’appel:

le try catch est dans bidule
bidule appelle machin
 dans machin pas de try catch
 machin appelle trucmuche
dans trucmuche pas de try catch.
trucmuche plante, c’est le catch de bidule qui reçoit l’erreur

Utilisateur anonyme · Answer

A quelle ligne?
As tu bien sélectionné un port? Cette erreur vaut dire qu’un objet est null.
Or si tu n’as rien sélectionné dans le comboxbox SelectedValue est null

Utilisateur anonyme · Answer

Mets un point d’arrêt sur la ligne 57.
Avec la souris passe sur tous comChange, Choixcom et selectedvalue pour voir lequel des ces 3 objets est null.

Il faudra ensuite déterminer pourquoi

Utilisateur anonyme · Answer

Tu peux poster une capture d’écran juste avant de cliquer sur le bouton?

Utilisateur anonyme · Answer

Là je ne vois pas.

Peux tu mettre ton projet zippé sur un serveur en ligne (cjoint.com par exemple), et poster le lien ici.
Je n'ai plus accès au PC avec des ports com et avec le confinement c'est pas pour tout de suite mais je peux essayer de voir quand même.

Utilisateur anonyme · Answer

Dans le xaml, supprime SelectedValuePath="Content"  il n'y a pas de propriété Content dans une string

WPF ComboBox et String

8 réponses

Discussions similaires