C# baies de Passage entre la classe A et de Classe B

J'essaie d'obtenir array[i] de la classe B de ma classe A.

Dans la classe B, j'ai:

public array getArray() {
    return array[i];
}

Comment puis-je obtenir le plein array pour faire ce travail?

combobox1.items.add(getArray());

Je suis en train d'essayer cela, mais il ne fonctionne pas. Pouvez-vous s'il vous plaît répondre avec un exemple de code qui explique les deux méthodes dans des classes différentes qui envoyer et obtenir array[i].

  • Terrible de réponses à cette question.
InformationsquelleAutor Ivan | 2011-01-31
  1. 1

    Vous avez besoin de donner class A l'accès à l'ensemble de la matrice, et pas seulement une valeur unique. Par exemple 

    class B {
  Object[] _array;
  public Object[] GetArray() {
    return _array;
  }
}

    Maintenant class A peut consommer tous les éléments et de remplir entièrement la zone de liste modifiable

    foreach (object element in classBInstance.GetArray()) {
  combobox1.Items.Add(element);
}
    • C'est sale. En orienté objet monde, vous ne devriez jamais vous retourner votre tableau interne directement. -1
    • l'utilisateur semble être assez nouveau dans la langue, et je suis d'opter pour la plus simple solution.
    • En fait le plus simple de code serait une propriété: public Object[] Montableau { get; private set; } dans la classe B.
    • J'aimerais éviter que parce que c'est à l'aide d'un plus avancées fonctionnalité du langage.
    • OK, je pense que tout le monde ici a un morceau de la vérité.
    InformationsquelleAutor JaredPar
  2. -1

    Bien votre exemple, vous ai-je, mais n'est pas lancé n'importe où, vous avez besoin de passer comme ceci:

    public array getArray(int i)
{
    return array[i];
}



combobox1.items.add(getArray(5));
    • Le même que ci-dessus: C'est sale. En orienté objet monde, vous ne devriez jamais vous retourner votre tableau interne directement. -1
    InformationsquelleAutor hitmanDX
  3. -1

    Votre question n'est pas très clair, donc je vais essayer de répondre en fonction de ce que j'ai compris à votre problème.

    class A 
{
    string[] array;
    public string[] GetArray() { return array; }
}

    ensuite, vous pouvez écrire du code comme celui-ci à remplir un combobox

    A items = new A();
//fill in array in A
comboBox1.DataSource = items.GetArray();

    ou

    foreach(string item in items)
{
    comboBox1.Items.Add( item );
}

    Je ne définissez des listes à l'aide de la DataSource membre de tous les temps, surtout quand il va partir enums pour les chaînes de caractères.

    Exemple:

    comboBox1.DataSource = System.Enum.GetNames(typeof(System.DayOfWeek));
    • Le même que ci-dessus: C'est sale. En orienté objet monde, vous ne devriez jamais vous retourner votre tableau interne directement. -1
    • vous ne quand vous vous souciez de la vitesse, et ne pas avoir à copier les valeurs de 10 000 fois par seconde. Pas tout le monde est la construction d'Bancaire code, qui a besoin de ce niveau de sécurité. M'épargner la programmation orientée objet de l'idéalisme, et de répondre de façon réaliste à la question posée.
    InformationsquelleAutor ja72
  4. -1

    Vous à la parcourir par le biais de votre tableau:

    foreach(var item in GetArray())
{
     combobox1.Items.Add(item);
}
    • Eh bien, si j'ai bien compris foreach(var item dans ClassB.GetArray()) { combobox1.Éléments.Add(item);}, avec référence à classb?
    • Le même que ci-dessus: C'est sale. En orienté objet monde, vous ne devriez jamais vous retourner votre tableau interne directement. -1
    InformationsquelleAutor Steve B
  5. -2

    Bien que d'autres réponses sont techniquement de travail, à mon avis, on ne devrait jamais retourner un tableau interne directement, car elle va à l'encontre des bonnes pratiques en programmation orientée objet. Donc, je voudrais les mettre en œuvre interface IEnumerable de la classe ou au moins de retour IEnumerator ou IEnumerable qui ferait de la collection retournée en lecture seule.

    Exemple:

    class ClassB {
  int[] array;
  public IEnumerable<int> GetValues() { return array; }
}

