是否从此代码正确绘制了此序列图？

Question

我必须仅使用这段代码来绘制序列图（作为大学作业的一部分）。这是我对图表的外观的想法，但是我真的很想如果有更多经验的人可以检查一下并给我任何反馈。 我最大的难题是以下几行： -私人警报警报= GlobalSettings.GetCurrentAlarmSetting（）; //我是否在此处创建一条创建消息（如我所做的那样），或者？ -lights = lightsController.GetAllLights（）; //我正确执行了吗？我不知道如何表示方程运算符。

不幸的是，我不得不将其裁剪为2张图片：

public class AlarmController
    {
        private Alarm alarm = GlobalSettings.GetCurrentAlarmSetting();
        private User user = GlobalSettings.GetHouseOwner();
        private LightsController lightsController = new LightsController();

        public void ActivateSecurityMeasures(Sensor sensor)
        {
            List<Light> lights; 
            if (alarm.IsActivated())
            {
                lights = lightsController.GetAllLights();
                alarm.ActivateSoundAlarm();
                user.SendNotification("Home alarm is activated");
            }
            else
                lights = lightsController.GetSensorLights(sensor); 

            foreach (Light item in lights)
            {   item.TurnOn(); }
        }
    }

Answer 1

似乎不应使用创建消息来创建警报和用户，因为您的源代码未显示这两个对象是从AlarmController创建的。它们只是从GlobalSettings中的静态操作中检索。似乎＃8.3，＃8.8，＃8.9和＃8.10消息也是不必要的，因为＃8.2和＃8.7返回光表示AlarmController引用了Light对象并可以在＃8.11中调用