C++ code ausbessern?

Vorab: Ist nicht dringend weil ich mit dem coden wieder aufhören muss. Habe vor 2 Tagen angefangen (nur c++). Wie würdet ihr das anders machen?

Ich bin mir sicher, dass man die Definitionen leichter hätte machen können, nur weiß ich nicht genau wie.

Bild zum Beitrag

Habe die Funktion nicht als main, weil ich noch was anderes probieren wollte, also der „Fehler“ ist mir bewusst.

1 Antwort

regex9

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Cplusplus, Code, Programmiersprache

19.12.2023, 23:42

Mehrere der deklarierten Variablen (kilometer, kilometers, Km) werden gar nicht genutzt. Sie können also raus.
Zuweisungen wie kilometers zu kilometer, miles zu mile oder mile zu input (der Wert wird eh darauffolgend wieder überschrieben) sind ohne Sinn.
C++ hat explizit Datentypen (wie std::string), damit man C-Typen (wie const char*) nicht mehr wirklich braucht.
Einige der Strings bei deinen Ausgaben (z.B. "mi" und " = " könnten direkt zu einem Literalwert verbunden werden. Das würde den Code an der Stelle auch einfacher lesbar gestalten.
Bezüglich der Benamung deiner Variablen wäre es besser, einen einheitlichen camelCase-Stil zu wählen (km, factorKm). Der Name deiner Funktion sollte eindeutiger sein, sodass man bestenfalls ihren Körper nicht erst lesen muss, um zu verstehen, was sie tut. Auch der input-Variable würde ich einen eindeutigeren Namen spendieren (z.B. numberUnit).
Der Rückgabetyp der Funktion ist (logisch betrachtet) falsch. Sie tut nichts, daher wäre void richtig. Die main-Funktion sollte stattdessen einen expliziten Rückgabewert zurückgeben.
Die Bedingung input == mile ist nur wahr, wenn der Nutzer für input nichts eingibt.

Seal6

Fragesteller

20.12.2023, 07:20

mile und miles hab ich geschrieben, dass es egal ist ob man 1mile oder 100miles eingibt.

regex9

20.12.2023, 10:03

@Seal6

Dafür müsstest du input mit entsprechenden Werten vergleichen:

if (input == "mile" || input == "miles") {

Deine Variablen mile und miles hingegen zeigen auf leere Strings.

string mile;
cout << mile; // empty output ("")

Wenn du mit ihnen arbeiten wolltest, müsstest du ihnen entsprechende Werte geben und sie dann in die Prüfung einbauen.

string mile = "mile";
string miles = "miles";

if (input == mile || input == miles) {
  // ...
}

Hallo Leute, ich habe gehört, dass man bei Programmiersprachen sich beim Code ziemlich verbocken kann mit Memory Leaks und anderen Risiken beim unsicheren Code.

Also ist es eine bessere Idee in einer Virtuelle Maschine zu Coden, falls zu solchen Probleme kommt, dass man ein Backup macht, die virtuelle Maschine löscht und dann eine neue Maschine aufsetzt.

Mir ist es bewusst, dass virtuelle Maschine auch Performanceeinbrüche haben und wenn man kein guten PC/Laptop hat, ist sowas schon unmöglich. Und falls man eine Grafikkarte braucht, falls man ein rechenintensives Spiel programmiert, bräucht man 2 davon wenn man QEMU benutzt, bei Virtualbox ist es unmöglich und Virtualbox wäre aus meiner Sicht auch kaum geeignet dafür.

Was denkt ihr dazu?

...zur Frage

Welche ist die beste App zum Coden am iPad?

Ich habe angefangen an Pc zu coden (mit vs code) doch ich würde doch auch gerne mal am IPad Programmieren. Da es keine vs code app giebt wollte ich fragen welche app ihr empfehlen würdet.

...zur Frage

Programmiersprachen Schnelligkeit?

Ich weiß das man das nicht so vergleichen kann. Aber allgemein wie ist das Ranking dieser Sprachen anhand der Geschwindigkeit: C, C++, C#, Rust;. Und sollte man Rust lernen?

...zur Frage

C++ Void Funktionen? Ausgabe ohne cout?

Halli Hallo, Freunde. Vielleicht finde ich den Code noch, aber wir hatten mal eine Funktion, wo im Void keine Ausgabe war, die Funktion jedoch auch ohne std::cout eine Map geprintet hat. Die Frage kam bei meiner mündlichen Prüfung. Ich hatte jedoch keine Ahnung. Die Funktion hatten wir in einer cpp datei definiert, in der hpp als private member function deklariert, wenn ich mich nicht irre und in der main hatte man die Kommandozeilenargumente.Ich meinte es könne mit der main zusammenhänge.Mein Professor meinte es habe mit der Funktion an sich zu tun... In welchem Szenario kann den bitte eine void Funktion was printen ohne std:: cout?

Bitte helfen!

...zur Frage

brauch ich bei malloc immer ein free() in C?

Sagen wir ich habe eine Funktion func in der erstelle ich ein array mithilfe von malloc ohne es freizugeben da ich dieses array zurückgeben will. Demnach kann ich doch dieses array nur in der funktion freen in der es sozusagen ankommt oder?

...zur Frage

"cout" ist keine Member von "std"?

Hy,

ich wollte mit c++ anfangen, jedoch leider habe ich schon hier ein Problem. Der folgende Code ist laut VS absoluter Müll, aber wieso?:

#include <iostream>

int main()
{
	std::cout << "Hello, world!";
	return 0;
}

...zur Frage

Kann man mit c++ ingame Trainer(Hacks, Cheats)? erstellen?

Moin moin, bitte beachtet nicht meine Rechtschreibung ich bin nicht der beste darin.

Meine frage ist oben schon beschrieben, kann man mit C++ in games hacks kreieren.

Diese frage stellt sich auch nicht aus dem nicht, ich habe nämlich auch nicht vor dies zu tun denn da gehören natürlich auch sachen wie reverse Engineering dazu.

Ich bin aber momentan dabei fleißig zu Lernen (auch schon etwas länger) wie Lua und C++. Und dann gucke ich halt auch mal solche sachen nach, halt aus der neugier.

Dann gab es auch momente in den ich gelesen habe das leute in fivem anstatt mit Lua nun mit C# hacks/mod menüs machen wo sich mir die frage stellt wieso?? C# ist doch eher gut fürs framework und wieso dann von lua weggehen?

Letztendlich ist die ENDFRAGE, was wäre denen nun die beste sprache für sowas?
das man mit c++ hacks etc. machen kann weiß ich. Aber wieso andere sprachen.

Danke fürs durchlesen, und wie schon gesagt. ICH WILL NICHT HACKEN/CHEATEN oder sonstiges sonder es ist rein aus Neugier.

...zur Frage

Wie soll ich das denn jetzt Handeln?

Hallo. Ich hoffe die Programmiererschaft ist aus ihrem kleinem Schläfchen erwacht und kann mir hier etwas weiterhelfen. Ich mache (immernoch) eine Coding-Challenge. Dachte sie ist Lustig, langsam wird sie Sinnlos .__.

Erstmal die zurzeitige Stage:

An sich wurde alles gesagt. Das Problem mit den max. 10 Packages hab ich bereits gelöst. Nun bin ich aber bei der byte abfrage ... Die ist dann schon Lustiger. Hier mal meine add Funktion.

void add(Package package) {
  // Check if package already in map.
  std::list < std::string > info_list({
    package.name,
    std::to_string(package.versionMajor),
    std::to_string(package.versionMinor)
  });
  if (saved_packages.count(info_list)) {
    return;
  }
  uint32_t used_size = 0;
  for (it_helper = saved_packages.begin(); it_helper != saved_packages.end(); it_helper++) {
    used_size += it_helper -> second.size;
  }
  // INFO: saved_packages.size() starts (idk why) by 2. So 12 means 10 packages.
  if (saved_packages.size() >= 12 && used_size >= 100) {
    throw std::exception();
  }
  saved_packages.insert(std::pair < std::list < std::string > , Package > (info_list, package));
}

Simpel ... Naja. Also das && falsch ist, dass weiß ich. Wollte es nur damit mal ausprobieren. Also bei "&&" kommt folgendes raus:
"The add method threw no exception despite exceeding the limit of 10 packages." und es macht Sinn. Schließlich war das Maximum erreicht, die bytes aber noch frei. Dennoch sollte man es ja nicht zulassen. Wenn ich jetzt aber || mache:

"The add method threw an exception before reaching the limit of 100 bytes." was auch Sinn macht. Denn die 10 Packete wurden vermutlich gesendet, bevor die 100 Bytes erreicht waren. Nun ... Was jetzt? Ich hab tatsächlich keine einzige Idee gerade, währe dankbar für bisschen Hilfe. Danke im Vorraus und lg.

...zur Frage

Wie setzt ihr eure Klammern beim Programmieren?

Option 1:

int main() {
  printf("Hello World!");
}

Option 2:

int main()
{
  printf("Hello World!");
}

...zur Frage

Was programmiert ihr am Liebsten?

Was programmiert ihr für Anwendungen an Liebsten. Ich z.B: Apps obwohl ich es nicht kann und Programmiersprachen in C++/C#

...zur Frage

c++ gute IDE die nicht so groß sind?

Ich suche gute Compiler, die nicht so groß sind außer visual studio. Denn visual studio ist für viel zu groß.

...zur Frage

In C ein unendliches Array?

das hier: char *array;

ist doch ein länge mit einer unendlichen länge, weil normalerweise macht man ja:

char array[10];

oder char array[ ] = "abcdefgehi";

wie kann ihc bei char *array die länge festlegen?

...zur Frage

VS Code Einstellunegn?

Hallo! Für die Uni benötige ich ein lauffähiges VS Code-Programm mit C++20. Leider verzweifle ich mittlerweile. Ich habe so viele verschiedene Tutorials angeschaut, dass ich das Gefühl habe, in sämtlichen Einstellungen herumgedoktert zu haben, und jetzt funktioniert nichts mehr. Beim Ausführen des Programms (#include <iostream> int main() { std::cout << "Hello World!" << std::endl; return 0; }) mit dem Namen test.cpp erhalte ich folgende Fehlermeldungen. Im Anhang sind die JSON-Dateien sowie die Fehlermeldung beim Debuggen zu sehen. Der Compilerpfad ist C:\msys64\ucrt64\bin und ist auch so unter den Systemeigenschaften eingestellt.

...zur Frage

Wie kriege ich die Byte Anzahl eines arrays in C raus?

also char *array = malloc(sizeof(char)*13);

ich hab dann mit sizeof(array) geguckt und mit printf ausgegben, und siehe da es kommt 8 raus, also wie geht es richtig? anscheinend gibt es hier die bit pro array eintrag aus, hab auch befüllt gehabt, kam das gleiche raus

...zur Frage

Was möchtest Du wissen?

Deine Frage stellen