Тренировка навыков быстрого программирования

[Bepec]

Это и есть недостаток этого сервиса Моя программа не проходила пятый тест, причём все входные данные указанные в задаче и всевозможные их вариации проходят успешно

[8Observer8]

Это и есть недостаток этого сервиса Моя программа не проходила пятый тест, причём все входные данные указанные в задаче и всевозможные их вариации проходят успешно

Спасибо за ответ! У меня задача не проходит первый тест

[Bepec]

Возможно ты неправильно понял - входные данные в этом сервисе неизвестны. Потому определить где ошибка невозможно

[8Observer8]

Возможно ты неправильно понял - входные данные в этом сервисе неизвестны. Потому определить где ошибка невозможно

Какие входные данные для моей программы могут могут приводить к неверному результату? Есть варианты?

[Bepec]

Неа В том вся и прелесть

[8Observer8]

Я имел ввиду, что может я задачу неверно понял? Наверняка, в моём алгоритме ошибка. Скорее всего есть такие входные данные, которые моя программа неверно обрабатывает. Буду думать.

[Igors]

1)

Код

C++ (Qt)
void bulls_and_cows(int first, int second, int &nbulls, int &ncows) 
 

Часто пишут так

Код

C++ (Qt)
void bulls_and_cows(int first, int second, int * nbulls, int * ncows) 
 

Смысл: вызывающий видит вход/выход, Если же ссылки, то можно так

Код

C++ (Qt)
void bulls_and_cows(int iFirst, int iSecond, int & oBulls, int & oCows) 
 

2)

Код

C++ (Qt)
    // Проверяем, что числа четырёхзначные
    if ( (vec_first.size() != nsize) && (vec_second.size() != nsize) ) {
        return;
    }
 

Правильно || (вместо &&), и возврат кода ошибки никто не отменял. И если у Вас цикл до nsize, то откуда возьмется др число эл-тов?

3) Не выделена ф-ция создания массива/контейнера из числа, все свалено в  bulls_and_cows, в результате она смотрится тяжеловато.

4) Неуемное использование std:: Чем не устраивал простенький char[4] (вместо векторов)? Да, это не общее решение - но нужна ли здесь какая-то общность? С др стороны по скорости "просто массив" заметно выигрывает. Вообще зачем подсчеты в 2 циклах? Почему бы не написать непредвзято и незатейливо, напр

Код

C++ (Qt)
void CalcBullsCows( const char * s1, const char * s2, int count, int & num_Bull, int & num_Cows )
{
  num_Bull = num_Cows = 0;
  for (int i = 0; i < count; ++i) {
    if (s1[i] == s2[i]) {
     ++num_Bulls;
     continue;
    }
 
   for (int j = 0; j < count; ++j) {
    if (s1[j] == s2[i]) 
     if (s1[j] != s2[j])  { // на всякий случай учтем что цифры могут повторяться
       ++num_Cows;
        break;   
     }
    }
  }
}

[8Observer8]

Спасибо огромное! Обязательно всё разберу, даже может за сегодня

У меня вопрос. Какую букву лучше поставить перед именем "fileName" (см. функцию ниже) . С одной стороны это имя выходного файла, а с другой - входной параметр для функции:

Код

C++ (Qt)
/**
 * Записываем результат выходной в файл
 *
 * @param oFileName Имя выходного файла
 * @param vec Массив, который нужно записать в файл
 * @return Код ошибки: 0 - если ошибок нет
 *                     1 - если файл не удалось открыть
 *                     2 - если в файле не удалось записать
 */
int writeResult(const std::string& oFileName, int iBulls, int iCows) {
 
    // Открываем файл для записи
    std::ofstream out;
    out.open(oFileName.c_str());
    if (!out.is_open()) {
        return 1;
    }
 
    // Записываем данные в файл
    out << nbulls << " " << ncows << std::endl;
 
    // Закрываем файл и возвращаем код ошибки
    out.close();
    return 0;
}
 

[Old]

У меня вопрос. Какую букву лучше поставить перед именем "fileName" (см. функцию ниже) . С одной стороны это имя выходного файла, а с другой - входной параметр для функции:

Не надо никаких букв, просто давайте нормальные имена параметрам и переменным.
Это имя результирующего файла.

[OKTA]

Лично по мне удобна вот такая нотация - один раз определил _In_ и _Out_ и пользуйся на здоровье)

Код:

void func(
  _Out_  int a,
  _In_   int b,
 );

[8Observer8]

Лично по мне удобна вот такая нотация - один раз определил _In_ и _Out_ и пользуйся на здоровье)

Код:

void func(
  _Out_  int a,
  _In_   int b,
 );

Первый раз такое вижу. И это будет компилироваться? Пожалуйста, напишите маленький примерчик по вызову такой функции

Не надо никаких букв, просто давайте нормальные имена параметрам и переменным.
Это имя результирующего файла.

Old, мне очень хочется узнать Ваше мнение, куда прижимать значок '&'? И ещё вопрос: в следующей функции уместны ли буквы 'i' и 'o' перед именами:  Bulls и Cows

Код

C++ (Qt)
void bulls_and_cows(int iFirst, int iSecond, int & oBulls, int & oCows);

Лично мне хочеться написать вот так:

Код

C++ (Qt)
void bulls_and_cows(int firstNumber, int secondNumber, int & nBulls, int & nCows);

[Igors]

У меня вопрос. Какую букву лучше поставить перед именем "fileName" (см. функцию ниже) . С одной стороны это имя выходного файла, а с другой - входной параметр для функции:

Все меряется "относительно ф-ции" поэтому "i". Если используется и как вход и как выход то "io". Если файл для записи, то неплохо напр "iOutFileName". Эти соглашения необязательны, но многие их используют, и в этом есть смысл.

А вот возвращать 1 (0) - явный признак что "не профи" Обязательно должны быть enum или константы или даже #define (что кому нравится) - но никак не числа вбитые в код.

Код

C++ (Qt)
 
if (!out.is_open()) {
 return errFileOpen;

[carrygun]

...куда прижимать значок '&'?

По этому поводу можно почитать вот это.

[OKTA]

Да все просто! Делаете define и затем используете в объявлении и определении функций) препроцессор потом уберет все эти _In_ & _Out_!

Код:

#define _In_
#define _Out_

void func(
  _Out_  int a,
  _In_   int b,
 );

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

Код:

BOOL WINAPI CryptImportPublicKeyInfoEx2(
  _In_   DWORD dwCertEncodingType,
  _In_   PCERT_PUBLIC_KEY_INFO pInfo,
  _In_   DWORD dwFlags,
  _In_   void *pvAuxInfo,
  _Out_  BCRYPT_KEY_HANDLE *phKey
);

[Igors]

Лично мне хочеться написать вот так:

Так и пишите, не надо следовать каким-то догмам и.т.п. Знать "как делают другие" полезно - но не более того

When strictly following a rule makes your code look bad, feel free to break it