Funktion mit 2 Rückgabewerten

trequ

Hallo,

Ich habe folgende Funktion:
(Ich muss das ganze mal auf GetLogicalDrives() umbauen, das ist bestimmt weniger Code, ändert aber an dem Problem nichts.)

int find_used_dl(){
	int dl_cnt = 0;			// Zählt die Anzahl der gefunden Laufwerke
	CString dl[26];			// Array für A-Z
	CString result[26][2];	// Ergebnissarray
	int result_cnt = 0;		// Arrayzähler
	int drivetype;			// Laufwerkstyp
	int cnt = 65;			// Schleifenzähler

	for(cnt; cnt<(65+26); cnt++ ) {  // Wiederhole für jeden Laufwerksbuchstaben
		dl[cnt-65] = cnt;
		drivetype = GetDriveType( dl[cnt-65] + ":" );
		if (drivetype != 1){
			result[result_cnt][0] = dl[cnt-65];
			result[result_cnt][1] = drivetype;
			result_cnt++;
			dl_cnt++;
			cout << "find_used_dl(): Habe Laufwerk " << dl[cnt-65] << ": gefunden! Typ: " << drivetype << "\n";
		}
	}
	return dl_cnt;
}

Die Funktion gibt per return die Anzahl der verwendeten Laufwerke zurück.
z.B. Festplatte C: und mit CD-ROM X: gibt 3.

Nun will ich aber auch result[][] haben.

Dazu einige Verständnissfragen:

1. Was habe ich da mit [][] erzeugt?
In der Perl-Welt würde ich sagen Hash...?
Die Idee sieht so aus:
result[0][0] = "C";
result[0][1] = 3;
result[1][0] = "D";
result[1][1] = 3
result[2][0] = "X";
result[2][1] = 5;
Welchen Datentyp sollte result[][] haben? Ist CString ok?

2. Die Funktion gibt im Moment die Anzahl der gefunden Laufwerke aus.
Allergings muss result[][] ja auch ausgegeben werden.
Dass man mit return nur 1 Wert zurückliefern kann, ist mir klar.
Folglich muss ich einen Pointer als Parameter übergeben (richtig?)
Hab das mal mit 3 int-Werten getestet, das hat funktioniert, und die int Werte wurden auch in der main-Funktion geändert.
Allerdings habe ich mit einem Pointer auf ein Array nur Fehlermeldungen geerntet.
Warscheinlich weil ich keine Ahnung von den ganzen Datentypen in C++ habe... Bin eben Perl-Umsteiger und tue mich mit den ganzen Datentypen wahnsinnig schwer...

mfg

trequ

estartu

Zwei Rückgabewerte gehen nicht.
Du kannst aber mit Referenzen und Zeigern arbeiten, die du als Parameter übergibst.

Als Datentyp für das Array würde ich BYTE vorschlagen. Da drin kann man Zahlen und einzelne Zeichen speichern.
Den Schleifenzähler kannst du gleich im Kopf eintragen.
dl_cnt ist das selbe wie result_cnt - kann man also weglassen.

Belege vor dem Aufrufen den Speicherplatz für result!!!!!!!!! Das sieht genauso aus, wie du schon hattest. Der Funktion musst du nur den Namen des Array übergeben.

int find_used_dl(BYTE* result)
{
    int result_cnt = 0;        // Arrayzähler

    for(int cnt = 0; cnt < 26; cnt++ ) 
   {  // Wiederhole für jeden Laufwerksbuchstaben
        CHAR dl = cnt + 65; // Damit kriegt man den Buchstaben
        int drivetype = GetDriveType( dl + ":" );
        if (drivetype != 1)
	{
            result[result_cnt][0] = dl;
            result[result_cnt][1] = drivetype;
            result_cnt++;
            cout << "find_used_dl(): Habe Laufwerk " << dl << ": gefunden! Typ: " << drivetype << "\n";
        }
    }
    return result_cnt;
}

So, ich habe das jetzt weder getestet noch kompiliert - also sag, wenn es nicht geht.
Oder guck selbst auf dein Original, was ich da anders gemacht habe.

Nachtrag:
Da ich mich mit Zeigern auch nicht gerade gut auskenne, hier noch ein Vorschlag:
Nimm ein CArray. Ja, das ist Speicherplatzverschwendung, aber es ist einfach viel angenehmer zu handhaben.

Freundlicherweise gibt es schon ein CByteArray. Das erspart dir ein typedef.
Hier nochmal der Code mit CByteArray:

#include "afxtempl.h"

