Функция C # не возвращает строку, как ожидалось

Question

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

Функция 1

private void getRelatedNews(TaxonomyData taxData, string related, string contentTitle)
{
    foreach (TaxonomyItemData item in taxData.TaxonomyItems)
    {
        if (taxData.TaxonomyName.Equals(contentTitle) && taxData.TaxonomyItemCount != 0)
        {
            related += string.Format("<li><a href='{0}'\">{1}</a></li>", item.TaxonomyItemId.ToString(), item.TaxonomyItemId.ToString());
        }                   
    }
    // Show all its sub categories
    foreach (TaxonomyData cat in taxData.Taxonomy)
    {   
        getRelatedNews(cat, related, contentTitle);
    }   
    cbContent2.Text += related;
}

Функция 2

private string getRelatedNews(TaxonomyData taxData, string related, string contentTitle)
{
    foreach (TaxonomyItemData item in taxData.TaxonomyItems)
    {
        if (taxData.TaxonomyName.Equals(contentTitle) && taxData.TaxonomyItemCount != 0)
        {
            related += string.Format("<li><a href='{0}'\">{1}</a></li>", item.TaxonomyItemId.ToString(), item.TaxonomyItemId.ToString());
        }                   
    }
    // Show all its sub categories
    foreach (TaxonomyData cat in taxData.Taxonomy)
    {   
        getRelatedNews(cat, related, contentTitle);
    }   
    return(related);
}

Я думаю, что что-то идет не так в разнице между cbContent2.Text + = related и return (related) - любые идеи о том, как заставить Function2 выдавать тот же вывод, что и Function1, будут великими ...

Answer 1

Попробуйте изменить

getRelatedNews(cat, related, contentTitle);

на

related += getRelatedNews(cat, related, contentTitle);

Вы всегда должны стараться избегать "+ =" в цикле со строками.Поскольку они являются неизменяемыми, вы должны сделать копию для каждой итерации в цикле.Для небольшой коллекции вы можете не заметить снижения производительности, но для больших коллекций это оказывает огромное влияние на производительность.Попробуйте использовать StringBuilder () для построения строки.StringBuilder гораздо эффективнее.

Answer 2

// this code
     getRelatedNews(cat, related, contentTitle);
// changed to 
     related = getRelatedNews(cat, related, contentTitle);
// works

Answer 3

На самом деле проблема здесь:

private string getRelatedNews(TaxonomyData taxData, string related, string contentTitle)
{
    foreach (TaxonomyItemData item in taxData.TaxonomyItems)
    {
        if (taxData.TaxonomyName.Equals(contentTitle) && taxData.TaxonomyItemCount != 0)
        {
                related += string.Format("<li><a href='{0}'\">{1}</a></li>", item.TaxonomyItemId.ToString(), item.TaxonomyItemId.ToString());
         }                   
     }
     // Show all its sub categories
     foreach (TaxonomyData cat in taxData.Taxonomy)
     {   
    // this code
         getRelatedNews(cat, related, contentTitle);
    // should be changed to 
         related += getRelatedNews(cat, related, contentTitle);
     }   
     return(related);
 }

Answer 4

Как выглядит вызов функции? Вы делаете что-то подобное?

string blah = getRelatedNews(x, "", y);

Если это так, измените его на:

string buffer,blah = getRelatedNews(x, buffer, y);

Также измените эту строку

    getRelatedNews(cat, related, contentTitle);

до

    related += getRelatedNews(cat, related, contentTitle);

Answer 5

Проблема связана с (подождите) related. У вас происходит рекурсия В первой функции вы рекурсивно вызываете функцию и всегда объединяете результат с элементом управления в конце функции (достигается при каждом рекурсивном выполнении). Во-вторых, вы все еще действуете рекурсивно, но никогда не получаете результат от этих рекурсивных вызовов. Строка неизменна, related не обновляется этими рекурсивными вызовами функций автоматически. Начните фиксировать результат и оцените его, чтобы увидеть, соответствуют ли результаты ожидаемым.

Answer 6

Строки неизменны.Вы не можете изменить строку с помощью «+ =».

Вы устанавливаете свой параметр related на новую строку, но не изменяете строку, которая была передана.

Answer 7

вам нужно перейти на родство + = getRelatedNews (...)