void f()
{
   //...
   instance.f1(); // 21-я строка   
}

G>P.S. Нет — излишним префиксам!

44.1. Не используйте в качестве имен тарабарщину.

Отличный образец такого подхода можно наблюдать в любом предлагаемом Microsoft примере программы, хотя эта проблема ни в коем случае не ограничивается корпорацией Microsoft. Все демонстрационные программы Microsoft Windows включают тип переменной в ее имя. Например, объявление типа:

const char *str;

будет сделано следующим образом:

LPCSTR lpszstr;

Переведите lpszstr как "указатель типа long с именем str на строку, оканчивающуюся 0". На самом деле здесь несколько проблем, не последней из которых является тот факт, что LPCSTR скрывает наше объявление указателя. Тем не менее обсуждаемое правило посвящается проблеме самого имени.

Этот стиль выбора имен называется "венгерской" записью по названию родины руководителя отдела программирования Microsoft Чарльза Саймони, который его изобрел. (А не потому, что его использование придает программам Microsoft такой вид, как будто они написаны на венгерском языке.)

Венгерская запись целесообразна для языка ассемблера, в котором все, что вы знаете о переменной — это ее размер. Включение информации о типе в имя переменной позволяет вам контролировать правильность ее использования. Языки более высокого уровня типа С и С++ используют для этой цели объявление переменных.

Доктор Саймони несколько раз в печати защищал такой метод записи, но я бы не стал его рекомендовать для программ на С или С++. По моему мнению, венгерская запись не дает ничего, кроме ухудшения читаемости программ. Простые str или string значительно легче читаются и содержат ту же информацию. Если вам на самом деле нужно узнать тип, то для этого достаточно вернуться к определению.

Существует и более распространенный, хотя и менее радикальный прием, при котором имена указателей начинают символом p. Эта практика тоже загромождает программу. Вы ведь не начинаете имена целочисленных переменных типа int символом i, переменных типа double — d, а функций — f? Очевидным исключением является случай, когда у вас есть объект и указатель на этот объект в одной и той же области видимости:

char str[128], *pstr = str;

c другой стороны, для указателя, вероятно, лучше содержательное имя. Сравните:

char str[128], *first_nonwhite = str;
while ( isspace(*first_nonwhite) )
++first_nonwhite;
// В этой ситуации имя *first_nonwhite говорит вам гораздо
// больше о том, что делает переменная, чем предыдущее "*pstr".

STDMETHODIMP CSRCOMMOPCGroup::AddItems( 
/* [in] */                        DWORD count,
/* [size_is][in] */                OPCITEMDEF *itemArray,
/* [size_is][size_is][out] */    OPCITEMRESULT **addResults,
/* [size_is][size_is][out] */    HRESULT **errors
)
{
    LogPrint(L"IOPCItemMgt::AddItems\n");
    if(m_public)        return OPC_E_PUBLIC;
    if(count<=0)        return E_INVALIDARG;
    if(!itemArray)        return E_INVALIDARG;
    if(!addResults)        return E_INVALIDARG;
    if(!errors)            return E_INVALIDARG;
    *addResults=0;
    *errors=0;
    T_CoAutoArr<HRESULT>        hr(count);
    T_CoAutoArr<OPCITEMRESULT>    ir(count);
    if(!hr||!ir)return E_OUTOFMEMORY;
    UINT fails=0;
    {//Sync
        C_SyncHelper lockDItem(g_DItemSync, C_SyncHelper::StateWrite);
        C_SyncHelper lockThis(this, C_SyncHelper::StateWrite);
        for(UINT i=0;i<count;i++)
        {
            T_CoPtr<COPCGroupItem> p=new COPCGroupItem(this);
            int h=m_Items.Include(p);
            hr[i]=p->Init(&itemArray[i], &ir[i], h);
            if(FAILED(hr[i]))
            {
                fails++;
                m_Items.Exclude(h);
                ir[i].dwAccessRights        =0;
                ir[i].dwBlobSize            =0;
                ir[i].hServer                =g_ItemIndex.end();
                ir[i].pBlob                    =0;
                ir[i].vtCanonicalDataType    =VT_ERROR;
                ir[i].wReserved                =0;
            }
        }
    }
    *addResults=ir.Detach();
    *errors=hr.Detach();
    if(fails)    return S_FALSE;
    else        return S_OK;
}

