Hab da ein kleines Problem mit Funktoren und unary_function<>. Ich möchte lediglich, dass jeder Eintrag im vector mit 10 dividiert wird. Hierzu folgenden Code:

#include <iostream>
#include <vector>
#include <algorithm>
#include <iterator>
using namespace std;

struct divide10:public std::unary_function<int,float>{
	float operator()(const int j) const { return j/10;	}
};

int main(){
	typedef ostream_iterator<int> osIt;
	vector<int> v1,v2;
	v1 += 12,23,34,45,56,67,78,89,90;
	v2 += 1,1,1,1,1,1,1,1,1;
	transform(v1.begin(),v1.end(),osIt(cout,", "),divide10());

	return 0;
}

Leider meckert der Compiler 'instantiated from here' beim Aufruf von transform().
Leider komme ich auch mit den Büchern hier nicht weiter.
Kann mir jemand schnell einen Tipp geben?

Danke im Voraus!
mfg
rox

Leider meckert der Compiler 'instantiated from here' beim Aufruf von transform().

Was meckert er denn wo an?

Wie gesagt, die Fehlermeldung ist:

../main.cpp:16:   instantiated from here

beim Funktionsaufruf von

transform(v1.begin(),v1.end(),osIt(cout,", "),divide10());

**** Full rebuild of configuration Debug for project Z_diverses ****

make -k clean all 
make: Warning: File `objects.mk' has modification time 0.7 s in the future
rm -rf  ./main.o  ./main.d Z_diverses.exe

Building file: ../main.cpp
Invoking: GCC C++ Compiler
g++ -IC:\cygwin\usr\include\boost-1_33_1 -O0 -g3 -Wall -c -fmessage-length=0 -omain.o ../main.cpp
../main.cpp: In function `int main()':
../main.cpp:29: warning: unused variable 'c1'
/usr/lib/gcc/i686-pc-cygwin/3.4.4/include/c++/bits/stl_algo.h: In function `_OutputIterator std::transform(_InputIterator, _InputIterator, _OutputIterator, _UnaryOperation) [with _InputIterator = __gnu_cxx::__normal_iterator<int*, std::vector<int, std::allocator<int> > >, _OutputIterator = std::ostream_iterator<int, char, std::char_traits<char> >, _UnaryOperation = divide10]':
../main.cpp:44:   instantiated from here
/usr/lib/gcc/i686-pc-cygwin/3.4.4/include/c++/bits/stl_algo.h:789: warning: passing `float' for converting 1 of `std::ostream_iterator<_Tp, _CharT, _Traits>& std::ostream_iterator<_Tp, _CharT, _Traits>::operator=(const _Tp&) [with _Tp = int, _CharT = char, _Traits = std::char_traits<char>]'
Finished building: ../main.cpp

Building target: Z_diverses.exe
Invoking: GCC C++ Linker
g++ -oZ_diverses.exe ./main.o
Finished building target: Z_diverses.exe

Build complete for project Z_diverses

Build complete for project Z_diverses

Wo ist das Problem?

Dein eigentliches Problem sind die falschen Datentypen. (int / int -> int)

typedef ostream_iterator<float> osIt;
// ...
transform(v1.begin(), v1.end(), osIt(cout, ", "), bind2nd(divides<float>(), 10.f));

Du solltest die Warnings posten, nicht dein ganzes verdammtes Build-Log. ...

Na, doch kein Grund, unfreundlich zu werden ....
Schließlich wurde er mehrfach deutlich aufgefordert, "alles zu zeigen"; wenn er das nun tut (und der output auch noch so klein ist), finde ich das voll in Ordnung.

@"roxor": Anscheinend passt die Meldung nicht ganz zu Deinem Quellcode (Zeile 44 gibt's da nicht) - vermutlich hast Du den gekürzt (was ich eigentlich begrüße)

"../main.cpp:44: instantiated from here " ist dabei lediglich eine Erläuterung zu der eigentlich Warnung:
"warning: passing `float' for converting 1 of `std::ostream_iterator<...'"

Es kann für die Fehlersuche sehr hilfreich sein, zu wissen, wo welches template instantiiert wurde (Merke: Der Compiler erzeugt von jedem template bei Bedarf eine "echte" (konkrete) Klasse/Funktion mit den entsprechenden template-Parametern ... diesen Vorgang nennt man "template-Instanziierung").

Beim "Teilen durch 10 von Integern" würde ich an Deiner Stelle nochmal überdenken, was Du wirklich willst....

Gruß,

Simon2.

roxor schrieb:

struct divide10:public std::unary_function<int,float>{
	float operator()(const int j) const { return j/10;	}
};

Das funktioniert zwar, liefert aber vermutlich das falsche Ergebnis (Stichwort: Ganzzahl-Rechnung). richtig wäre wohl "return (float)j / 10;".

v1 += 12,23,34,45,56,67,78,89,90;
v2 += 1,1,1,1,1,1,1,1,1;
}

Diese beiden Zeilen sind Unsinn (erstaunlich, daß dein Compiler das geschluckt hat) - vector<> hat keinen operator+= und selbst wenn, würde der nur den letzten Wert dieser Sammlung bekommen.

...

v1 += 12,23,34,45,56,67,78,89,90;
v2 += 1,1,1,1,1,1,1,1,1;
}

Diese beiden Zeilen sind Unsinn (erstaunlich, daß dein Compiler das geschluckt hat) - vector<> hat keinen operator+= und selbst wenn, würde der nur den letzten Wert dieser Sammlung bekommen.

Nutzt er vielleicht boost/assign/std/vector.hpp ?
( http://www.boost.org/libs/assign/doc/index.html )
da geht das nämlich ...

Gruß,

Simon2.

finix schrieb:

Du solltest die Warnings posten, nicht dein ganzes verdammtes Build-Log. ...

Na, doch kein Grund, unfreundlich zu werden ....
Schließlich wurde er mehrfach deutlich aufgefordert, "alles zu zeigen"; wenn er das nun tut (und der output auch noch so klein ist), finde ich das voll in Ordnung.

Genau, "Was meckert er denn wo an?" war die Aufforderung das ganze Build-Log zu posten. Und jemanden auf die vielfältigen Probleme seiner Fragetaktik hinzuweisen ist wirklich so was von unfreundlich. Danke, Simon2.

...nicht dein ganzes verdammtes Build-Log. ...

Ist einfach unfreundlich, fertig.
Übrigens ist sein "ganzes verdammtes Build-Log" deutlich kürzer als der Source-code der meisten Fragesteller hier.

finix schrieb:

...Das ist alles? Wirklich? Ganz, ganz ehrlich? ...

Ist auch nicht unbedingt entgegenkommend und dass er daraufhin nicht einfach nochmal das schickt, was er als "alles" empfindet, sondern "alles, was der Compiler rausspuckt", empfinde ich wirklich nicht als Dämlichkeit.

Überdenke einfach mal den Ton, mit dem Du hier auftrittst ... sprichst Du so auch mit Deinen Freunden ?

Gruß,

Simon2.