Добавление кода города в программу

Question

По сути, я получил эту программу за одно задание. Он запрашивает любые телефонные номера длиной 7 цифр, например, 930-1892. тогда он скажет ваши цифры в качестве вывода. Теперь я решил, что предпочел бы добавить код города к 7 цифрам, всего до 10 цифр. Я хочу вывести его в формате, подобном [phone].

Я удивительно борюсь с этим, я не могу заставить программу выводить 10 цифр, включая код города, а не только 7 цифр. Но программа, как и сейчас, должна скомпилировать, запустить и вывести 7-значный номер телефона, к которому вы добавили.

#include<iostream>
#include<string>
using namespace std;

/* Global Variables */
int prefix[3];
int number[4];
string phoneNumber;

// main routine
int main()
{
  // local variables
  int input_done = 0; // flag to control input loop
  int ascii_0 = 48;
  int i = 0;

  while (input_done == 0)
  {
    // Ask the user for input
    cout << "Please enter your 7 digit phone number (for example: 123-4567)" << endl;
    getline(cin, phoneNumber);

    // split up the input and check it for validity
    if (phoneNumber.length()==8)
    {
      input_done = 1;  // assume number is correct until otherwise

      for (i = 0; i <= 7; i++)
      {
        if (i == 3)
        {
          if (phoneNumber[3] != '-')
            input_done = 0;
        }
        else
        {
          if ( (phoneNumber[i] < '0') || (phoneNumber[i] > '9') )
            input_done = 0;
        }
      }

      // assign values to individual array elements if checked out okay
      if (input_done == 1)
      {
        for (i = 0; i <= 2; i++)
        {
          prefix[i] = phoneNumber[i] - ascii_0;
          number[i] = phoneNumber[i+4] - ascii_0;
        }
        number[3] = phoneNumber[7] - ascii_0;
     }
    }

    if (input_done != 1)
      cout << "There is a problem with what you entered, please try again.\n";

  }

  // report
  cout << "I have your phone number as: ";

  for (i = 0; i<3; i++)
    cout << prefix[i];
  cout << "-";
  for (i = 0; i<4; i++)
    cout << number[i];
  cout << endl;

  return 0;
}

Answer 1

Сначала есть несколько незначительных проблем с подходом, который вы выбрали. Сначала вы используете много магических чисел, что делает ваш код сложным для поддержки и развития, так как вам нужно беспокоиться об обновлении большего количества кода, увеличивая вероятность появления ошибок. Во-вторых, нет необходимости конвертировать телефонный номер в отдельные номера, это просто усложняет управление значениями в долгосрочной перспективе. Вы также ожидаете, что дефис будет находиться в определенном месте, снова используя магические числа, которые затруднят добавление поддержки кодов городов. Следующий подход исключает использование магических чисел, а также использование индексов и массивов и использует преимущества итераторов.

#include <iostream>
#include <string>
using namespace std;

// main routine
int main()
{
    // local variables
    bool isValidNumber = false;
    string phoneNumber;

    while (isValidNumber == false)
    {
        // Ask the user for input
        cout << "Please enter your 10 digit phone number (i.e. 900-976-8008)" << endl;
        std::getline(cin, phoneNumber);

        isValidNumber = true;

        //  Validate and clean up the phone number
        for (std::string::iterator it = phoneNumber.begin(); it != phoneNumber.end();)
        {
            //  Check for characters we want to ignore
            if (*it == '(' || *it == ')' || *it == '-' || *it == ' ')
            {
                it = phoneNumber.erase(it);
            }
            //  Check for numbers since we really want to keep them
            else if (*it >= '0' || *it <= '9')
            {
                ++it;
            }
            //  Check characters that are considered invalid
            else
            {
                isValidNumber = false;
            }
        }

        //  Make sure the number of required digits are present. Add an additional
        //  check for numbers without area codes if you like.
        if (phoneNumber.size() != 10)
        {
            isValidNumber = false;
        }

        if (isValidNumber == false)
        {
            cout << "There is a problem with what you entered, please try again.\n";
        }
    }


    //  Split the number and print it
    std::string areacode;
    std::string prefix;
    std::string number;

    areacode = phoneNumber.substr(0, 3);
    prefix = phoneNumber.substr(3, 3);
    number = phoneNumber.substr(7, 4);

    std::cout
        << "I have your phone number as: ("
        << areacode
        << ") "
        << prefix
        << '-'
        << number
        << '\n';

    return 0;
}

Я также рекомендую не размещать using namespace std заявления в области имен. Это просто плохая форма и может привести к конфликтам имен.