void find_used_dl(CByteArray& result)
{
    for(int cnt = 0; cnt < 26; cnt++ ) 
   {  // Wiederhole für jeden Laufwerksbuchstaben
        CHAR dl = cnt + 65; // Damit kriegt man den Buchstaben
        int drivetype = GetDriveType( dl + ":" );
        if (drivetype != 1)
	{
            result.Add(dl);
            result.Add(drivetype);
            cout << "find_used_dl(): Habe Laufwerk " << dl << ": gefunden! Typ: " << drivetype << "\n";
        }
    }
}

Nun hat das aber einen kleinen Haken: Da es kein fertiges 2D-Array gibt, musst du ebn wissen, dass ungerade Elemente der Buchstabe sind und gerade der Typ.
Aufrufen kannst du das nun so:

CByteArray arrLW;
find_used_dl(arrLW);
for (int i = 0; i < arrLW.GetSize(); i+=2) // Es wurde was gefunden
{
    AfxMessageBox(arrLW.GetAt(i) + _T(" ist ") + arrLW.GetAt(i+1));
}

Okay?

Bummsfallera

Ich persönlich würde versuchen ein struct hernehmen und nacher den Zeiger auf
dieses struct als Rückgabewert ausgeben.

trequ

Hallo,

danke für die Antwort.

@Bumsfallera:
Ich mache seit 3 Tagen C++. Vorher nur Perl/TK.
Was ist ein struct ???

@estartu_de:
Ich rufe die Funktion in der main so auf:

BYTE result[26][2];
BYTE *pointer;
pointer = &result[26][2];
int used_dl_cnt = find_used_dl(pointer);

Definiert habe ich das ganze in der .h Datei so:

int find_used_dl(BYTE*);

Die Funktion (übernommen, ohne Änderung)

int find_used_dl(BYTE* result) 
{ 
    int result_cnt = 0;        // Arrayzähler 

    for(int cnt = 0; cnt < 26; cnt++ ) 
   {  // Wiederhole für jeden Laufwerksbuchstaben 
        CHAR dl = cnt + 65; // Damit kriegt man den Buchstaben 
        int drivetype = GetDriveType( dl + ":" ); 
        if (drivetype != 1) 
    { 
            result[result_cnt][0] = dl; 
            result[result_cnt][1] = drivetype; 
            result_cnt++; 
            cout << "find_used_dl(): Habe Laufwerk " << dl << ": gefunden! Typ: " << drivetype << "\n"; 
        } 
    } 
    return result_cnt; 
}

Das ergibt diese Fehler:
<PFAD>\device.cpp(47) : error C2109: subscript requires array or pointer type
<PFAD>\device.cpp(47) : error C2106: '=' : left operand must be l-value
<PFAD>\device.cpp(48) : error C2109: subscript requires array or pointer type
<PFAD>\device.cpp(48) : error C2106: '=' : left operand must be l-value

Gibt es in C++ einen Datentyp der nach dem Thema "Halt die Klappe und mach einfach" arbeitet?
Ich versuche C++ zu lernen und jeder 2. Fehler lautet auf einen feherhaften Datentyp.
Es ist mir ehrlich gesagt §%"§%" egal, ob ein long oder double mehr Speicher addressieren, als ein int.
Mir kommt es so vor, dass es in C++ mindesten 100 Funktionen zum umwandlen von Datentypen geben muss...?
Kann doch gar nicht so schwer sein, oder?

mfg
trequ

estartu

Hi, hilft dir das Edit oben weiter? Die Antworten haben sich genau überschnitten.

Eine allgemeine Sache: In C ist der Name eines Arrays der Zeiger auf das 0. Element. Das hilft bei dem Zeigerkram weiter.

Ein struct ist eine Zusammenfassung von mehreren Daten zu einem neuen Typ.
Sowas wie ein Punkt, der ist ja auch x und y Koordinate.

Ja, es gibt viele Funktionen zum Umwandeln von Datentypen, auch wenn sich einige auch so verstehen. Schau dir mal Casts an.

Bummsfallera

ein struct kann man in etwa mit einer Tabelle vergleichen.
ihn kann man mit verschiedenen Möglichkeiten initialisieren (einfach mal nach
struct hier suchen).

z.b.

struct person
{
   CString name;
   CString vorname;
   int alter;
   int schuhgroesse;
}pos[100];

//dieser kann 100 Datensätze verwalten

//speichern wie folgt:
pos[0].name = "Wurst";
pos[0].vorname = "Hans";
pos[0].alter =66;
pos[0].schuhgroesse = 48;

angaben ohne gewähr, da ich auf der arbeit sitze und das nur nebenher tippel.

vg!

trequ

