Registryeintrag mit zwei Werten lesen

CStoll

Da fehlen aber noch einige Grundlagen, bevor der Programmtext halbwegs lesbar ist.

(btw, solche Ausdrücke wie "DaValuePart = ("%s",value+s);" sind unsinnig - das ist identisch zu "DaVAluePart = value+s;")

mrkorn

CStoll schrieb:

Da fehlen aber noch einige Grundlagen, bevor der Programmtext halbwegs lesbar ist.

Ja ich weiß, aber ich hab momentan keine Zeit um die vor Jahren "eigentlich" schonmal erlernten Grundlagen aufzufrischen! Geht bei mir gerade nur durch learning-by-doing...
Und solange ich in der Lage bin meinen Code zu verstehen, passt das erst mal...

CStoll schrieb:

(btw, solche Ausdrücke wie "DaValuePart = ("%s",value+s);" sind unsinnig - das ist identisch zu "DaVAluePart = value+s;")

Guat da haste natürlich auch wieder mal recht.

mrkorn

Moin

es hat sich noch ein kleines Problem aufgetan, und mit meinen mangelnden Grundlagen hab ichs noch nicht hinbekommen.
Und zwar bekomme ich eine Warnung:
warning C4172: returning address of local variable or temporary

Jetzt weiß ich nicht wie ich die Funktion umschreiben muss, damit er mir einfach ein char zurück gibt. Kann man mir noch helfen?

Wenn ich anfange die Funktion als "char Registrylesen(...)" zu schreiben, dann zieht das einen riesigen Rattenschwanz nachsich. Gibt es vielleicht eine einfachere Lösung als die ganze Funktion umzuschreiben?

Gruß

CStoll

Wie gesagt, da fehlen Grundlagen - char ist nur ein einzelnes Zeichen (also zu klein für den kompletten Registry-Eintrag) und dein char DaValue[MAX_PATH]; ist eine lokale Variable, die am Funktionsende wieder zerstört wird (das ist es, was die Warnung dir sagen will - du gibst einen Zeiger auf etwas zurück, was nicht (mehr) existiert).

Eine einfache Lösung für das Problem wäre es, den zurückzugebenden Speicher per new[] anzufordern (aber dann muß das Hauptprogramm ihn aufräumen) oder als Parameter an die Funktion zu übergeben. Noch eleganter ist es natürlich, mit CString oder std::string zu arbeiten.

mrkorn

jaja die grundlagen...

Also mit CString oder std::string hatte ich bis jetzt noch nix mit zu tun.
Aber die Möglichkeit einen Parameter mit der Funktion zu übergeben hört sich interessant an.

Ich nehm mal an das meine Funktion dann keinen Rückgabewert mehr gibt. Ich aber beim Funktionsaufruf noch eine Variable mitgebe, die ich dann in der Funktion fülle.
Etwa in diese Richtung:

void RegistryLesen(char * Rückgabewert, HKEY MainKey, DWORD Reg_Type, LPCWSTR Subkey, LPCWSTR Value)

Liegt ich da soweit richtig? Werds gleich mal testen...

CStoll

mrkorn schrieb:

jaja die grundlagen...

Also mit CString oder std::string hatte ich bis jetzt noch nix mit zu tun.
Aber die Möglichkeit einen Parameter mit der Funktion zu übergeben hört sich interessant an.

Die Lösung dazu ist eigentlich ganz einfach - du änderst den Rückgabetyp der Funktion auf CString und lässt den Rest so, wie er ist (im nächsten Schritt wäre es vielleicht hilfreich, die ganzen <cstring>-Funktionen durch Operator-Aufrufe zu ersetzen).

Ich nehm mal an das meine Funktion dann keinen Rückgabewert mehr gibt. Ich aber beim Funktionsaufruf noch eine Variable mitgebe, die ich dann in der Funktion fülle.
Etwa in diese Richtung:
void RegistryLesen(char * Rückgabewert, HKEY MainKey, DWORD Reg_Type, LPCWSTR Subkey, LPCWSTR Value)
Liegt ich da soweit richtig? Werds gleich mal testen...