файл а - std::string sMessage;
файл б - std::string MessageString;
файл в - std::string strMsg;

Отличный образец такого подхода можно наблюдать в любом предлагаемом Microsoft примере программы, хотя эта проблема ни в коем случае не ограничивается корпорацией Microsoft. Все демонстрационные программы Microsoft Windows включают тип переменной в ее имя.

(С)Николай Носов.
— Вот когда полетишь на Луну, — сказали коротышки профессору Звёздочкину, — тогда и будешь выступать на трибуне. А пока посиди здесь, на травке.

CString (или std::string) = CFormatString("его возраст - %i лет, его з/п - %.2f$, имя ему - %s") 
                    << age << зарплата << name;

// В объявлении класса
CString tax(_T("1000$"));
// В его методе (другой файл, строка так пятитысячная)
CString (или std::string) = CFormatString("его возраст - %i лет, его з/п - %.2f, имя ему - %s")
<< age << tax << name;

С>>// В объявлении класса
С>>CString tax(_T("1000$"));
С>>// В его методе (другой файл, строка так пятитысячная)
С>>CString (или std::string) = CFormatString("его возраст - %i лет, его з/п - %.2f, имя ему - %s")
С>><< age << tax << name;
С>>

std::ostrstream str;
str << "его возраст - " << age << " лет, его з/п - " << tax << " имя ему - " << name << '\0';
std::string myStr = str.str();

Использование ОО-технологий (не разрушающих стек) при отсутствии должного контроля за типами позволяет перевести систему из класса работающих с фатальными ошибками в класс работающих просто некорректно (но не вывалювающихся). Этого явно мало, чтобы выкинуть венгерку, но её применение становится менее оправданным. Впрочем, есть целая куча других причин от неё отказаться.

DG>CString (или std::string) = CFormatString("его возраст - %i лет, его з/п - %.2f$, имя ему - %s") 
DG>                    << age << зарплата << name;
DG>

CString (или std::string) = CFormatString("его возраст - {0} лет, его з/п - {1}, имя ему - {2}") << age << std::setprecision(2) << зряплата << name;

РМ>CString (или std::string) = CFormatString("его возраст - {0} лет, его з/п - {1}, имя ему - {2}") << age << std::setprecision(2) << зряплата << name;
РМ>

CString (или std::string) = CFormatString("его возраст - {0} лет, его з/п - {1:f2}, имя ему - {2}") 
    << age << зряплата << name;

A note from Dr. GUI: Long, long ago in the early days of DOS, Microsoft's Chief Architect Dr. Charles Simonyi introduced an identifier naming convention that adds a prefix to the identifier name to indicate the functional type of the identifier.

This system became widely used inside Microsoft. It came to be known as "Hungarian notation" because the prefixes make the variable names look a bit as though they're written in some non-English language and because Simonyi is originally from Hungary.

S> Как сделать без префиксов? Не называть же параметр countOfSomeClass

SomeClass( int count )
{
    this.count = count;
}

Do not use instance fields that are public or protected (Public or Protected in Visual Basic). If you avoid exposing fields directly to the developer, classes can be versioned more easily because a field cannot be changed to a property while maintaining binary compatibility. Consider providing get and set property accessors for fields instead of making them public.

void SomeClass::SetTimeRange( TimeRange i_exclusiveTimeRange )
{
// legacy code...

   TimeRange inclusiveTimeRange = i_exclusiveTimeRange;
   inclusiveTimeRange.SetEndTime( i_exclusiveTimeRange.GetEndTime() - oneMinute );

// legacy code...

   doSomething( inclusiveTimeRange );

// legacy code...

   doSomethingElse( i_exclusiveTimeRange );
}