C++ linked list class. Lyckas inte med Remove funktion.

void Sorted_List::remove(int value)
{
  Node* tmp = first; //Toppen
  Node* old = tmp; //Vad är tmp? Har du definierat Node *tmp i class Sorted_List? Inte ett bra namn, och onödigt i de flesta fall
  if (first->data == value)
  {
    tmp = first->next; //Nu har du 'first' som nod 0 och 'tmp' som nod 1
    delete first; //Nu tar du bort nod 0 ('first')
    first = tmp->next; //Nu sätter du nod 2 som 'first'. Det skall stå first = tmp;
  }
  else
  {
    while (tmp->data != value)
    {
      if (tmp->next == nullptr)
      {
        return; //Här vill du antingen lägga till en error funktion eller "return false;" för att indikera att inget togs bort.
         //Det kräver att du ändrar void till bool naturligtvis. Om det är kritiskt att du tar bort något skall det vara en error
         //funktion istället så att allting går sönder efter en plan istället för att du får felsöka i timmar.
      }
      old = tmp;
      tmp = tmp->next;
    }
    old->next = tmp->next;
    delete tmp;
  }
  size_of_list -= 1; //Detta är att be om problem när du ändrar saker senare eller lägger ytterligare funktioner 
  //för att modifiera din LL. Gör hellre en funktion 
  //GetLength() {
  //int nLength = 0; 
  //Node* index = first; 
  //while (index->data != nullptr) {
    //++nLength; 
    //index = index->next;
  //} 
  //return nLength} för att få en absolut värdering varje gång. 
}

//I class Sorted_List{
  //Node *_head; //_head->next pekar alltid på första elementet. _head används inte självt.

bool Sorted_List::remove(int value) //bool så vi kan se hur det går.
{
  Node* tmp = _head; 
  
  while (tmp->next->data != value) {
    if (tmp->next == nullptr) {
      return false; 
    }
    tmp = tmp->next;
  }
  Node* kill = tmp->next;
  tmp->next = tmp->next->next;
  delete kill;

  size_of_list = GetLength();  
  return true;
}

int GetLength() {
  int nLength = 0; 
  Node* index = _head->next; 
  while (index->data != nullptr) {
    ++nLength; 
    index = index->next;
  } 
  return nLength;
}