Ja, das sieht gut aus (eventuell sollte das Hauptprogramm auch noch dazusagen, wieviel Platz hinter "Rückgabewert" tatsächlich zur Verfügung steht). Allerdings mußt du nicht unbedingt void zurückgeben, sondern kannst die Rückgabe auch für Fehler- und Statuswerte oder z.B. die Länge des verfügbaren Strings nutzen.

mrkorn

Also ich hab da mal ein kleines Beispiel für CSrings aus dem Internet versucht:
CString gray("Gray");
CString cat("Cat");
CString graycat = gray + cat;

Bringt mir einige Fehler:
error C2065: 'CString' : undeclared identifier
error C2146: syntax error : missing ';' before identifier 'gray'
error C2065: 'gray' : undeclared identifier
error C2146: syntax error : missing ';' before identifier 'cat'
error C2065: 'cat' : undeclared identifier
error C2146: syntax error : missing ';' before identifier 'graycat'
error C2065: 'graycat' : undeclared identifier

Muss ich dafür noch eine bestimmte Headerdatei includen? (hab auf einigen Seiten geschaut, und keiner gibt ne Headerdatei an)

--------------------------------------------------

Wenn ich die Funktion so wie sie beim letzten post war aufrufe, dann muss ich als ersten Parameter auch wieder eine andere Variable übergeben? Habs so mal versucht:

char * DNSback;
RegistryLesen(DNSback, HKEY_LOCAL_MACHINE, REG_MULTI_SZ, L"Comm\\DM90001\\Parms\\TcpIp", L"DNS");
printf("UEBERGEBEN DNS: %s\r\n",DNSback);

Kompiliert ohne Fehler, aber in meiner Ausgabe mit printf erscheint keinerlei Inhalt!?

CStoll

mrkorn schrieb:

Also ich hab da mal ein kleines Beispiel für CSrings aus dem Internet versucht:
CString gray("Gray");
CString cat("Cat");
CString graycat = gray + cat;

Bringt mir einige Fehler:
error C2065: 'CString' : undeclared identifier
error C2146: syntax error : missing ';' before identifier 'gray'
error C2065: 'gray' : undeclared identifier
error C2146: syntax error : missing ';' before identifier 'cat'
error C2065: 'cat' : undeclared identifier
error C2146: syntax error : missing ';' before identifier 'graycat'
error C2065: 'graycat' : undeclared identifier

Muss ich dafür noch eine bestimmte Headerdatei includen? (hab auf einigen Seiten geschaut, und keiner gibt ne Headerdatei an)

Ja, da brauchst du den Header <afx.h> (siehe MSDN) - für std::string dann natürlich <string>.

Wenn ich die Funktion so wie sie beim letzten post war aufrufe, dann muss ich als ersten Parameter auch wieder eine andere Variable übergeben? Habs so mal versucht:
char * DNSback;
RegistryLesen(DNSback, HKEY_LOCAL_MACHINE, REG_MULTI_SZ, L"Comm\\DM90001\\Parms\\TcpIp", L"DNS");
printf("UEBERGEBEN DNS: %s\r\n",DNSback);
Kompiliert ohne Fehler, aber in meiner Ausgabe mit printf erscheint keinerlei Inhalt!?

Daß du überhaupt bis zum printf() kommst, ist ja erstaunlich - das Hauptprogramm muß natürlich auch genug Speicher bereitstellen, wo die Daten reingepackt werden können (entweder als char-Array oder über malloc()/new[]), sonst schreibst du irgendwo in den Speicher.

mrkorn

Nachdem ich <afx.h> und <string> eingebaut habe, bekomme ich diese Fehler:
c:\programme\windows ce tools\wce500\standardsdk_500\mfc\include\wcealt.h(245) : error C2084: function 'void *__cdecl operator new(unsigned int,void *)' already has a body
c:\programme\windows ce tools\wce500\standardsdk_500\mfc\include\wcealt.h(251) : error C2084: function 'void __cdecl operator delete(void *,void *)' already has a body

CStoll

Das ist seltsam - was für Header verwendest du denn sonst noch?

mrkorn

Das sind alle:
#include <windows.h>
#include <stdio.h>
#include <winioctl.h>
#include <stdlib.h>
#include "httpext.h"
#include "creg.hxx"
#include <string>
#include <afx.h>