ASP.NET, EF로 블로그를 만들고 있는데 맞게 코딩하고 있는걸까요?

Question

저는 현재 ASP.NET Core + Entity Framework Core로 블로그를 만들려고 하고 있습니다.그런데 블로그에 태그 기능을 추가해서 만들고 싶은데 태그를 여러 개 중복되지 않게 추가하고 싶어서 BlogPostTag를 만들었고 [Table("blog_post_tag")] [Index("PostId", Name = "FK_blog_post_tag_blog_post")] [Index("TagId", Name = "FK_blog_post_tag_tag")] public partial class BlogPostTag { [Key] [Column("id", TypeName = "bigint(20)")] public long Id { get; set; } [Column("post_id", TypeName = "bigint(20)")] public long PostId { get; set; } [Column("tag_id", TypeName = "bigint(20)")] public long TagId { get; set; } [ForeignKey("PostId")] [InverseProperty("BlogPostTags")] public virtual BlogPost Post { get; set; } [ForeignKey("TagId")] [InverseProperty("BlogPostTags")] public virtual Tag Tag { get; set; } }그 결과 아래와 같이 정말 여러 리포지토리를 참조해야하고 트랜잭션을 위해서 UnitOfWork라는 것을 만들어서 사용하게 되었습니다.그리고 코드를 작성하였는데 public class BlogPostService : IBlogPostService { private readonly IBlogPostRepository _blogPostRepository; private readonly IUserRepository _userRepository; private readonly ITagRepository _tagRepository; private readonly IBlogPostTagRepository _blogPostTagRepository; private readonly IUnitOfWork _unitOfWork; public BlogPostService(IBlogPostRepository blogPostRepository, IUserRepository userRepository, ITagRepository tagRepository, IBlogPostTagRepository blogPostTagRepository, IUnitOfWork unitOfWork) { _blogPostRepository = blogPostRepository; _userRepository = userRepository; _tagRepository = tagRepository; _blogPostTagRepository = blogPostTagRepository; _unitOfWork = unitOfWork; } public async Task AddAsync(BlogPostRequestDto requestBlogPost) { Debug.Assert(requestBlogPost != null, "BlogPost must not be null"); Debug.Assert(!string.IsNullOrWhiteSpace(requestBlogPost.Email), "Email must not be null or empty"); Debug.Assert(!string.IsNullOrWhiteSpace(requestBlogPost.Title), "Title must not be null or empty"); Debug.Assert(!string.IsNullOrWhiteSpace(requestBlogPost.Content), "Content must not be null or empty"); Debug.Assert(requestBlogPost.BlogPostTags != null, "BlogPostTags must not be null"); try { await _unitOfWork.BeginTransactionAsync(); var user = await _userRepository.GetByEmailAsync(requestBlogPost.Email); Debug.Assert(user != null, "User must not be null"); var blogPost = new BlogPost { Title = requestBlogPost.Title, Content = requestBlogPost.Content, UserId = user.Id }; blogPost = await _blogPostRepository.AddAsync(blogPost); Debug.Assert(blogPost.Id > 0, "BlogPost Id must be greater than 0"); blogPost.BlogPostTags = new List<BlogPostTag>(); foreach (var tagName in requestBlogPost.BlogPostTags) { var tag = await _tagRepository.GetByNameAsync(tagName); if (tag == null) { tag = new Tag { Name = tagName }; await _tagRepository.AddAsync(tag); Debug.Assert(tag.Id > 0, "Tag Id must be greater than 0"); blogPost.BlogPostTags.Add(new BlogPostTag { PostId = blogPost.Id, TagId = tag.Id, Post = blogPost, Tag = tag }); } } foreach (var blogPostTag in blogPost.BlogPostTags) { await _blogPostTagRepository.AddAsync(blogPostTag); Debug.Assert(blogPostTag.Id > 0, "BlogPostTag Id must be greater than 0"); } await _unitOfWork.CommitTransactionAsync(); } catch (Exception ex) { Debug.WriteLine(ex.Message); Debug.Assert(false, "An error occurred while adding a blog post"); await _unitOfWork.RollbackTransactionAsync(); } } }이렇게 내가 하고 있는 게 맞나 싶은 코드가 나왔습니다. (물론 Copilot의 도움을 많이 받아서 그렇겠지만요...)아마도 Spring에서도 비슷한 형태일텐데 이렇게 DB에 BlogPost를 추가하고, Tag를 추가하고, 그리고 BlogPost와 Tag를 연결하는 BlogPostTag를 추가하는 순서로 코드가 나오는 게 맞는 걸까요?아니면 제가 복잡하게 생각하거나 잘못 생각하고 있는걸까요?아직 DB 연결조차 확인을 못해서 유닛 테스트 조차 못해보고 질문 올려서 죄송합니다.코드를 봐달라고 부탁할 사람이 없어서 오키에 올리게 되었습니다.

개나소나고생 · Accepted Answer

Exception에 Debug.WriteLine이라는건 이거는 정말 옛날방식 코드인데요..로그 파일 남기면 되지 않나요?

캐티 · Answer

일부러 복잡한 것이 아니고 하드 코딩 자체가 복잡해지는 경향이 있스빈다. 애초에 만국 플랫폼 공통되는 부분이지 않나요.database 서비스를 넣고 컨트롤러와 각 페이지를 출력하는 부분으로 나누었었는데 머 정답이 없스빈다. 많이 하다 보면 이케이케 하여야 깔끔하고 간결해진다는 결론에 이르러서 더 많이 고쳐보면 좋스빈다.

ASP.NET, EF로 블로그를 만들고 있는데 맞게 코딩하고 있는걸까요?

2개의 답변

Q&A 목록