Hallo,

ich habe das jetzt eingebaut:
<PFAD>\DriveLetter.cpp(44) : error C2297: '<' : illegal, right operand has type 'int (__thiscall CByteArray::*)(void) const'

Es geht um die Zeile:
for (int i = 0; i < arrLW.GetSize; i+=2) // Es wurde was gefunden

Ich habe auch versucht es anders auszugeben:
cout << arrLW[0];
Keine Fehler beim erstellen, dafür extremer Programmabsturz...

#################

Ich glaube ich habe das mit dem struct verstanden, aber ich weiß nicht, wie ich das verwenden soll.

Mein Ziel ist es, eine Datei zu erstellen, welche alle Funktionen die ich mit den Laufwerken machen muss, beinhaltet.
Ich dachte, es reicht aus eine .cpp und eine .h Datei in das Projekt zu kopieren und die .h Datei zu includen.
Das struct müsste dann auch in die .cpp datei, oder?

Noch was zu Thema struct:
http://msdn.microsoft.com/library/en-us/devio/base/ntms_driveinformation.asp
Ist ein struct, welches in der Ntmsapi.h enthalten ist.
Ich habe daher dies probiert:

#include "Ntmsapi.h"
...
cout << _NTMS_DRIVEINFORMATION.Number;

Dies liefert jedoch:
<PFAD>\1.cpp(40) : error C2275: '_NTMS_DRIVEINFORMATIONA' : illegal use of this type as an expression
<PFAD>\include\ntmsapi.h(328) : see declaration of '_NTMS_DRIVEINFORMATIONA'
Warum?

mfg
trequ

Bummsfallera

mit struct kannste datensätze einfacher handlen, weils einfach übersichtlicher
ist und du nur 1 zeiger brauchst, um auf die entsprechenden datensätze zuzu-
greifen (*pos).

estartu

Oh Gott, da fehlt die Klammer!

Sorry!!!!!!

for (int i = 0; i < arrLW.GetSize(); i+=2) // Es wurde was gefunden

trequ

Hallo,

....logisch, die Klammern sind hilfreich. (Hätte ich auch selber draufkommen können)

Ich habe den Code jetzt umgeschrieben:

int find_used_dl(CByteArray& result)
{ 
    int		result_cnt = 0;     // Arrayzähler
	int		drivetype;			// Laufwerkstyp
    DWORD   drives;				// Verwendete Laufwerke
    char    drive[8] = "A:\\";  // Laufwerksbuchstabe

	drives = GetLogicalDrives();

    for(int i=0; i<26; i++) { 
        if(drives) {
            drive[0] = i + 'A'; 
			drivetype = GetDriveType(drive);
			if(drivetype != 1){
				result_cnt++;
				//result.Add(drive);  <<-- Ergibt Fehler
				result.Add(drivetype); 
				cout	<< "Drive: "
						<< drive
						<< " Typ: "
						<< drivetype
						<< "\n";
			}
		}
    }

    return result_cnt; 
}

Es werden bei mir 12 Laufwerke gefunden (passt zum Arbeitsplatz) auch Laufwerksbuchstabe und Typ werden richtig ausgegeben.
Allerdings funktioniert das mit dem Array immer noch nicht.
Die Zeile
result.Add(drive);
erzeugt
<PFAD>\device.cpp(53) : error C2664: 'Add' : cannot convert parameter 1 from 'char [8]' to 'unsigned char'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast

Ich habe die Zeile auskommentiert und den Wert von GetSize in der main ausgelesen. 12. Passt. Auch der return (int) hat den Wert 12.
Allerdings stehen nur Herzchen in dem Array:

CByteArray arrLW; 
		int used_dl_cnt = find_used_dl(arrLW); 
		for (int i = 0; i < arrLW.GetSize(); i+=2) // Es wurde was gefunden 
		{ 
			cout << arrLW.GetAt(i) 
				 << arrLW.GetAt(i+1)
				 << "\n"; 

		}
		cout	<< arrLW.GetSize()
				<< "\n---\n";

erzeugt:
☻

12
---
Ich werde mit dem ganzen Datentypen von C++ noch Wahnsinnig.
Gibt es keine Datentyp so nach dem Motto "Hat die Klappe und mach einfach"

mfg
trequ

Bummsfallera

gibt es gott sei dank NICHT.
entweder würde dann nur unnötig speicher verpulvert oder es gäbe Overflows am
laufenden band

da musst du wohl durch *hehe*

trequ

