github/msft-calculator
1
0
Fork 0
mirror of https://github.com/Microsoft/calculator.git synced 2025-08-22 06:13:14 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

Improved readablity of OperandLength and StandardScientificOperandLength

Browse source 
Moved to switch-case for OperandLength
This commit is contained in:
Nicholas Baron 2019-03-26 10:03:29 -07:00
commit 5937345151
1 changed files with 15 additions and 22 deletions
Download patch file Download diff file Expand all files Collapse all files

37
src/CalcViewModel/Common/CopyPasteManager.cpp
View file

@ -459,34 +459,27 @@ bool CopyPasteManager::TryOperandToULL(const wstring& operand, int numberBase, u
}
size_t CopyPasteManager::OperandLength(wstring operand, ViewMode mode, CategoryGroupType modeType, int programmerNumberBase)
{
if (mode == ViewMode::Standard || mode == ViewMode::Scientific)
{
return StandardScientificOperandLength(operand);
}
else if (mode == ViewMode::Programmer)
{
return ProgrammerOperandLength(operand, programmerNumberBase);
}
else if (modeType == CategoryGroupType::Converter)
{
{
if(modeType == CategoryGroupType::Converter) {
return operand.length();
}
return 0;
switch(mode) {
case ViewMode::Standard:
case ViewMode::Scientific:
return StandardScientificOperandLength(operand);
case ViewMode::Programmer:
return ProgrammerOperandLength(operand, programmerNumberBase);
default:
return 0;
}
}
size_t CopyPasteManager::StandardScientificOperandLength(wstring operand)
{
// Loop until a decimal is found or if the end is reached
bool hasDecimal = false;
for (size_t i = 0; i < operand.length() && !hasDecimal; i++)
{
if (operand[i] == L'.')
{
hasDecimal = true;
}
}
{
const bool hasDecimal = operand.find('.') != std::wstring::npos;
if (hasDecimal)
{