Kleines Progrämmchen, was sagt ihr dazu?

[Panke]

Meint ihr alles in 8 Zeilen oder nur die Main Funktion?

Tante Edith:

Soo, gerade eben mal rumgetüftelt. Kleiner ist bei mir nicht drin:

#include <cstdlib>
#include <iostream>
#include <fstream>
#include <string>

using namespace std;

int main(int argc, char *argv[])
{
	cout << "Quelle: ";
	string quelle;
	cin  >> quelle;

	ifstream in(quelle.c_str(), ios_base::binary);

	cout << "Ziel: ";
	string ziel;
	cin >> ziel;

	ofstream out(ziel.c_str(), ios_base::binary);

	cout << "Key: ";
	string key;
	cin  >> key;

	int i=0;
	char c = in.get();
	while(!in.eof())
	{

		c = c^key[i];
		out << c;
		if(i<=key.size())
			++i;
		else i=0;
		c = in.get();
	}

    system("PAUSE");
    return EXIT_SUCCESS;
}

[volkard]

Panke schrieb:

Soo, gerade eben mal rumgetüftelt. Kleiner ist bei mir nicht drin:

uih! das ist aber hübsch geworden.

#include <cstdlib> //weg
#include <iostream>
#include <fstream>
#include <string>

using namespace std;

int main(int argc, char *argv[]) //woziu die parameter?
{
	cout << "Quelle: ";
	string quelle;
	cin  >> quelle;

	ifstream in(quelle.c_str(), ios_base::binary);

	cout << "Ziel: ";
	string ziel;
	cin >> ziel;

	ofstream out(ziel.c_str(), ios_base::binary);

	cout << "Key: ";
	string key;
	cin  >> key;

	int i=0; //aua
	char c = in.get(); //aua
	while(!in.eof())
	{

		c = c^key[i];
		out << c;
		if(i<=key.size())
			++i;
		else i=0;
		c = in.get();
	}

    system("PAUSE"); //weg
    return EXIT_SUCCESS;
}

irendwas an der schleife stört noch.
sagen wir mal, i sei nötig. und daß wir nicht den modulo-operator wollen una uch kein ?: oder -=bool(...).
dann ist

if(i<=key.size())
			++i;
		else i=0;

schon beinahe so gut wie

++i;
if(i==key.size())
   i=0;

beim enlesen nimmt man gerne

while(!in.get(c))

außerdem würde ichg nicht auf c rumschreiben, sondern einfach nur char(c^key[i]) ausgeben.

schauste mal, ob was von meiner neuerlichen mecker einbauebar ist?

