- android - 多次调用 OnPrimaryClipChangedListener
- android - 无法更新 RecyclerView 中的 TextView 字段
- android.database.CursorIndexOutOfBoundsException : Index 0 requested, 光标大小为 0
- android - 使用 AppCompat 时,我们是否需要明确指定其 UI 组件(Spinner、EditText)颜色
最近我参加了一次工作面试,面试官给了我一段代码来解决它违反 SOLID 原则的问题,但我不是专家程序员,由于缺乏知识找不到任何问题,现在我请求你帮助我
这段代码有什么问题?
using System.Data.SqlClient;
using System.Linq;
using System.Web.Mvc;
using System.IO;
namespace InterviewTest.Controllers
{
public class EMailController : Controller
{
const string logFile = "log.txt";
// this is a method which takes an id or a username after searching in
//database this method sends an email to the searched id or username
//then this operation Is stored in a log file...
public ActionResult Index()
{
string connectionString = "Server=myServerAddress;Database=myDataBase;User Id=myUsername;Password = myPassword;";
SqlConnection con = new SqlConnection(connectionString);
con.Open();
SqlCommand cmd = new SqlCommand("select * from tblUsers where id = " + int.Parse(Request.QueryString["id"]) + " or username = '" + Request.QueryString["username"] + "'" , con);
SqlDataReader reader = cmd.ExecuteReader();
reader.Read();
string email = reader.GetString(2);
emailer.Instance.send(email);
FileStream fs = System.IO.File.OpenWrite("c:\\myapp\\" + logFile);
StreamWriter sw = new StreamWriter(fs);
sw.Write(DateTime.Now.ToString() + " sent email to " + email);
fs.Close();
return View();
}
// This is a class Which is responsible for sending email
public class emailer
{
private static emailer instance = null;
public static emailer Instance
{
get {
if (instance == null)
{
instance = new emailer();
}
return instance;
}
}
internal void send(string email) {
try {
// Suppose this piece of code has been implemented
}
catch (Exception ex)
{
Console.Write(ex.ToString());
}
}
}
}
}
最佳答案
(我假设你熟记这些原则,所以我不会解释它们的含义)
单一职责:因为 Controller 负责从数据库中检索数据(通常通过存储库或服务-存储库组合完成)以及电子邮件发送,这应该在服务中使用 Controller 。
开闭原则:email sender class和controller在同一个地方实现,显然不对扩展开放,对修改关闭
接口(interface)隔离:根本不用
依赖倒置:根本没有使用,例如存储库类和电子邮件发送服务应隐藏在接口(interface)(即 IEmailSender、IMyDataRepository)后面,并且 Controller 应使用那些不知道/不关心确切实现的接口(interface)。如果与依赖注入(inject)结合使用会更好 -> Controller 将通过使用 Unity、SimpleInjector 等获得在构造函数中实现这些接口(interface)的类的实例。
Liskov:没有使用类层次结构和接口(interface)等。
如果我必须实现这样的东西:
public class EmailController : Controller
{
// Interface segregation applied
private IEmailSendingService emailService;
private IUserService userService;
private ILoggingService loggingService
// Dependency inversion principle / Dependency injection applied
public EmailController(IEmailService mailSrvc, IUserservice usrSvc, ILoggingService log)
{
this.emailService = mailSrvc;
this.userService = usrSvc;
this.loggingService = log;
}
public ActionResult SendEmail()
{
try
{
var emailAddress = this.userService.GetEmail(...);
// validate email address, maybe through another service
if(Validator.ValidateEmail())
{
// Single responsibility applied
this.emailService.SendEmail(emailAddress);
}
}
catch(MailNotFoundException ex)
{
this.loggingService.LogError("Email address not found for xy user");
return NotFound();
}
catch(EmailSendingFailedException ex)
{
this.loggingService.LogError("Could not send email because xyz");
// return internalservererror etc.
}
catch(Exception ex)
{
this.loggingService.LogError("...");
}
// return whats needed
}
}
这个例子并不完美,但你可以理解它的要点:)
关于c# - 违反了哪些 SOLID 原则? (我应该怎么做才能解决这个 Controller 问题?),我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/54550903/
Yii::$app->runAction('new_controller/new_action', $params); 我相信这可以用来从另一个 Controller 调用 Controller Ac
这个问题类似于 this ,但我需要访问父成员(不是控制)。我不知道是否可以不使用依赖注入(inject)。 例如,我有一个父级,有一个成员调用用户,我需要从子 Controller 访问用户。 最佳
我有包含 2 个布局的根布局:- 选项面板- 绘制区域 我正在尝试的是访问 OptionsPaneController 中的 DrawAreaController 以调用其绘制方法。下面是 Optio
我的应用程序的 View Controller 层次结构设置如下: UIViewController | UITabBarController | UINavigationCo
我的应用程序的 View Controller 层次结构设置如下: UITabBarController | UINavigationController | | |
当我第一次为我目前在 Storyboard 中开发的应用程序创建基础布局时,我分两步完成: 选择我的 View Controller 并使用 Editor->Embed In->Navigation
设计要求: 显示用户可以选择的项目列表 选择一个项目后,使用后退按钮将用户带到一个新 View 。新 View 应在底部包含第一个屏幕中不存在的选项卡列表 单击选项卡中的项目时,应出现一个带有后退按钮
将父 Controller 设置为“parentCtrl as vm”,并将子 Controller 设置为“childCtrl as vmc”,以避免名称冲突,并且效果良好。 如何在子 Contro
我已经阅读了一些答案,例如关闭当前的 ViewController,但我的情况有所不同,因为我正在展示另一个 ViewController。 虽然我无法访问它的属性,但此代码显示了带有导航 Contr
如我所见,如果我们要实例化一个Model(例如,名为Post),我们只需调用: $post = new Post(); 现在,我还想实例化一个Controller(例如,名为Post,并为此 Cont
我已经疯狂地在整个网络上搜索解决我的问题的方法,但目前还没有。我的问题是我必须检查是否在 HTTP 请求中获得特定文本,该请求在一个 while 循环中,如果我这样做了,那么我应该离开循环并继续线程,
我想用this.get('controllers.pack.query');要得到App.PackQueryController在 App.PackController ,但失败了。 我认为问题是 E
我刚开始使用 Laravel。当我使用 codeigniter 或 zend 框架时,我可以将我的 Controller 组织到一个单独的目录中。例如,我可以创建“user/permission.ph
在 emberjs 前 2 我们可以从另一个 Controller 访问 Controller 或 Controller 中的任何方法 以下方式: App.get('router').get('nav
这可能是非常简单的实现,但我是 iOS 编程的新手,我似乎被卡住了。 所以,基本上,我有一个选项卡式应用程序。我决定除了标签栏之外还需要一个导航栏。为此,我放置了标签栏 Controller ,然后添
我有这个列表 Controller , define([ 'jquery', 'app' ], function ($,app) { app.controller("ListC
我有 3 个 Controller :RootController、FirstController 和 SecondController。我想从 RootController -> FirstCont
我有以下 Controller : /controllers/api/base_controller.rb /controllers/api/v1/articles_controller.rb 当为文
我是 Angular JS 的新手,尝试在另一个 Controller 中调用一个 Controller ,但出现以下错误。 ionic.bundle.js:21157 TypeError: $con
我有一个标签栏 Controller 和它的 3 个 child ,我还有另一个 View ,我制作了一个从 child 到 View Controller 的自定义转场,还有一个从 View Con
我是一名优秀的程序员,十分优秀!