class ClassA {
  void DoTheJob(ClassB source) {
    foreach(int item in source.GetValues()) {
      combobox.Items.Add(item);
    }
  }
}

    Une autre solution propre est d'ajouter "Faire avec" valeurs de la fonctionnalité de la classe B. il nous suffit Alors de fournir délégué. Exemple:

    class ClassB {
  int[] array;
  public void DoWithValues(Action<int> action) {
    foreach(int item in array) action(item);
  }
}

class ClassA {
  ComboBox combobox;
  void DoTheJob(ClassB source) {
    source.DoWithValues(item => combobox.Items.Add(item));
  }
}

    La prochaine et troisième exemple montre comment faire tout ClassB énumérable avec foreach:

    static void Main(string[] args) {
  ClassB b = new ClassB();
  foreach(int i in b) Console.Write(i + " ");
}

class ClassB : IEnumerable<int> {

  int[] array;

  public ClassB() {
    array = new int[20];
  }

  public IEnumerator<int> GetEnumerator() {
    return (array as IEnumerable<int>).GetEnumerator();
  }

  IEnumerator IEnumerable.GetEnumerator() {
    return array.GetEnumerator();
  }
}

    mise à jour: Comme ja72 montré, cette technique simple qui vient de publie l'énumérateur du tableau interne, n'est pas physiquement masquer le tableau, comme résultant de IEnumerator peuvent être précipités sur int[].

    mise à jour #2: Mon frère de toujours utiliser Clone() pour retourner une copie de tableau. Il est le plus lent, mais il dit qu'il a sauvé sa carrière un peu de temps (il travaille sur un grand projet au sein d'une équipe).

    • Pouvez-vous montrer un exemple de l'interface IEnumerable car j'ai googlé mais n'a jamais obtenu un succès qui donne un droit de réponse
    • Ivan, si vous avez un tableau interne, vous pouvez le retourner comme IEnumerable. Look à la méthode GetValues() dans mon premier exemple. C'est la voie la plus facile. Je vais vous montrer une autre possibilité (faire de l'ensemble de la classe B énumérable) dans le troisième exemple.
    • -1. Votre code suppose class A est un wrapper (ne peut avoir qu'une collection ou un tableau qu'il représente) ou elle retourne un tableau (masqué comme IEnumarable<> qui peut encore endommager le tableau d'origine et est contre les principes de la programmation orientée objet (par vos commentaires). Vous pouvez toujours faire int[] list = (int[])GetValues()
    • Sans en savoir plus sur Ivan le code de cet exemple odeurs de complexité inutile.
    • Mon code ne présumez pas qu'il y a un tableau dans la classe. Ivan lui ai demandé de me montrer comment faire de l'ensemble de la classe énumérable, et j'ai montré de la façon la plus simple. Votre autre plainte est beaucoup plus intéressant. Oui, c'est une question classique: Si le recenseur peut être jeté en arrière de type tableau, est-il correct ou pas? En fait, l'héritage ne doit pas être utilisé comme une technique de protection. Si vous voulez physiquement cacher le fait qu'il peut être jeté en arrière, je peux donner un autre exemple avec un rendement de commande.
    • Le rendement est une solution acceptable, et je suis d'accord que l'héritage ne doit pas être utilisé comme une technique de protection, et vous downvoted ma réponse avec l'excuse de la pure programmation orientée objet protège mieux (ce qui est balivernes en fait). Veuillez revoir vos commentaires et de votes et d'être plus cohérent.
    • Je vous remercie pour vos précieux commentaires. Mais encore, je pense que ma solution est un peu meilleure que la vôtre. Bien que, techniquement, les deux sont de la même "d'insécurité", ma façon transparente dit "C'est IEnumerable, c'est en lecture seule." Et j'ai aussi ajouter une note de mon frère: Il dit qu'il a toujours faire un Clone() sur les tableaux, et qu'il a sauvé sa carrière quelques temps déjà. 🙂

    InformationsquelleAutor Al Kepp