und dann mußte halt schauen, ob du wirklich so stark reduzierten code gerne hast, oder leiber etwas mehr schreibst. vielleicht strebste ja auch mal eine stellung an, wo man nach anzahl der codezeilen entlohnt wird (soll's ja geben). oder willst spieleprogrammierer werden, da gehört codebloat zu den grundtugenden.

[volkard]

die 6 zeilen waren eine nicht sinnvoll erreichbare vorgabe. da hab ich etwas übertrieben, damit du in die richtige richtung läufst.

[???????????]

und wieviel zeilen sind sinnvoll?

[Panke]

++i;
if(i==key.size())
   i=0;

Nutze ich denn da das letzte Zeichen des Strings?

Wenn ich

system("Pause");

weglasse, schließt das Konsolenfenster immer sofort.
Ist der Befehl so grausam, dass sich da etwas eigenes schreiben lohnt?

Wenn deine If Abfrage funktioniert, könnt ich mir folgendes vorstellen, um das i lokaler zu halten:

for(int i = 0; !in.get(c); ++i)
{
    out << c^key[i];
    if(i==key.size()) //nicht eher i>key.size()?
        i=0;
}

Die Argumente stammen noch aus der Compilervorlage. Habe sie einfach nicht gelöscht.

[Panke]

??????????? schrieb:

und wieviel zeilen sind sinnvoll?

Schätze mal, Programm ist klein, schnell und erfüllt den erdachten Zweck ohne an Lesbarkeit zu verlieren.

Edit: Okay .. daneben

[volkard]

??????????? schrieb:

und wieviel zeilen sind sinnvoll?

besonders schöne programme haben höchstens 250 zeilen, das weiß ja jeder. man kann zufälligerweise besonders schöne programme nie in mehrere kleinere dateien aufteilen, so daß sets eine kleine datei eine zeile weniger als eine andere kleine datei hat. aber man kann besonders schöne programme auf x verschiedene weisen in kleinere dateien mit der gleichen zeilenanzahl aufteilen, wobei x immer gerade ist. ist das nicht seltsam?
das gesuchte programm ist ein besonders schönen programm. wenn ich dir nun sage, daß es auch besonders schöne programme gibt, die größer als das gesuchte programm sind und welche, die kleiner sind, isses klar, wieviele zeilen es hat.
viel spaß beim knobeln.

[niemand]

In Deiner Originalversion war genau an der Stelle ein Fehler den volkard korrigiert hat

out << c^key[i];
 if(i<key.size())  // i <= key.size() wäre falsch
    ++i;
 else i=0;

key[key.size()] ist wohl immer '\0' aber eigentlich nicht zulässig. Ausserdem macht das mit c^'\0' auch nicht so viel Sinn.

volkard hat diesen Fehler mit korrigiert:

out << c^key[i];
++i;
if(i==key.size())
   i=0;

Jetzt ist i immer kleiner key.size().

Ausserdem ist

i = (i + 1) % key.size();

noch kürzer, erfüllt den Zweck und ist schlechter lesbar...

[Panke]

Das c++ strings \0 terminiert sind, wusste ich gar nicht ..
Dann geht das natürlich so wie bei mir nicht, auch wenn es sich nicht wirklich ausgewirkt hat.

i = (i + 1) % key.size();

Dann doch lieber ein paar Zeilen mehr.

[chris' 1]

huhu,
@Panke:
womit hast du C++ gelernt/lernst du?

[Panke]

Bücher aus der örtlichen, allerdings schlecht bestückten Bücherei.
Ich empfehle auf jeden Fall mehrere zu lesen, da sie unterschiedliche Bereiche unterschiedlich gut abdecken und ein bisschen Wiederholung sicherlich nicht schadet. Dann gibts hie und da noch ein paar Tuts und Ebooks (klick mal oben auf "Tutorials" ) die einem sicherlich auch bei bestimmten Problemen weiterhelfen. Bei den file streams z.B. musste ich auch erstmal bisschen Lesestoff zusammensuchen, damit's dann klappt.

Edit: Ach ja, und jetzt versuche ich es, indem ich Sachen wie das hier programmiere und das Wissen, das mir dazu fehlt, einfach zusammensuche, wie ich es gerade brauche.

[volkard]

Panke schrieb:

Das c++ strings \0 terminiert sind, wusste ich gar nicht ..

das wußte bisher keiner. steht nichtmal im standard, so geheim war das.

[terraner]

zum code: anstatt jedesmal ein neues string-objekt zu erstellen, könntest du das alte auch einfach überschreiben.

[terraner]

niemand schrieb:

key[key.size()] ist wohl immer '\0' aber eigentlich nicht zulässig.

also erstens sind string nicht nullterminiert, wie volkard gerade schon gesagt hat, und zweitens wäre dann höchstens key[strlen(key) -1] == '\0'.

[niemand]

Ganz so einfach ist es nicht. Wenn schon dann Standard dann richtig:

21.3.4 basic_string element access [lib.string.access]
const_reference operator[](size_type pos) const;
reference operator[](size_type pos);
1 Returns: If pos < size(), returns data()[pos]. Otherwise, if pos == size(), the const version returns charT(). Otherwise, the behavior is undefined.

C++ String müssen nicht 0-terminiert sein. Der Standard verlangt dies nicht. Allerdings besteht die Möglichkeit, das die STL Implementation Deiner Wahl, hinter den String ein '\0' macht. Damit ist die Implementation von string::c_str() sehr einfach, ausserdem lässt sich so obige Regel auch einfach erledigen. Damit besteht die Chance, dass s[s.size()] == '\0' ist, obwohl der Zugriff im nicht const-Fall undefiniert ist. Verlassen sollte sich keiner darauf. Aufgrund obiger (komplizierter) Regel würde ich s[s.size()] meiden (obwohl es nicht immer ein Fehler sein muss).

EDIT: komma entfernt
EDIT2: den beim entfernen eingebauten Tippfehler beseitigt

[Panke]

Danke!

[the truth]

du hast talent

[Taurin]

Panke schrieb:

Meint ihr alles in 8 Zeilen oder nur die Main Funktion?

Alles mit drum und dran. Hab aber bei der Eingabe geschummelt

1 #include <fstream>
2 int main (int argc, char *argv[]) {
3     std::ifstream in(argv[1]);    // erstes Argument:  Eingabedatei
4     std::ofstream out(argv[2]);   // zweites Argument: Ausgabedatei
5     unsigned counter = 0;         // aktuelle Position im key-string (mod strlen)
6     char     ch = 0;              // aktuelles Zeichen
7     while(in.get(ch)) out.put(ch ^ (argv[3])[(counter++)%strlen(argv[3])]); 
8 }

Aufruf mit: Progarmmname <input> <output> <key>

[Herr Neumann]

Nicht schlecht Panke, wie lange programmierst du schon?

[Panke]

Taurin: Hübsch Wär für mich vor diesem Thread aber kaum lesbar gewesen.

Neumann:
Nicht lange. Zwei Monate? Vorher aber schon mit MSW Logo im Informatikunterricht gespielt.