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

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?

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

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.

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!

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

Sorry!!!!!!

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

....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

da musst du wohl durch *hehe*

mfg
trequ

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

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

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

geht:

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

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

#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!

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...