Флуд из Помогите с подсчётом строк и символов (C/C++)

[Serg79]

Начало здесь

https://forum.shelek.ru/index.php/topic,17902.0.html

*************


LogRus, а это будет работать?
Вроде я С++ немного знаю, но при взгляде на Твой код начинаю понимать, что С++ то оказывается не знаю и самое главное, Я слабо понимаю то что на нем написано.

[Алексей++]

Serg79, та у Логруса инстинкт - всё переделывать на STL И я даже думаю, что это работает )

[Антон (LogRus)]

даже работает
Serg79, я тоже не знаю а выпендриваться люблю.
А написано на нём следующее:
1. создан функтор, конструктор которого принимает ссылку на счётчик
2. в функтор добавлен оператор () с одним аргументом (этот аргумент ему подсунет алгоритм)
3. использован алгоритм for_each который для каждого из элементов на интервале от str до str + sizeof(str) - 1 вызовет у переданного функтора оператор () и передаст туда элемент, в нашем случае char

Код полностью эквивалентен коду Алексей1153++ кстати кусок switch я от туда стащил

[Serg79]

Код полностью эквивалентен коду Алексей1153++ кстати кусок switch я от туда стащил

Да, кстати 'switch' Алексей1153++ понравился, неординарный подход . Кстати, генерируемый компилятором код при этом получается самым оптимальным.

[Алексей++]

оптимальный, но не наглядный - это я специально для Аполлона так сделал, пусть ещё и опечатку найдёт )

[Джон]

Лёш, не только опечатку, но и ошибку.

[Алексей++]

Джон, нет, тока очепятка, алгоритм работает

[Джон]

Ну он и с ошибкой работает. Ошибка не влияет на результат, но это ошибка поскольку ты выбрал такой подход.

[Алексей++]

ну даже интересно стало, что за ошибка ?

[Serg79]

Блин, Вы меня заинтриговали, собрал исходник:

Код:

#include <stdio.h>

int main(void)
{
        int count = 0;
        int c;

        while ((c = getchar()) != EOF) {
                switch(c) {
                        case 'a': c = 'A';
                        case 'A': count++;
                }
                putchar(c);
        }

        fprintf(stderr,"\n%d\n",count);
        return 0;
}

все работает. Подмена выполняется, считает правильно.

[Вад]

Алексей1153++, sizeof, надо полагать

[Serg79]

Алексей1153++, sizeof, надо полагать

Не, sizeof здесь отработает правильно.

[Алексей++]

Вад, что не так ? ) Статический массив, вернёт количество байт, как и положено

[Вад]

Serg79, здесь - да
Алексей1153++, вот именно, что он статический. Можешь представить себе задачу, где такое решение понадобится?

[Алексей++]

Вад, ну так если был не статический массив, я бы sizeof не применил

[Джон]

Serg79, так ты исправил ошибку.

[Джон]

Да не не заморачивайтесь. Тут проблема в break - 1. после него не стоит ; и 2. он там не нужен. Что из этих двух ошибка, а что опечатка выбирайте сами.

[Алексей++]

Джон, break ? ) Это не ошибка

а ; - это опечатка )))

[Джон]

зы Ну и ещё один момент, но тут спорный вопрос - проблема с условием. В условии не стоит, что надо подсчитывать количество 'А', а только 'а'. В этом случае решение не верно.

[Джон]

Джон, break ? ) Это не ошибка

Тогда всё правильно - именно ошибка. Иначе объясни зачем он там нужен? Ты ведь выбрал непонятный стиль решнеия.

[Serg79]

Походу Джон не вкурил как это работает.

[Алексей++]

Джон, break там не мешает абсолютно, я просто по привычке ставлю, чтобы в будущий блок не залесть потом Бывало такое.

ну а насчёт подсчёта ты прав, тогда надо так:

Код:

  if(str[i]=='a')
  {
     str[i]='A';
     dwdCountA++
  }

