4

Estou fazendo um exercício em C# e não estou conseguindo encontrar o erro no meu código, pois já verifiquei a resolução do problema e não encontro a diferença na sintaxe do código que está acarretando o erro.

Estou recebendo o seguinte retorno no console:

Process is terminating due to StackOverflowException.

Segue abaixo os códigos:

Program.cs

using System;
using Course.Entities;

namespace Course
{
    class Program
    {
        static void Main(string[] args)
        {
            Comment c1 = new Comment("Have a nice trip!");
            Comment c2 = new Comment("Wow that's awesome!");
            Post p1 = new Post(
                DateTime.Parse("21/06/2018 13:05:44"),
                "Travelling to New Zealand",
                "I'm going to visit this wonderful country!",
                12);
            p1.AddComment(c1);
            p1.AddComment(c2);

            Comment c3 = new Comment("Good night");
            Comment c4 = new Comment("May the force be with you");
            Post p2 = new Post(
                DateTime.Parse("28/07/2018 23:14:19"),
                "Good night guys",
                "See you tomorrow",
                5);
            p2.AddComment(c3);
            p2.AddComment(c4);

            Console.WriteLine(p1);
            Console.WriteLine(p2);
        }
    }
}

Comment.cs

namespace Course.Entities
{
    class Comment
    {
        public string Text { get; set; }

        public Comment()
        {

        }

        public Comment(string text)
        {
            Text = text;
        }
    }
}

Post.cs

    using System;
    using System.Collections.Generic;
    using System.Text;

    namespace Course.Entities
    {
        class Post
        {
            public DateTime Moment { get; set; }
            public string Title { get; set; }
            public string Content { get; set; }
            public int Likes { get; set; }
            public List<Comment> Comments { get; set; } = new List<Comment>();

            public Post()
            {

            }

            public Post(DateTime moment, string title, string content, int likes)
            {
                Moment = moment;
                Title = title;
                Content = content;
                Likes = likes;
            }

            public void AddComment(Comment comment)
            {
                Comments.Add(comment);
            }
            public void RemoveComment(Comment comment)
            {
                Comments.Remove(comment);
            }

            public override string ToString()
            {
                StringBuilder sb = new StringBuilder();
                sb.AppendLine(Title);
                sb.Append(Likes);
                sb.Append(" Likes - ");
                sb.AppendLine(Moment.ToString("dd/MM/yyyy HH:mm:ss"));
                sb.AppendLine(Content);
                sb.AppendLine("Comments:");
                foreach (Comment c in Comments)
                {
                    sb.AppendLine(c.Text);
                }
                return ToString();
            }
        }

}

1 Resposta 1

4

O problema é o ToString(). Ali está chamando ele mesmo na hora de retornar alguma coisa, então entra em laço infinito e quebra. Acho que esqueceu de usar o objeto de texto ali, seria algo assim:

return sb.ToString();

Esse uso de ToString() é errado e abusivo.

Eu acho que poderia melhorar algumas coisas, por exemplo permitir que o construtor aceite os comentários. Mas isso é algo para se pensar. Eu não sei se não daria para evitar esse StringBuilder.

Eu dei uma modernizada e simplificada (eu acho mais fácil de ler) e ficou assim:

using System;
using static System.Console;
using System.Collections.Generic;
using System.Text;

class Program {
    static void Main() {
        var p1 = new Post(DateTime.Now, "Travelling to New Zealand", "I'm going to visit this wonderful country!", 12);
        p1.AddComment(new Comment("Have a nice trip!"));
        p1.AddComment(new Comment("Wow that's awesome!"));
        var p2 = new Post(DateTime.Now, "Good night guys",  "See you tomorrow", 5);
        p2.AddComment(new Comment("Good night"));
        p2.AddComment(new Comment("May the force be with you"));
        WriteLine(p1);
        WriteLine(p2);
    }
}

class Comment {
    public string Text { get; set; }
    public Comment() {}
    public Comment(string text) => Text = text;
}
class Post {
    public DateTime Moment { get; set; }
    public string Title { get; set; }
    public string Content { get; set; }
    public int Likes { get; set; }
    public List<Comment> Comments { get; set; } = new List<Comment>();
    public Post() {}
    public Post(DateTime moment, string title, string content, int likes) {
        Moment = moment;
        Title = title;
        Content = content;
        Likes = likes;
    }
    public void AddComment(Comment comment) => Comments.Add(comment);
    public void RemoveComment(Comment comment) => Comments.Remove(comment);
    public override string ToString() {
        var sb = new StringBuilder();
        sb.AppendLine(Title);
        sb.Append(Likes);
        sb.Append(" Likes - ");
        sb.AppendLine(Moment.ToString("dd/MM/yyyy HH:mm:ss"));
        sb.AppendLine(Content);
        sb.AppendLine("Comments:");
        foreach (Comment c in Comments) sb.AppendLine(c.Text);
        return sb.ToString();
    }
}

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

1
  • 1
    Nossa, que falta de atenção a minha. Você tem razão, eu esqueci de adicionar o objeto sb, por isso que estava dando o erro. Estava analisando o seu código e não conhecia essa forma de usar => no método. Muito obrigado pela ajuda!
    – wmazoni
    Commented 24/10/2019 às 7:45

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .