Array als Rückgabetyp ??

  • C++

  • Mr Crown
  • 3017 Aufrufe 14 Antworten

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Array als Rückgabetyp ??

    Hallo FSB-ler,

    Ich habe mal versucht eine Funktion zu schreiben, mit der man eine
    Zahl Ziffer für Ziffer in ein Array einlesen kann. Soweit so gut, nur
    wenn ich jetzt auf die Funktion zugreifen möchte gibt das Probleme.
    Ich möchte auch, dass die Funktion "inArray" "swapArray" zurück
    an main gibt (dass da "return 0" steht ist natürlich falsch), wobei
    "swapArray[i]" die in ein Array umgewandelte Zahl ist und "i" die
    Anzahl der Ziffern. Ich bin jetzt soweit, dass ich rausgefunden habe,
    dass mein Vorhaben nur mit einem Zeiger funktioniert, was für
    mich ein Problem darstellt.
    Vielleicht kann mir ja jemand den Code korrigieren und es so
    schreiben, dass "swapArray[i]" an main übergeben wird.

    Übrigens bekomme ich folgende Fehlermeldung an der Stelle,
    wo in main() auf die Funktion "in Array()" zugegriffen wird:

    "cannot convert `int*' to `int (*)[10]' for argument `2' to `int inArray(int, int (*)[10])' "

    PHP-Quellcode

    1. #include <iostream>
    2. #include <conio.h>
    4. using namespace std;
    6. int inArray (int zahl, int (*array)[10])
    7. {
    8. int b=0, c=0, i=0, d=0;
    10. do
    11. {
    12. b = zahl / 10;
    14. b *= 10;
    16. *array[c] = zahl-b;
    18. zahl /= 10;
    20. c++;
    21. }
    22. while (zahl != 0);
    24. i=c-1;
    26. int swapArray[i];
    28. do
    29. {
    30. swapArray[d] << *array[i];
    32. i--;
    33. d++;
    34. }
    35. while (i >= 0);
    37. return 0;
    38. }
    41. int main ()
    42. {
    43. int a=0, ar[10];
    45. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    47. cin >> a;
    49. inArray (a, &ar[10]);
    51. getch();
    53. return 0;
    54. }
    Alles anzeigen

  • Mmh,

    int ar[10] ist keine richtiger Typ. Entweder du übergibst nur int* oder eine andere Variante: Du verwendest für das Array eine Klasse und übergibts dann
    der Funktion "inArray" eine Referenz auf die neu erzeugte Klasse (call by reference). In der Funktion kannst du dann das Array nach belieben ändern.

    class Arrayclass
    {
    int ar [10];
    }

    int inArray (int zahl, Arrayclass& ar)
    {
    ...
    ar.ar[c] = zahl-b;
    ...
    }
    der Funktionsaufruf kann dann so erfolgen
    Arrayclass myarray;
    inArray(a,&myarray);

    Man kann sich auch an der Stelle einen eigenen << operator für die Arrayclass vorstellen oder direkt mit dem Container "vector" arbeiten.

    Gruß Ricci123456

  • Habe das jetzt mal mit der Klasse versucht, bekomme jetzt aber noch mehr Fehlermeldungen: "13 new types may not be defined in a return type "

    Ich bin mit den Klassen auch noch nicht so vertraut und Zeiger sowie Refernzen bringen mich immer durcheinander. Ich poste den Code jetzt nochmal, vielleicht kann mir noch jemand einen Tipp geben und das mit dem Rückgabetyp ist auch noch nicht gelöst.
    Danke für die Mühe.

    PHP-Quellcode

    1. #include <iostream>
    2. #include <conio.h>
    4. using namespace std;
    6. class Arrayclass
    7. {
    8. public:
    9. int ar [10];
    10. }
    12. int inArray (int zahl, Arrayclass& ar)
    13. {
    14. int b=0, c=0, i=0, d=0;
    16. do
    17. {
    18. b = zahl / 10;
    20. b *= 10;
    22. ar.ar[c] = zahl-b;
    24. zahl /= 10;
    26. c++;
    27. }
    28. while (zahl != 0);
    30. i=c-1;
    32. int swapArray[i];
    34. do
    35. {
    36. swapArray[d] << *array[i];
    38. i--;
    39. d++;
    40. }
    41. while (i >= 0);
    43. return 0;
    44. }
    47. int main ()
    48. {
    49. int a=0;
    51. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    53. cin >> a;
    55. Arrayclass myarray;
    57. inArray(a, &myarray);
    59. getch();
    61. return 0;
    62. }
    Alles anzeigen

  • Bin mehr in Vb.net unterwegs hab aber auch einige Kentnisse in c++. Ich will dir mal versuchen zu helfen.:D

    Wenn du mit einer Funktion etwas zurück gibst dann musst du immer ein Töpfchen aufhalten :D:D:D
    Also nochmal bisschen besser erklärt in deiner main musst du eine Variable deklarieren und dem den rückgabewert der Funktion zuweisen.

    PHP-Quellcode

    1. array=inArray (a, &ar[10]);

    Ich beziehe mich auf den code den du zuerst gepostet hast.

  • Habe es nun geschafft. Danke.
    Nur jetzt habe ich ein weiteres Problem: Und zwar möchte ich in der Funktion "swapArray()" das Array "ar[]" umdrehen. Nur bekomme ich beim Funktionsaufruf folgende Fehlermeldung:
    44 cannot convert `int*' to `int**' for argument `1' to `int swapArray(int**, int*)'

    PHP-Quellcode

    1. #include <iostream>
    2. #include <conio.h>
    4. using namespace std;
    6. int swapArray(int *array[], int *index)
    7. {
    8. int d=*index;
    9. int swapper[d], e=0;
    11. do
    12. {
    13. swapper[e] << *array[d];
    15. e++;
    16. d--;
    17. }
    18. while (d >= 0);
    20. return swapper[e];
    22. }
    24. int inArray (int *ar, int *zahl)
    25. {
    26. int b=0, c=0, i=0;
    28. do
    29. {
    30. b = *zahl / 10;
    32. b *= 10;
    34. ar[c] = *zahl-b;
    36. *zahl /= 10;
    38. c++;
    39. }
    40. while (*zahl != 0);
    42. *zahl = c;
    44. swapArray (ar, &c);
    46. return ar[i], *zahl;
    47. }
    50. int main ()
    51. {
    52. int a=0, myarray[10];
    54. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    56. cin >> a;
    58. inArray(myarray, &a);
    60. for (int j=0; j<a; j++)
    61. {
    62. cout << myarray[j];
    63. }
    65. getch();
    67. return 0;
    68. }
    Alles anzeigen

    Was soll ich tun ?

    Danke an alle, die mir versuchen zu helfen.

    Mit freundlichen Grüßen

    Mr Crown

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von Mr Crown ()

  • Vorweg: Dies ist die Lösung des Problems:

    int swapArray(int array[], int *index)
    {
    array[d];

    e++;
    d--;px"> swapper[e] << array[d];

    e++;
    d--;

    }
    while (d >= 0);

    return swapper[e];px"> int d=*index;
    int swapper[d], e=0;

    do
    {
    array[d];

    e++;
    d--;px"> swapper[e] << array[d];

    e++;
    d--;

    }
    while (d >= 0);

    return swapper[e];

    }


    Kurze Erklärung: Ein Array ist in C sowas wie ein Zeiger. Angenommen, du hast ein Array int myArray[10] deklariert.

    Um die Speicheradresse des 1. Elements von myArray[] zu bekommen, kannst du schreiben:


    int *ptr = &myArray[0];


    myArray[0] gibt dabei den Inhalt der Speicheradresse des 1. Elementes an; das &-Zeichen wiederum gibt uns die Referenz, also die Speicheradresse selbst vom 1. Element zurück.

    Anstelle dessen gibts für das 1. Element auch die Kurzschreibweise

    int *ptr = myArray;


    Merke: Der Bezeichner eines Arrays ohne alles gibt die Speicheradresse des 1. Elementes an!

    Um auf das 4. Element zugreifen zu können, kannst du mittels Zeigerarithmetik schreiben:

    int *ptr = myArray + 3;


    oder halt kurz in Arrayschreibweise:

    int *ptr = &myArray[3];



    Ergo: es ist also egal, ob in einer Funktion als Parameter ein Zeiger oder ein Array definiert wird:


    void foo(int myArray[]) ~= void foo(int *ptrArray)


    Du hast jedoch geschrieben:

    int swapArray(int *array[], int *index)


    also einen Zeiger auf einen Zeiger(!) definiert (es macht an manchen Stellen durchaus Sinn, sowas zu tun, nur leider nicht hier ;) )

    denn:


    int swapArray(int *array[], int *index) ~= int swapArray(int **ptrArray, int *index)


    Und weil der Aufruf von swapArray bei Dir mit

    swapArray (ar, &c);


    erfolgt, übergibst Du (unwissentlich) nur einen Zeiger auf das Array ar, jedoch nicht wie fälschlich deklariert, einen Zeiger auf einen Zeiger. Deshalb beschwert sich der Compiler.


    Hoffe, das war verständlich. Weiterhin viel Erfolg,

    quappo

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von quappo ()

  • Was du tun sollst? Verstehen was das du da programmiert hast.
    Das geht nämlich komplett in die Hose. Deine "geswapptes" Array löst sich nach dem Funktionsaufruf in Luft auf.

    int swapArray(int *array[], int *index)
    {
    int d=*index;
    int swapper[d], e=0;

    ...
    return swapper[e]; <- swapper gibt nur den int an der Stelle e zurück nicht das Array swapper
    }
    ...
    swapArray (ar, &c);
    ...
    "ar" ist ein zeiger auf int und wenn du diesen Zeiger weitergeben willst muss die Funktion auch einen Typ int* aktzeptieren. Das ist bei dir nicht der Fall. "array[]" ist in deinem Fall ein int* und mit "int* array[]" wäre das int**. Deswegen meckert der Compiler.

    return ar, *zahl; <-das geht auch nicht.

    Ich setze mich morgen mal an meinen Compiler und versuchs noch mal dir zu helfen.

    MFG Ricci123456

    PS: Um einfach nur die Zahlen in umgekehrter Reihenfolge auszugeben kannst du auch die Schleife rückwärts laufen lassen.

  • nichts für ungut, Ricci, aber mit der Änderung aus meinem Post funktioniert sein Code. Es gibt sicherlich einen eleganteren Stil, aber wichtig ist das Verständnis, warum etwas funktioniert oder halt auch nicht funktioniert.

    edit:

    ich habs unter osx mit XCode kompiliert und da geht alles einwandfrei. Hab gar nicht auf diese merkwürdige return ar, *zahl; Geschichte geachtet -> warum meckert der Compiler da nicht?! DAS verstehe ICH grad nicht.

    Also nochmal @Crown:

    int myFunction(int param1, int *array) {

    param1 = 1; // aendert nichts ausserhalb von myFunction
    array[0] = 2; // setzt auch ausserhalb von myFunction das 1. Element = 2
    return 3; // ist der eigentliche Rueckgabewert der Funktion (int)

    }

    das return statement bezieht sich auf den Rückgabewert der Funktion (der ist hier int). Ein Rückgabewert ist ein [I]Einzelwert    , kein Tupel! Mehrere Rückgabewerte können in einem struct verpackt werden oder halt in den Parametern.

    Letztere Variante ist unter C die gebräuchliche Variante. Der int-Rückgabewert wird nur benutzt, um anzuzeigen, ob ein Fehler in der Methode aufgetreten ist, oder nicht. Dadurch, dass der zweite Parameter ('array') ein Zeiger ist, wirkt sich alles, was du innerhalb der Methode an der Variable veränderst auch außerhalb der Methode auf (du schreibst ja direkt in die Speicherstelle rein!).

    Die erste Variante wiederum setzt den Wert von param1 innerhalb von myFunction auf 1, sobald die Methode verlassen wird, hat dies aber keine Auswirkungen mehr auf den Rest.

    Gute n8!

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von quappo () aus folgendem Grund: muss doch noch etwas eingestehen

  • Danke erstmal für die Antworten. Ich glaube, dass ich da wirklich ein sehr vernetztes und unübersichtliches Feld von Zeigern erschaffen habe.
    Dank quappo meckert mein Compiler nicht mehr, aber wie Ricci123456 schon gesagt hat wird nicht das Array swapper zurückgegeben.
    Ich habe das ganze Problem nun mit <vector> aus der STL gelöst, nachdem ich darauf aufmerksam gemacht worden bin. Die ist viel einfacher, da ein vector ein sich anpassendes array ist, aber das wisst ihr wahrscheinlich schon.

    PHP-Quellcode

    1. #include <iostream>
    2. #include <conio.h>
    3. #include <vector>
    5. using namespace std;
    7. int inArray (vector<int> *vec, int *zahl)
    8. {
    9. int b=0, c=0, i=0;
    11. do
    12. {
    13. b = *zahl / 10;
    15. b *= 10;
    17. (*vec).push_back( *zahl-b );
    19. *zahl /= 10;
    21. c++;
    22. }
    23. while (*zahl != 0);
    25. reverse((*vec).begin(), (*vec).end());
    26. }
    29. int main ()
    30. {
    31. int a=0;
    33. vector<int> myvector;
    35. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    37. cin >> a;
    39. inArray(&myvector, &a);
    41. vector<int>::iterator k;
    43. for ( k=myvector.begin() ; k < myvector.end(); k++ )
    45. {
    46. cout << *k;
    47. }
    49. getch();
    51. return 0;
    52. }
    Alles anzeigen


    Trotzdem würde ich das Problem auch gerne mit Arrays lösen, also wenn jemand weiß wie, immer her damit. Ansonsten reicht es mir im Moment auch dieses Problem mit <vector> gelöst zu haben, da man so weit ich weiß einen Vector auch in ein Array umwandeln kann.
    Vielen Dank für eure Hilfe

    Mit freundlichen Grüßen

    Mr Crown

  • Irgendwie ist es ärgerlich, wenn man sich hier nen Wolf schreibt und Du nicht mal probierst, Deinen Code zu verstehen. :confused:

    daher ohne Kommentar:

    PHP-Quellcode

    1. #include <iostream>
    3. using namespace std;
    5. int inArray (int *ar, int zahl)
    6. {
    7. int idx = 0;
    9. while (zahl > 0) {
    10. ar[idx++] = zahl % 10;
    11. zahl /= 10;
    12. }
    14. return idx; // Länge der Zahl in Zeichen
    15. }
    18. int main ()
    19. {
    20. int myarray[10];
    21. int zahl = 0;
    22. int len = 0;
    24. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    25. cin >> zahl;
    27. len = inArray(myarray, zahl);
    29. for (int i=0; i<len; i++) cout << myarray[i];
    32. getchar();
    34. return 0;
    35. }
    Alles anzeigen

  • Sorry wenn es falsch rüber gekommen ist. war keine Absicht. Hab das alles ohne Compiler hingeschnuddelt.

    Hier noch meine getestete Variante.

    C-Quellcode

    1. #include <iostream>
    2. #include <vector>
    3. #include <functional>
    4. #include <algorithm>
    7. using namespace std;
    9. int inVector (vector<int>& iv, int zahl)
    10. {
    11. const int zehn=10;
    12. modulus<int> x;
    13. do
    14. {
    15. iv.push_back (x(zahl,zehn));
    16. zahl/= zehn ;
    17. }
    18. while (zahl != 0);
    19. return iv.size();
    20. }
    22. ostream& operator<< (ostream & os, vector <int>& v)
    23. {
    24. for (vector <int>::iterator i1 = v.begin( ) ; i1 != v.end( ) ; ++i1 )
    25. os << " " << *i1;
    26. return os;
    27. }
    29. int main()
    30. {
    31. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    32. int a=0;
    33. cin >> a; // hier nur Zahlen eingeben
    34. vector<int> vx;// leerer int-vector
    35. vector<int> vy(inVector (vx, a));// inVector füllt vx und gibt die Anzahl der Einträge an den Constructor von vy
    36. reverse_copy ( vx.begin( ), vx.end( ), vy.begin( ));// Function aus der standardlibary
    37. cout << "original: " << vx << std::endl;// Ausgabe der Originalwerte
    38. cout << "reverse : " << vy << std::endl;// Ausgabe in umgekeherter Reihenfolge
    39. cin >> std::ws;
    40. return 0;
    41. }
    Alles anzeigen

    Grüße Ricci123456

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von Broken Sword ()

  • quappo schrieb:

    Irgendwie ist es ärgerlich, wenn man sich hier nen Wolf schreibt und Du nicht mal probierst, Deinen Code zu verstehen. :confused:



    Tut mir leid, wenn Du das denkst und es so rüberkommt, aber ich hänge mich voll rein. Meine Zeit ist gerade etwas begrenzt, darum kann ich das jetzt nicht sofort durcharbeiten, denke aber das ich mich morgen mal damit ausgibig beschäftigen werde.
    Wie gesagt Zeiger sind nicht so mein Ding und deshalb war es nötig, dass ihr mir etwas auf die Sprünge helft. Danke.
    Ich habe ja auch geschrieben, dass ich das Problem auch gerne mit Arrays gelöst hätte und darum werde ich mich jetzt ein Bisschen und morgen ausgibig damit beschäftigen um alles das zu verstehen was ihr geschrieben habt. Erstmal vielen vielen Dank für die tolle Hilfe und entschuldigung, dass es so rüberkommt als würde ich nicht mal versuchen meinen Code zu verstehen.
    Mit freundlichen Grüßen

    Mr Crown

  • Fands nur etwas komisch, dass Du nach einer konkreten Sache fragst, dann aber mit keinem Wort drauf eingehst und hier einen "egal, ich hab jetzt was ganz anderes gemacht" Post reinstellst. Da machts dann keinen Sinn, Fragen zu beantworten.

    Aber das ist wohl falsch rübergekommen, also Schwamm drüber :bier1:

    Mal Titten auf den Tisch: Wofür brauchst Du das? Eigeninteresse oder Schul-/Uniaufgabe? Einzelfragen beantworte ich immer gern, aber mal eben (C-)Programmieren beibringen klappt nicht.

    Schnapp Dir ein gutes c++-Buch und arbeite es durch. Es soll ja Cracks geben, die drei Tutorials querlesen und dazu die API-Doku und damit loszaubern können. Ich kanns leider nicht.:( Du wahrscheinlich auch nicht ;)

    Ein guter Tipp: :read: "Einstieg in C++" von Arnold Willemer / ISBN 3-89842-649-1 :read:

    Dazu dann noch irgendwas über grafische Programmierung je nach Gusto vielleicht Visual C++ oder Qt oder ... -> die Erfolgserlebnisse motivieren ungemein, wenn die "Hallo Welt"-Programme plötzlich nicht mehr in der Konsole sondern in nem Fenster auftauchen ;)

    Was Dein Problemchen hier angeht: Konkrete Fragen beantworte ich gerne weiter.

    Auf gutes Gelingen,

    Quappo

  • Am Anfang war es wirklich nur eine Konkrete Frage. Ich dachte, ich hätte bei diesem komplizierten Problem einen kleinen Denkfehler, dabei war das ganze gar nicht so einfach wie ich dachte.
    Natürlich sollte und soll das nicht darauf hinauslaufen, dass mir hier irgendwer Nachhilfe in C++ geben soll.
    Ich habe schonmal einen großen Teil des Buches "C++ für Dummies" durchgearbeitet und ab und zu mal hier ein Tutorial mal dort. Dann habe ein Jahr Pause gemacht und vor einem Monat wieder angefangen. Aber ich mich jetzt fast nur mit Mathematischen Sachen beschäftigt, bei denen man hauptsächlich Schleifen und Modulo (%) braucht.
    Jetzt fiel mir ein, es wäre praktisch, wenn man mal eine Funktion hätte, die Zahlen zerlegt (ihr wisst was ich meine) und so bin ich zu diesem Problem gekommen.

    Ich glaube ich bräuchte ein Buch, in dem es nach jedem Kapitel ein paar aufgaben gibt, damit ich mein neu Erlerntes gleich auf die Probe stellen kann.:read:
    Dieses Buch sollte für Einsteiger sein, aber nicht unbedingt von Hello World an. Und es sollte viel über OOP drin sein. Also ich schau ob ich was finde.
    Ist bestimmt besser als überall was falsches aufzuschnappen (nicht im FSB natürlich) und dann mit so einem Problem wie diesem hier irgendwelche Experten zu nerven, die wahrscheinlich besseres zu tun haben.
    Viele Dank für Euer Verständnis
    Morgen versuche ich noch mal meinen Ursprungscode zu überarbeiten.

    Was für ein Roman:D

    Mr Crown

  • Ich habe das POroblem jetzt gelöst. Ich habe eure Beiträge gelesen, verstanden und in meinen Code mit eingebracht. Das Problem mit den Zeigern war nach dem Beitrag von quappo erledigt. Danke.
    Ich poste jetzt einfach mal meinen gesamten Code, wobei ich nur Änderungen in der Funktion "swapArray()" unternommen habe:

    PHP-Quellcode

    1. #include <iostream>
    2. #include <conio.h>
    4. using namespace std;
    6. int swapArray(int array[], int *index)
    7. {
    9. int d=*index;
    10. int swapper[d], e=0;
    12. do
    13. {
    15. swapper[e] = array[d-1]; //Das Array "array[]" wird umgedrht in das
    16. //das Array "swapper[]" geschriben
    17. e++;
    18. d--;
    20. }
    21. while (d >= 0);
    23. for (int j=0; j<=(*index); j++) //In dieser Schleife wird das Array "swapper[]"
    24. { //in das Array "array[]" zurückgeschrieben
    25. array[j] = swapper[j];
    26. }
    27. }
    29. int inArray (int *ar, int *zahl)
    30. {
    31. int c=0, i=0;
    33. do
    34. {
    35. ar[c] = *zahl % 10;
    37. *zahl /= 10;
    39. c++;
    40. }
    41. while (*zahl != 0);
    43. *zahl = c;
    45. swapArray (ar, &c);
    46. }
    49. int main ()
    50. {
    51. int a=0, myarray[10];
    53. cout << "Bitte geben Sie eine Zahl zwischen 0 und 2.1 Mrd. ein : ";
    55. cin >> a;
    57. inArray(myarray, &a);
    59. for (int j=0; j<a; j++)
    60. {
    61. cout << myarray[j];
    62. }
    64. getch();
    66. return 0;
    67. }
    Alles anzeigen


    Jetzt ist das vielleicht alles etwas umständlich, aber der Lerneffekt war groß (jedenfalls bei mir) und es klappt so, wie ich es mir am Anfang vorgestellt habe und mein Compiler scheint auch ganz zufrieden zu sein.

    Danke nochmal für eure großartige Hilfe.

    Mit freundlichen Grüßen

    Mr Crown