.log Sortieren ??

Werner Salomon

.. wenn Du erst 2Wochen C++ hinter Dir hast, sollte man vielleicht noch erwähnen, dass der dritte Parameter von std::sort einen sogenannte Funktor aufnehmen kann, der Dir hier gute Dienste leistet.
Ein Funktor ist eine Klasse, mit einer Methode ohne Namen. Sieht so aus

class Compare // ein Funktor zum Vergleich zweier Texte
{
public:
    bool operator()( const std::string& text1, const std::string& text2 )
    {
        if( /* text1 kleiner text2 */ )
            return true;
        else
            return false;
    }
};

Ein Objekt diesen typs kannst Du dann an std::sort übergeben. Also etwa

std::sort( von, bis, Compare() );

Zum Vergleich der Texte musst Du in Deinen Fall die Zahl am Anfang auslesen, dazu benutzt man in C++ die Klasse std::istringstream.

Eine alternative Lösung besteht darin die Zahlen gleich beim Einlesen des Log-Files zu speichern. Also etwa in einer Struktur

struct Zeile
{
    int m_nummer; // Zahl am Anfang
    std::string m_restText;
};

Das Prinzip mit sort und Funktor bleibt gleich, lediglich der Typ der verglichen wird, ändert sich von std::string nach struct Zeile.

Gruß
Werner

myFile.open

mhh.. klappt alles nicht so wirklich

CStoll

Und wir sollen jetzt raten, was du geschrieben hast? Zeig doch einfach etwas Code - mit einer konkreten Frage (oder Compilermeldung), dann können wir auch weiterhelfen.

myFile.open

Nein, sollte keiner raten

Bastel ja noch

myFile.open

int main ( )
{

	int num;
	vector<string> vect;
	ifstream quelle;												

	quelle.open("text.log", ios_base::in);						// Datei öffnen

	if (!quelle){												// Datei nicht gefunden
				cerr << "no file available\n";					// Fehlerausgabe
				}	

	else		{

				quelle.getline (num, 4)

					// wie nun richtig an vector übergeben? (mit push_back)

so bin Mittag machen, Mahlzeit!

und danke für alles :p

CStoll

Erstmal: getline() mit einem int zu verwenden geht schonmal nicht - die istream-Methode arbeitet mit char-Arrays, die globale Funktion mit strings.

Zweitens: getline() liest genau eine Zeile ein - um die gesamte Datei zu bekommen, mußt du es in einer Schleife aufrufen.

string zeile;
while(getline(quelle,zeile))//das liest so lange, bis es auf Fehler bzw. EOF trifft
  vect.push_back(zeile);

Drittens: Ja, push_back() ist richtig.

myFile.open

beim getline wollte ich tricksen, aber hab gemerkt, dass es 1.nicht geht und 2.schwachsinn war - danke

edit// oja, bestens. hat soweit funktioniert

jetzt mach ich mich mal an das sortieren

#include <iostream>
#include <string>
#include <fstream>
#include <vector>
#include <algorithm>

using namespace std;

struct entry
{
   int nr;
   string log;
};
class Compare
{
public:
    bool operator()( const entry& en1, const entry& en2 )
    {
        return en1.nr < en2.nr;
    }
};
int main()
{
    ifstream in("log.txt");
    string buff;
    entry neueintrag;
    vector <entry> log;
    while(!in.eof())
    {
      getline(in,buff,'.');
      neueintrag.nr = atoi(buff.c_str());
      getline(in,buff);
      neueintrag.log = buff;
      log.push_back(neueintrag);
    }
    in.close();
    sort(log.begin(),log.end(),Compare());
    for(unsigned i=0;i<log.size();i++)
    {
      cout << log[i].nr << " " << log[i].log <<endl;
    }
    cin.get();
    return 0;
}

myFile.open

ouh, sehr nice.

danke vielmals, war schon fast zu viel des guten..

funzt aber nicht zu 100%

ich bastel mal weiter

Werner Salomon

myFile.open schrieb:

funzt aber nicht zu 100%

wo ist denn das Problem?

Wenn Du fleißig übst, Dich auch in die Geheimnisse der STL einarbeitest, dann kannst Du bald auch solche Programme hinschreiben.

#include <iostream>
#include <fstream>
#include <string>
#include <vector>
#include <algorithm>    // sort
#include <iterator>     // i/ostream_iterator<>

struct entry
{
    int m_nummer;
    std::string m_restText;
};
// --   Entry lesen
std::istream& operator>>( std::istream& in, entry& e )
{
    return std::getline( in >> e.m_nummer, e.m_restText );
}
// --   Entry schreiben
std::ostream& operator<<( std::ostream& out, const entry& e )
{
    return out << e.m_nummer << e.m_restText << std::endl;
}

class Compare
{
public:
    bool operator()( const entry& en1, const entry& en2 ) const
    {
        return en1.m_nummer < en2.m_nummer;
    }
}; 

int main()
{
    std::ifstream quelle( "log.txt" );
    if( !quelle.is_open() )
    {
        std::cerr << "Fehler beim Oeffnen der Datei" << std::endl;
        return -1;
    }
    std::vector< entry > log( (std::istream_iterator< entry >( quelle )), (std::istream_iterator< entry >()) );
    std::sort( log.begin(), log.end(), Compare() );
    std::copy( log.begin(), log.end(), std::ostream_iterator< entry >( std::cout ) );
    return 0;
}

Du siehst das Lesen, sortieren und ausgeben ist jeweils eine Zeile Code. Die kleinen Helferlein nicht mitgerechnet

Gruß
Werner

lässt sich aber leider nicht compilieren.
Hängt in der Zeile

std::vector< entry > log( (std::istream_iterator< entry >( quelle )), (std::istream_iterator< entry >()) );

(Compiler meldet Ausdruckssyntax)

Werner Salomon

lern_ender schrieb:

lässt sich aber leider nicht compilieren.

welchen Compiler benutzt Du?

falls es eine Version 6 einer bekannten großen Softwarefirma ist, so geht es nicht. Ändere dann die Zeile 41 nach

std::vector< entry > log;
    std::copy( (std::istream_iterator< entry >( quelle )), std::istream_iterator< entry >(), std::back_inserter( log ) );

Gruß
Werner

Artchi

Ich würde nicht die Zeile ändern, sondern die Versionsnummer des Compilers ändern.

Artchi schrieb:

Ich würde nicht die Zeile ändern, sondern die Versionsnummer des Compilers ändern.

Würde ich ja gerne. Leider macht mein Chef da nicht mit.
Warum hat der BCB denn damit ein Problem ?

Artchi

Weil er wahrscheinlich nicht gerade sehr ISO-C++-konform zu sein scheint. Verrate uns doch mal deinen Compiler und Versionsnummer...

Borland Builder 6 (Build 10.166)

myFile.open

sehr nett leute, danke