Почему значения мусора приходят после баланса счета

Вы вставляете в std::cout, среди прочего, результат, возвращаемый функцией display который должен быть int, но, учитывая, что ваша функция не имеет оператора возврата, все возможно, в основном вы отправляете неопределенное значение в ostream cout, а это мусор по определению.

В этом случае функция display() должна вернуть void. У вашей версии в подписи написано, что она возвращает int, но тогда вы ничего не возвращаете. Это приводит к неопределенному поведению.

Также не рекомендуется использовать using namespace std и #include<bits/stdc++.h>.

Читайте здесь Почему «используется пространство имен std;» считается плохой практикой?

А здесь Как #include <bits/stdc++.h> работает в C++?

#include <iostream>

class Bankaccount
{
public:
    int accnumber, accbalance;

    void display()
    {
        std::cout << "Account number is: " << accnumber << "\n";
        std::cout << "Account balance is: " << accbalance << "\n";
    }
};

int main()
{
    Bankaccount a;
    Bankaccount b;

    a.accnumber = 123456;
    a.accbalance =50;

    b.accnumber = 67890;
    b.accbalance = 2000;
    std::cout<<"Account details of A\n\n";
    a.display();                             // this is how to use display

    std::cout<<"\nAccount details of B\n\n";
    b.display();
    return 0;
}

Ваша функция int display() не возвращает int, поэтому после вызова этой функции у вас будет неопределенное поведение. Если он вернул int, это число было напечатано, но я подозреваю, что это не то, что вы хотели.

Мусор, который вы видите, — это int, выбранный из стека (потому что display() должен был поместить туда int). Его зачем-то поставили какой-то другой функцией, но теперь его нет, так что всякое может случиться. Чтобы избежать этого, вы можете объявить свою функцию void display(), но тогда вы не сможете ее транслировать, что, похоже, вы и хотите сделать.

Если вы хотите иметь возможность передавать свои объекты в потоковом режиме, вам необходимо определить операторы потоковой передачи для выполнения этой работы. Я заменил вашу функцию display() оператором исходящего потока (operator<<) здесь:

#include <iostream>
//#include<bits/stdc++.h> // non-portable, don't use it

// using namespace std; // brings in too much in the namespace
using std::cout; // prefer this or just write std::cout everywhere

class Bankaccount {
public:
    int accnumber, accbalance;

    friend std::ostream& operator<<(std::ostream& os, const Bankaccount& ba) {
        return os << "Account number is : " << ba.accnumber
                  << "\nAccount balance is: " << ba.accbalance << "\n";
    }
};

int main() {
    Bankaccount a;
    Bankaccount b;

    a.accnumber = 123456;
    a.accbalance = 50;

    b.accnumber = 67890;
    b.accbalance = 2000;
    // std::endl is approx. the same as "\n" + std::flush. You don't need flushing.
    cout << "Account details of A\n\n" << a << "\n";
    cout << "\nAccount details of B\n\n" << b << "\n";
}

Скорее всего, это то, чего вы хотели добиться:
https://wandbox.org/permlink/bpPth9WutHaiU5jQ

#include <bits/stdc++.h>
using namespace std;
class Bankaccount {
public:
    int accnumber, accbalance;
    std::ostream& display(std::ostream& out) const
    {
        out << "Account number is: " << accnumber;
        return out << "\nAccount balance is: " << accbalance;
    }
};

std::ostream& operator<<(std::ostream& out, const Bankaccount& acc)
{
    return acc.display(out);
}

int main()
{
    Bankaccount a;
    Bankaccount b;

    a.accnumber = 123456;
    a.accbalance = 50;

    b.accnumber = 67890;
    b.accbalance = 2000;
    cout << "Account details of A\n" << a << endl;
    cout << "\nAccount details of B\n" << b << endl;
    return 0;
}