lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0
 wlan0:       0       0    0    0    0     0          0         0        0       0    0    0    0     0       0          0
  eth0: 39072289   41342    0    0    0     0          0      3348  4363113   28697    0    0    0     0       0          0

d.h. von lo gibt es einmal die ganzen Daten dann von wlan0 und so weiter. Nun kann es aber auch sein das es nur 2 gibt oder auch 4. D.h. ich hab mir eine Funktion nextNet gebaut allerdings klappt die nicht so ganz.

bool SYSInfo::nextNet(){
  if(dev.empty()) return false;
  if(dev < 0){
    dev=dev.at(0);
    read_dev();
    return true;
  }
  std::vector<face>::iterator it;
  int i = 0;
  for ( it=dev.begin(); it < dev.end(); it++ ) {
    if(*it==dev && i + 1 < (int)dev.size()){
      dev=dev.at(i + 1);
      read_dev;
      return true;
    }
    i++;
  }
  return false;
}

-wenn dev leer = false
-wenn dev kleiner als 0
-dev auf die postion N ( in diesem fall 0)
-read_dev funktion
-gebe true zurück

-(it auf begin setzen / wenn it kleiner ist als dev.end / it soll um einen erhöht werden
-wenn it auf dev zeigt (false) i + 1 (true) kleiner als anzahl der elemente in dev
-setze dev auf position N (i + 1)
- read_dev funktion
-gebe true zurück

-i um 1 erhöhen

(hab mir gedacht so seht ihr ob ich es verstanden hab oder ob ich da was falsch eingebaut hab.)

so sieht mein vector aus:

struct face{ 
    std::string name; 
    unsigned int bytes_received; 
    unsigned int packets_received; 
    unsigned int errs_received; 
    unsigned int drop_received; 
    unsigned int fifo_received; 
    unsigned int frame_received; 
    unsigned int compressed_received; 
    unsigned int multicast_received; 
    unsigned int bytes_transmitted; 
    unsigned int packets_transmitted; 
    unsigned int errs_transmitted; 
    unsigned int drop_transmitted; 
    unsigned int fifo_transmitted; 
    unsigned int colls_transmitted; 
    unsigned int carrier_transmitted; 
    unsigned int compressed_transmitted; 
  };

class ....

....

  private:
    std::vector<face> dev;

read_dev:

void SYSInfo::read_dev(){ 
  std::ifstream file("/proc/net/dev"); 
  file >> skip_line; // Überspringe erste Zeile. 
  file >> skip_line; // Überspringe zweite Zeile. 
  for(face f; file>>f; dev.push_back(f)); 
}

Der Fehler ist das er meine Operatoren nicht anerkennt. Ich denke das passiert durch meine Operatoren überladung oben:

std::istream& skip_line(std::istream& in){ 
  return in.ignore(std::numeric_limits<std::streamsize>::max(),'\n'); 
} 

std::istream& operator>>(std::istream& in, face& f){ 
  return in 
  >> f.name 
  >> f.bytes_received >> f.packets_received >> f.errs_received >> f.drop_received >> f.fifo_received >> f.frame_received >> f.compressed_received >> f. multicast_received         >> f.bytes_transmitted >> f.packets_transmitted >> f.errs_transmitted >> f.drop_transmitted >> f.fifo_transmitted >> f.colls_transmitted >> f.carrier_transmitted >> f.compressed_transmitted; 
} 

std::ostream& operator<<(std::ostream& out, const face& f){ 
  return out 
  << f.name << "Bytes: " 
  << f.bytes_received    << " Packets: " << f.packets_received    << " Errs: " << f.errs_received       << " Drop: " << f.drop_received       << " Fifo: " 
  << f.fifo_received     << " Frame: " << f.frame_received      << " Compressed: " << f.compressed_received << " Multicast: " << f. multicast_received << " Bytes: " 
  << f.bytes_transmitted << " Packets: " << f.packets_transmitted << " Errs: " << f.errs_transmitted    << " Drop: " << f.drop_transmitted    << " Fifo: " 
  << f.fifo_transmitted  << " Coll: " << f.colls_transmitted   << " Carrier: " << f.carrier_transmitted << " Compressed: " << f.compressed_transmitted; 
}

Muss ich mir jetzt für jeden Operator etwas schreiben? oder kann man das irgendwie umgehen? Hier noch die Fehlermeldung:

131:12: Fehler: keine Übereinstimmung für »operator<« in »((exsystem::SYSInfo*)this)->exsystem::SYSInfo::dev < 0«

Ihr dürft mich nun in der Luft zerreißen.

dev < 0

nichts anfangen. Was soll das denn tun?

Dein Compiler kann mit

dev < 0

nichts anfangen. Was soll das denn tun?

Wenn dev kleiner als 0 ist soll er dev von vorne lesen. Da es doch dev[0],dev[1] und dev[2] (beliebig mehr oder auch weniger) gibt. D.h. in diesem Beispiel sollte er dann die Daten von lo ausgeben.
EDIT: So hab ich zumindest gedacht das er es tut.
EDIT2: Er kommt btw mit keinem Operator klar in der Funktion.

Wenn dev kleiner als 0

dev ist ein vector. Keine Zahl.

Vector kann genausowenig wie der string in deiner anderen Frage gleich, kleiner oder größer als 0 sein. Tu dir und uns den Gefallen und lies dir im Einsteigerbuch deiner Wahl nochmal den Teil zu Datentypen durch.

Wenn dev kleiner als 0 ist soll er dev von vorne lesen.

Wie kann ein Vektor kleiner als eine Zahl sein? Meinst du, dass die Länge kleiner als 0 sein soll? Das würde man dev.size() < 0 schreiben, aber das ist Blödsinn, weil eine negative Länge erstens keinen Sinn ergibt und zweitens size hier sowieso einen unsigned-Typ zurückgibt.

Wie kann ein Vektor kleiner als eine Zahl sein? Meinst du, dass die Länge kleiner als 0 sein soll? Das würde man dev.size() < 0 schreiben, aber das ist Blödsinn, weil eine negative Länge erstens keinen Sinn ergibt und zweitens size hier sowieso einen unsigned-Typ zurückgibt.

Ich wollte prüfen ob ich am Anfang des vectors bin. D.h. ich könnte das vllcht mit front machen?

EDIT: Mhm ... mist das dann auch wieder Quark da ich dann niemals bis zum for komme oder?

Und komm jetzt bitte nicht wieder mit der Ausrede, du seist auf Arbeit. Wenn du die Zeit hast, stundenlang deine Anfängerprobleme hier im Forum zu diskutieren, hast du auch die Zeit, dir kurz das entsprechende Grundlagenwissen anzueignen. Geht vermutlich sogar wesentlich schneller.

Sorry, wenn das jetz ein wenig direkt war, aber so langsam gehen mir deine Fragen und Probleme ziemlich auf die Nerven.

Lies dir bitte erstmal die Grundlagen zum Thema "vector" durch. Bspw. http://www.cplusplus.com/reference/stl/vector/.

Und komm jetzt bitte nicht wieder mit der Ausrede, du seist auf Arbeit. Wenn du die Zeit hast, stundenlang deine Anfängerprobleme hier im Forum zu diskutieren, hast du auch die Zeit, dir kurz das entsprechende Grundlagenwissen anzueignen. Geht vermutlich sogar wesentlich schneller.

Sorry, wenn das jetz ein wenig direkt war, aber so langsam gehen mir deine Fragen und Probleme ziemlich auf die Nerven.

Link bin ich schon lange drauf ... und ja see ya!

Was soll nextNet genau erreichen? Die Funktion macht so wie du sie konzipiert hast keinen wirklichen Sinn...

Ich wollte eine Funktion mit der ich alle werte aus dev ausgeben kann ohne das ich wie hier:

unsigned int SYSInfo::getBytesr(){ 
  return dev[0].bytes_received;   
}

die Zahl hinter dev ändern muss.

Ich wollte eine Funktion mit der ich alle werte aus dev ausgeben kann ohne das ich wie hier: [...] die Zahl hinter dev ändern muss.

"Die Zahl hinter dev" nennt sich Array-Index. Als Index kannst du statt der Konstanten auch eine Variable benutzen (=> Grundlagenbuch), und um die Variable hochzuzählen typischerweise eine Schleife (=> Grundlagenbuch):

//alle Werte aus dev ausgeben:
for (unsigned i = 0; i < dev.size(); ++i)
{
  cout << dev[i] << ' ';
}

Mal davon abgesehen, dass dein nextNet garnichts ausgibt.

Enno schrieb:

Ich wollte eine Funktion mit der ich alle werte aus dev ausgeben kann ohne das ich wie hier: [...] die Zahl hinter dev ändern muss.

"Die Zahl hinter dev" nennt sich Array-Index. Als Index kannst du statt der Konstanten auch eine Variable benutzen (=> Grundlagenbuch), und um die Variable hochzuzählen typischerweise eine Schleife (=> Grundlagenbuch):

//alle Werte aus dev ausgeben:
for (unsigned i = 0; i < dev.size(); ++i)
{
  cout << dev[i] << ' ';
}

Mal davon abgesehen, dass dein nextNet garnichts ausgibt.

wäre es mal so leicht

Ich muss auf jedes Element einzelnt mit einer getFunktion zugreifen können meine next Funktion soll nun meinen index ändern. Wahrscheinlich der größte müll den ich mal wieder rede.

for(std::vector<face>::const_iterator it = dev.begin(); it != dev.end(); ++it)
{
  it->getXYZ();
  it->getName();
}

wäre es mal so leicht

Das ist das, was du beschrieben hast. Leider wird man aus deinem ersten Post nicht so recht schlau. Was willst du erreichen?

Ich muss auf jedes Element einzelnt mit einer getFunktion zugreifen können meine next Funktion soll nun meinen index ändern.

Dann greif doch innerhalb einer Schleife mit dem Getter auf die Elemente zu - den index abzuändern übernimmt der Schleifenkopf doch.

//alle Werte aus dev ausgeben:
for (unsigned i = 0; i < dev.size(); ++i)
{
  cout << dev[i] << ' ';
}

Das Problem für mich hier ist, dass die schleife alle teile ausgibt und nicht nur den ersten "block" aus [0](oder halt [1] ...).
Von der theorie noch mal eben her:
setFunktion die den gewünschten Index setzt, nextFunktion die in den Index geht und getFunktion die den teil ausgibt.

Würde das so gehen ?

daddy_felix schrieb:

meinst du sowas:

for(std::vector<face>::const_iterator it = dev.begin(); it != dev.end(); ++it)
{
  it->getXYZ();
  it->getName();
}

Wäre das nicht nur eine Schleife die alle getFunktionen bestückt?

Von der theorie noch mal eben her:
setFunktion die den gewünschten Index setzt, nextFunktion die in den Index geht und getFunktion die den teil ausgibt.

Würde das so gehen ?

Du willst doch über alle Elemente laufen? Zumindest hast du das vorhin noch geschrieben. Dann musst du keinen Index setzen.

daddy_felix schrieb:

meinst du sowas:

for(std::vector<face>::const_iterator it = dev.begin(); it != dev.end(); ++it)
{
  it->getXYZ();
  it->getName();
}

Wäre das nicht nur eine Schleife die alle getFunktionen bestückt?[/quote]

das ist ein BEISPIEL. wie du den vector komplett durchläufst. Wenn du nur eine bestimmtes Eigenschft benötigst, dann greifst du eben nur auf diese zu:

cout << it->getVeryImportantWhatever() << " ";

Du willst doch über alle Elemente laufen? Zumindest hast du das vorhin noch geschrieben. Dann musst du keinen Index setzen.

meinst du sowas:

for(std::vector<face>::const_iterator it = dev.begin(); it != dev.end(); ++it)
{
  it->getXYZ();
  it->getName();
}

Wäre das nicht nur eine Schleife die alle getFunktionen bestückt?[/quote]

das ist ein BEISPIEL. wie du den vector komplett durchläufst. Wenn du nur ein bestimmtes Elemen benötigst, dann greifst du eben nur auf dieses Element zu:

cout << it->getVeryImportantElement() << " ";

[/quote]

Ja weiß ich also "bestückt" die schleife die getVeryImportantElemnt() mit einem wert.

Einfacher gehtz ja so:

unsigned int SYSInfo::getFifot(){ 
  return dev[0].fifo_transmitted;   
}

So bekomm ich die ja nun einzelnt. Ich will nun aber den ganzen "block" von index [n].

Das Problem für mich hier ist, dass die schleife alle teile ausgibt und nicht nur den ersten "block" aus [0](oder halt [1] ...).

Was verstehst du unter dem ersten Block? Meinst du sowas:

for( unsigne i = 0; i < dev.size(); ++i)
{
  cout << dev[i].bytes_received << '\n';
}

pumuckl schrieb:

Was willst du erreichen?

Enno schrieb:

Das Problem für mich hier ist, dass die schleife alle teile ausgibt und nicht nur den ersten "block" aus [0](oder halt [1] ...).

Was verstehst du unter dem ersten Block? Meinst du sowas:

for( unsigne i = 0; i < dev.size(); ++i)
{
  cout << dev[i].bytes_received << '\n';
}

pumuckl schrieb:

Was willst du erreichen?

In diesem fall der vector:

lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0
 wlan0:       0       0    0    0    0     0          0         0        0       0    0    0    0     0       0          0
  eth0: 39072289   41342    0    0    0     0          0      3348  4363113   28697    0    0    0     0       0          0

Meine vorstellung von "block" oder auch halt dem index[0] in diesem fall:

lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0

Achso, du willst also anscheinend alle Elemente von "face" ausgeben. In diesem Fall machst du entweder

cout << it->getName() << " " << it->getPacketsReceived() << " " << ...

oder, falls du dir noch mehr fortgeschrittenen Techniken ansehen willst: du kannst den Ausgabeoperator << für deinen Typ "face" überladen (-> Grundlagenbuch).

dann kannst du das einfach so schreiben:

cout << *it;

In diesem fall der vector:

lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0
 wlan0:       0       0    0    0    0     0          0         0        0       0    0    0    0     0       0          0
  eth0: 39072289   41342    0    0    0     0          0      3348  4363113   28697    0    0    0     0       0          0

Meine vorstellung von "block" oder auch halt dem index[0] in diesem fall:

lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0

Du willst also einfach nur den ersten Eintrag im vector ausgeben? Falls nicht, sag uns doch bitte, was du erreichen willst!

Du willst also einfach nur den ersten Eintrag im vector ausgeben? Falls nicht, sag uns doch bitte, was du erreichen willst!

Jain ich will eine Funktion die da durchlaufen kann und den entsprechenden index den ich eingebe dann auch ausgibt. Versteht man das?

Jain ich will eine Funktion die da durchlaufen kann und den entsprechenden index den ich eingebe dann auch ausgibt. Versteht man das?

Nein. Du willst genau einen Index eingeben und die Werte von dem Element an diesem Index ausgeben?

void PrintElement(size_t index)
{
  assert(index < dev.size());
  cout << dev[index].getName() << " " << dev[index].getPacketsLoss() << " " << ...;
}

so?

Enno schrieb:

Jain ich will eine Funktion die da durchlaufen kann und den entsprechenden index den ich eingebe dann auch ausgibt. Versteht man das?

Nein. Du willst genau einen Index eingeben und die Werte von dem Element an diesem Index ausgeben?

void PrintElement(size_t index)
{
  assert(index < dev.size());
  cout << dev[index].getName() << " " << dev[index].getPacketsLoss() << " " << ...;
}

so?

Mhm ... eher dachte ich an sowas:

unsigned int SYSInfo::setIndex(){
  std::cout << "Bitte Index eingeben: " << std::endl;
  std::cin >> i;
  return i;
}

void SYSInfo::nextNet(){
  for(i; i < dev.size(); i++){
    std::cout << dev[i] << '\n';    
  }
}

Nur gibt er jetzt alles ab dem eingegebenen wert aus und nicht nur den einen Index.

Es gilt:

index 0:

lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0

index 1:

wlan0:       0       0    0    0    0     0          0         0        0       0    0    0    0     0       0          0

index 2:

eth0: 39072289   41342    0    0    0     0          0      3348  4363113   28697    0    0    0     0       0          0

irgendwie kommst du mit deinen Indizes durcheinander. Gibt bitte mal ein konkretes Beispiel, also bei welcher Eingabe und welchen Datensatz was genau ausgegeben werden soll.

Es gilt:

index 0:

lo:  167027     548    0    0    0     0          0         0   167027     548    0    0    0     0       0          0

index 1:

wlan0:       0       0    0    0    0     0          0         0        0       0    0    0    0     0       0          0

index 2:

eth0: 39072289   41342    0    0    0     0          0      3348  4363113   28697    0    0    0     0       0          0

Bei meinem Code kommt für i = 1:

wlan0:Bytes: 0 Packets: 0 Errs: 0 Drop: 0 Fifo: 0 Frame: 0 Compressed: 0 Multicast: 0 Bytes: 0 Packets: 0 Errs: 0 Drop: 0 Fifo: 0 Coll: 0 Carrier: 0 Compressed: 0
eth0:Bytes: 51174195 Packets: 72809 Errs: 0 Drop: 0 Fifo: 0 Frame: 0 Compressed: 0 Multicast: 9927 Bytes: 7798016 Packets: 48352 Errs: 0 Drop: 0 Fifo: 0 Coll: 0 Carrier: 0 Compressed: 0

Es soll aber nur:

wlan0:Bytes: 0 Packets: 0 Errs: 0 Drop: 0 Fifo: 0 Frame: 0 Compressed: 0 Multicast: 0 Bytes: 0 Packets: 0 Errs: 0 Drop: 0 Fifo: 0 Coll: 0 Carrier: 0 Compressed: 0

Das heißt ich muss dem noch irgendwie sagen das dann stop ist.