c++ cout ändrar variabler!

Högst troligen läser du utanför minnet som är allokerat för arrayen.

T.ex.:

int array[10];

cout << array[10] << endl;

Det blir klart enklare om du kunde dela med dig av fler detaljer. Vad menar du exempelvis med "två testfall"? Innehållet kanske ändras varje gång du kör.

Innehåller vektorn flyttal så kan du inte lita på att de är exakt lika med något värde – den enda garantin du har är att de ligger inom en viss felmarginal. Sedan ska du nog inte använda kommaoperatorn i assert()... posta mer kod!

Det gör det visst.

Ett funktionsanrop som inte optimeras bort skriver nästan garanterat över en viss mängd stackminne. D.v.s. beroende på vilken ordning dina variabler är definierade, kanske precis den del av minnet som array[10] (det 11:e elementet) referar till i en array med 10 element.

En anna orsak som kan ske när talet är flyttal är om huruvida flyttalen fortfarande är kvar i registren eller ej (oftast högre precision i registren).

Som You nämnde kan du kolla om talen ligger inom en viss felmarginal.

bool float_equals(float lhs, float rhs)
{
    // Jag är inte säker på vilken konstant som skulle passa bäst.
    // Funktionen skulle kunna ta felmarginalen som argument för att passa
    // olika sammanhang.
    return abs(lhs - rhs) < 0.0000001;
}

Övriga kommentarer:

Templates kan inte kompileras separat, utan implementationerna måste inkluderas direkt. Det är därför det bara fungerar när du inkluderar vector.cpp. Du har två val för att att kunna inkludera vector.hpp (vilket du bör göra):

• Implementera funktionerna i en separat fil, men inkludera sedan denna sist i vector.hpp. Vanligt är att låta implementationsfilen heta något som vector.impl eller vector.hpp.impl för att skilja den från filer som ska kompileras separat.

• Implementera funktionerna direkt i headerfilen, antingen "inline" eller efter klassdeklarationen.

I vector.cpp använder du & istället för && som logiskt OCH. I det här fallet råkar koden fungera eftersom både vänster- och högerled är ett riktiga booleska uttryck, och alltså evalueras till antingen 0 eller 1, men det ska vara &&.

Din kod är inte exception safe, och kan därför läcka minne. Ett exempel är din copy constructor. Du allokerar rätt mängd minne, men skulle sedan ett element kasta ett undantag när det kopieras kommer minnet att läcka.

Vector<T>::operator= kommer att skriva längre än sin buffer om rhs är för lång.

Detta är nog bara temporärt i din kod, men i Vector<T>::insert pekar du om vec till en temporär buffer. Du har dock den "korrekta" koden utkommenterad, vilket får mig att tro att det är temporärt.