Der Mist hat mir grade noch gefehlt...
In Perl musste ich mir da nie große Gedanken drum machen...
Warscheinlich macht sich C++ im Lebenslauf so gut, wie alle sagen.
Aber warum ich von char nicht nach char[8] kann, verstehe ich nicht.
Was hat die [8] da zu bedeuten?

mfg
trequ

Bummsfallera

du meinst wohl

char variablenname[8];

das wäre ein array vom typ char in dem du 8 char-werte speichern kannst.

und: ja, c++ im lebenslauf isn echter kaufgrund

trequ

Hallo,

demnach ist char also ein einzelner Buchstabe...
-> Wer hat sich denn den Mist ausgedacht? Dann kann man ja gleich CString nehmen...

Sorry, aber das löst alles das Problem nicht...
Da muss ja irgend etwas falsch umgewandelt werden, die Sonderzeichen kommen ja irgendwo her...

mfg
trequ

Bummsfallera

geht nicht:

char    drive[8] = "A:\\";

geht:

char    drive[] = "A:\\";

Bummsfallera

versuch mal nen array im CString-Format, in dem du die laufwerksbezeichnungn
wie "A:\" zum schluss speicherst und ein char für den aktuellen laufwerks-
buchstaben zu ermitteln und dann in die CString incl. ":\" zu speichern

trequ

Hi,

wenn ich drive[] schreibe, kriege ich einen Fehler
"Unable to convert char[4] to char" oder so ähnlich.

Wenn ich statt CByteArray ein CString Array verwende, kriege ich lauter Zahlen anstelle von Sonderzeichen.

--> Bitte bedenken: Ich mache C++ erst seit 3 oder 4 Tagen...

Ich verstehe es einfach nicht:
In der Praxis wird man doch öfter mal ein Array von einer Funktion zurückgeben müssen.

Das kann doch nicht so schwer sein...

mfg
trequ

Bummsfallera

versuch mal einen CString-Array als membervariable einzufügen und verwende den.
du hast bei einer funktion nie einen array als rückgabewert, höchstens nen
zeiger auf diesen array.
für den laufwerksbuchstaben brauchst du keinen char-array, da nur 1 Buchstabe.
verwende ne einfache char-variable als zwischenspeicher um die dann in den
CString zu formatieren und das ":\" dranzuhängen

Bummsfallera

oder vielleicht hilft dir das weiter:

#include <string.h>
#include <stdio.h>

void main( void )
{
   char string[80];
   strcpy( string, "Hello world from " );
   strcat( string, "strcpy " );
   strcat( string, "and " );
   strcat( string, "strcat!" );
   printf( "String = %s\n", string );
}

Output:

String = Hello world from strcpy and strcat!

estartu

Hallo!

In deinem Beitrag von 16:37:24 19.08.2004 habe ich den Fehler gefunden:
Du hast den Variablentyp geändert. Ich hatte eine Variable dl vom Typ CHAR (den man problemlos in ein BYTE stecken kann) und du hast daraus einen char[8] gemacht. Das geht nicht. Diese Typen vertragen sich nicht.

Warum? Ein BYTE ist eine 8 Bit große Speicherstelle. Ein char auch. Ein char[8] sind 8 chars. Nun überleg mal, warum 8 Teile nicht auf den Platz von einem Teil passen.

Als Abhilfe kannst du drive[0] zuweisen. Das ist ja der Laufwerksbuchstabe und nur EIN Buchstabe und nicht gleich mehrere.

Du hast dir eine ziemlich anspruchsvolle Aufgabe gesucht, um dich mit C++ anzufreunden. Kann es sein, dass du den Fehler machst, dich wegen deiner schon vorhandenen Programmierkenntnisse zu übernehmen?
Du magst Pearl gut können, vielleicht sogar sehr gut - aber das heißt nicht, dass du andere Sprachen auch gut kannst. Fang langsam an oder ärgere dich nicht so sehr darüber, dass du kaum voran kommst. (Auch wenn dir ein Chef im Nacken sitzt - davon wirst du auch nicht schneller lernen.)

PS: Ohne in Büchern oder der MSDN zu lesen wirst du nicht viel Freude haben, denn es wird sich kaum jemand finden, der die Geduld hat, dir alles haarklein vorzukauen.
C ist nicht einfach, C++ ist noch viel mehr als C und die MFC ist nochmal mehr zu lernen - erkennst du nun, dass du es nicht in einer Woche verstehen wirst?
Schau ruhig mal nach, was ein Begriff bedeutet, der dir um die Ohren geschlagen wird. Wenn du nicht weißt, was eine Referenz ist, dann such die Definition, und frag nach, wenn du sie nicht verstehst. Oder arbeite dich durch ein Buch. Es gibt so viele...