[Serg79]

Ну не знаю, в задании не говориться подсчитать количество маленьких 'a', а большие 'A' не считать, там говориться Дана строка символов найти в ней   количество символов  (а)  И если они  маленькие, то перевести их в большие буквы  (А), т.е. подсчитывать все 'a' но если они маленькие то сделать их большими 'A'.

По моему так, если я все правильно понял.

[Джон]

Serg79, эммм даю справочку - Джон вкурил, ещё с первого раза и без проверки на компиляторе. Впрочем как и LogRus, корый в совём коде тоже break не поставил. Смотрите внимательней. А наличие в программе лишнего ничего не делающего кода в моём понимании есть ошибка. Если вы так не считаете, тады ой. Вобще-то программер несёт ответственность за каждую строчку, за каждую буковку и должен обосновать её наличие в программе.  Ладно ещё когда код в несколько тысяч строк и там сам чёрт ногу сломит, но в данном случае код не такой уж и большой. ИМХО разумеется.

По моему так, если я все правильно понял.

Абсолютно правильно понял - СЧИТАТЬ нужно только 'a'. А теперь добавь в строку примера A и они тоже посчитаются по приведённому выше решению, что явно не оговорено в условии. Вобще-то задание должно выполнятся буквально и не додумываться доп. условиями. Анекдот про прогрммера и яйца знаешь?

[Алексей++]

Джон, я обосновал - это привычка, выработанная граблями))

например

switch(...)
{
   case 1:
   case 2://break - не поставил
}


а через год добавляем "3" с другой логикой

switch(...)
{
   case 1:
   case 2:
   case 3:
}

и глюк закопошился, если ты забыл после "2" приписать break

[Serg79]

Ну незнаю Джон, там говориться найти в ней   количество символов  (а)  И если они  маленькие, то перевести их в большие буквы  (А), это подразумевает что a могут попадать как маленькие так и большие, но если a маленькая, то ее надо сделать большой.

Ну а break в операторе switch ни коим боком не влияет на что либо находясь в конце оператора switch. Т.е. эти записи идентичны:

Код:

switch (c) {
    case 'a':
    case 'A':
}

и

Код:

switch (c) {
    case 'a':
    case 'A':
        break;
}

[Джон]

По поводу условий я поэтому сразу оговорился:

тут спорный вопрос - проблема с условием.

Поэтому дальнейший спор не имеет смысла и хорошо, что теперь есть решение для обоих случаев.

Ну а break в операторе switch ни коим боком не влияет на что либо

Да нууу? Серьёзно? А где я сказал что он на что-то влияет? С таким же успехом можно впендюрить куда-нить в код int nX = 345; и это тоже ни на что не повлияет. А ещё можно так записать:

Код:

switch (c) {
    case 'a':
    case 'A':
        break;
    deafult:
        break;
}

И это тоже тобе будет ИДЕНТИЧНАЯ запись. Но зачем это делать? Записи идентичны только с точки зрения копмилятора, но как препод я бы обязательно на зачёте спросил зачем там нужен break? Обоснование см. выше.

Лёш, в данном случае это не обоснование, а отмазка.

[Алексей++]

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

[Serg79]

На счет писать break или не писать то это зависит от того кого читаешь, одни пишут по этому поводу, что писать break в конце является хорошей идеей, другие по этому поводу вообще ничего не говорят. Но то, что break в конце может кого то вводить в заблуждение, я об этом первый раз слышу.

[Джон]

Лёш, да я-то понял, что на афтамате.

Serg79, где я сказал про заблуждение? Я говорю, что он НЕ НУЖЕН в данном коде. Вот так, уже проще некуда.
Я это сразу автоматически отметил "про себя" как только увидел лёшкин код. Кстати ты его тоже в своём примере не поставил, а ведь ещё не знал, что я имею ввиду. А почему? Ну и кому спорим?