Подсчет символа «е»

Question

Подсчет символа «е»

1

Привет, мне нужно создать метод, который принимает слово message, и подсчитывает, сколько раз символ e появляется, это то, что у меня есть, но я всегда получаю 0. любые предложения?

public class run
{
  public static void main(String[] args)
  {


  String message ="message";


  int count=0;
  for(int i=0; i>=message.length()-1;i++)
  {

    char ch = message.charAt(i);
    char e='e';
    if( ch == e)
    {
     count = count +1;

    }

  }

  System.out.println(count);



 }

}

user2872194 13 март 2014, в 05:38

Источник

Теги:

java

character

3 ответа

0

Вот еще один рекурсивный метод:

public int countE(String str) {

    int count = 0;

    if (str.charAt(0) == 'e') 
        count++;

    if (str.length() > 1) {
        count += countE (str.substring(1));
    }

    return count;
}

Scary Wombat 13 март 2014, в 03:37

0

@ambigram_maker спасибо за редактирование.
Scary Wombat 13 март 2014, в 05:30
0

И когда вы передаете строку с большим количеством символов e , что, по вашему мнению, произойдет с точки зрения стековых фреймов? Тестовая программа, которую я написал, работает со строкой 4096 е, но при увеличении до 8192 мы видим Exception in thread "main" java.lang.StackOverflowError . Рекурсия - неподходящий инструмент для этой работы.
paxdiablo 13 март 2014, в 05:37
0

@paxdiablo На самом деле количество e s не имело бы значения, это была бы длина сообщения. Я согласен , что я не буду лично использовать рекурсию для этой проблемы , а также я хотел бы избежать разбора String с тысячами e с в них.
Scary Wombat 13 март 2014, в 05:53

Показать ещё 1 комментарий

0

Это рекурсивное решение:

public static int countChar (String message, char e)
{
    int charOccurences = 0;

    for (int i = 0 ; i < message.length () ; i++)
    {
        if (message.charAt (i) == e)
        {
            charOccurences++;
            message = message.substring (0, i) + message.substring (i + 1);
            return charOccurences + countChar (message, e); 
        }

    }
    return charOccurences;
}

Noob 13 март 2014, в 02:37

0

Почему я получаю отрицательные отзывы за все ???? Я пытаюсь помочь. Есть несколько способов решить проблему ... Я просто предлагаю другой. Почему это заслуживает отрицательных голосов?
Noob 13 март 2014, в 04:42
0

Если бы это не было гибридом цикла и рекурсивной логики, это могло бы чего-то стоить.
Scary Wombat 13 март 2014, в 04:42
0

Не плачь, Нуб. Исправьте свои проблемы (помните, что мы все еще учимся), и отрицательное мнение может исчезнуть.
Scary Wombat 13 март 2014, в 04:43
0

Я честно пытаюсь помочь. Но все, что я делаю, похоже, получает отрицательные отзывы, даже если это правильный ответ.
Noob 13 март 2014, в 04:45
0

Нуб, это, вероятно, становится недооцененным, потому что он использует рекурсию совершенно ненужным способом. Рекурсия действительно имеет определенную элегантность , но я не думаю , что это подходящая задача для того , чтобы решить. Это похоже на def add(a,b): if b == 0 return a else return add (a+1,b-1) :-) Другими словами, рекурсия хороша, если она может быстро сократить пространство потенциального решения ( а-ля бинарный поиск). В противном случае накладные расходы на вызовы функций (во времени и пространстве стека) могут быть проблематичными.
paxdiablo 13 март 2014, в 04:45
0

goo.gl/zcKhee
Noob 13 март 2014, в 04:47
0

@Noob Посмотрите на мой ответ. В вашей аналогии со связанной картиной ваш метод подобен воздушному шарику на автомобиле. Это все еще работает, но ...
Scary Wombat 13 март 2014, в 04:51
0

... это делает его более удивительным. Я не вижу проблемы.
Noob 13 март 2014, в 04:52
0

@Noob Жаль, это означает, что вы не учитесь. Зачем использовать циклическую логику и рекурсивную логику. Один или другой будет в порядке. Как сказал такой умный чувак: «Все должно быть сделано как можно проще, но не проще».
Scary Wombat 13 март 2014, в 05:33

Показать ещё 7 комментариев

Ещё вопросы

@ambigram_maker спасибо за редактирование.
И когда вы передаете строку с большим количеством символов e , что, по вашему мнению, произойдет с точки зрения стековых фреймов? Тестовая программа, которую я написал, работает со строкой 4096 е, но при увеличении до 8192 мы видим Exception in thread "main" java.lang.StackOverflowError . Рекурсия - неподходящий инструмент для этой работы.
@paxdiablo На самом деле количество e s не имело бы значения, это была бы длина сообщения. Я согласен , что я не буду лично использовать рекурсию для этой проблемы , а также я хотел бы избежать разбора String с тысячами e с в них.
Почему я получаю отрицательные отзывы за все ???? Я пытаюсь помочь. Есть несколько способов решить проблему ... Я просто предлагаю другой. Почему это заслуживает отрицательных голосов?
Если бы это не было гибридом цикла и рекурсивной логики, это могло бы чего-то стоить.
Не плачь, Нуб. Исправьте свои проблемы (помните, что мы все еще учимся), и отрицательное мнение может исчезнуть.
Я честно пытаюсь помочь. Но все, что я делаю, похоже, получает отрицательные отзывы, даже если это правильный ответ.
Нуб, это, вероятно, становится недооцененным, потому что он использует рекурсию совершенно ненужным способом. Рекурсия действительно имеет определенную элегантность , но я не думаю , что это подходящая задача для того , чтобы решить. Это похоже на def add(a,b): if b == 0 return a else return add (a+1,b-1) :-) Другими словами, рекурсия хороша, если она может быстро сократить пространство потенциального решения ( а-ля бинарный поиск). В противном случае накладные расходы на вызовы функций (во времени и пространстве стека) могут быть проблематичными.
@Noob Посмотрите на мой ответ. В вашей аналогии со связанной картиной ваш метод подобен воздушному шарику на автомобиле. Это все еще работает, но ...
... это делает его более удивительным. Я не вижу проблемы.
@Noob Жаль, это означает, что вы не учитесь. Зачем использовать циклическую логику и рекурсивную логику. Один или другой будет в порядке. Как сказал такой умный чувак: «Все должно быть сделано как можно проще, но не проще».

paxdiablo · Accepted Answer · 2014-03-13T02-32-00.000Z

for(int i=0; i>=message.length()-1;i++)

Это никогда не войдет в цикл (за исключением некоторых случаев кратковременного сообщения, где он будет оставаться в цикле довольно долгое время), так как у вас есть смысл сравнения по неправильному пути. Тебе нужно:

for (int i = 0; i < message.length(); i++)

И вам не нужны эти дополнительные переменные, это будет хорошо:

int count = 0;
for (int i = 0; i < message.length(); i++)
    if (message.charAt(i) == 'e')
        count++;

О человек, спасибо! Я не знаю, почему я не мог этого увидеть!