904919801

Build # 904919801

Build Type

github

Committed by GitHub

Commit Message

Merge 1afea8ac5 into df45d3ffc

Pull Request Pull Request #1168: Feature hit and death sounds

Run Details

2274 of 3706 branches covered (61.36%)

Branch coverage included in aggregate %.

15 of 15 new or added lines in 4 files covered. (100.0%)

53 existing lines in 19 files now uncovered.

6102 of 8077 relevant lines covered (75.55%)

33413.48 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

81.69

/Zilon.Core/Zilon.Core/MapGenerators/CellularAutomatonStyle/SplitToTargetCountRegionPostProcessor.cs

﻿using System;
using System.Collections.Generic;
using System.Linq;

using Zilon.Core.CommonServices.Dices;

namespace Zilon.Core.MapGenerators.CellularAutomatonStyle
{
    /// <summary>
    /// Try to split regions to smaller.
    /// </summary>
    internal class SplitToTargetCountRegionPostProcessor : IRegionPostProcessor
    {
        private readonly IDice _dice;
        private readonly IMapRuleManager _mapRuleManager;

        public SplitToTargetCountRegionPostProcessor(IMapRuleManager mapRuleManager, IDice dice)
        {
            _mapRuleManager = mapRuleManager;
            _dice = dice;
        }

        /// <summary>
        /// Метод отделяет от существующих регионов ячйки таким образом,
        /// чтобы суммарно на карте число регионов равнялось числу переходов + 1 (за стартовый).
        /// Сейчас все отщеплённые регионы в первую отщепляются от произвольных.
        /// Уже одноклеточные регионы не участвуют в расщеплении.
        /// </summary>
        /// <param name="draftRegions"> Текущие регионы на карте. </param>
        /// <param name="targetRegionCount"> Целевое число регионов. </param>
        /// <returns> Возвращает новый массив черновиков регионов. </returns>
        private RegionDraft[] SplitRegionsForTransitions(
            RegionDraft[] draftRegions,
            int targetRegionCount)
        {
            if (draftRegions == null)
            {
                throw new ArgumentNullException(nameof(draftRegions));
            }

            if (targetRegionCount <= 0)
            {
                throw new ArgumentException("Целевое количество регионов должно быть больше 0.",
                    nameof(targetRegionCount));
            }

            var regionCountDiff = targetRegionCount - draftRegions.Length;
            if (regionCountDiff <= 0)
            {
                return (RegionDraft[])draftRegions.Clone();
            }

            var availableSplitRegions = draftRegions.Where(x => x.Coords.Count() > 1);
            var availableCoords = from region in availableSplitRegions
                                  from coord in region.Coords.Skip(1)
                                  select new RegionCoords(coord, region);

            if (availableCoords.Count() < regionCountDiff)
            {
                // Возможна ситуация, когда в принципе клеток меньше,
                // чем требуется регионов.
                // Даже если делать по одной клетки на регион.
                // В этом случае ничего сделать нельзя.
                // Передаём проблему вызывающему коду.
                throw new CellularAutomatonException(
                    "Невозможно расщепить регионы на достаточное количество. Клеток меньше, чем требуется.");
            }

            var openRegionCoords = new List<RegionCoords>(availableCoords);
            var usedRegionCoords = new List<RegionCoords>();

            for (var i = 0; i < regionCountDiff; i++)
            {
                var coordRollIndex = _dice.Roll(0, openRegionCoords.Count - 1);
                var regionCoordPair = openRegionCoords[coordRollIndex];
                openRegionCoords.RemoveAt(coordRollIndex);
                usedRegionCoords.Add(regionCoordPair);
            }

            var newDraftRegionList = new List<RegionDraft>();
            var regionGroups = usedRegionCoords.GroupBy(x => x.Region)
                .ToDictionary(x => x.Key, x => x.AsEnumerable());

            foreach (var draftRegion in draftRegions)
            {
                if (regionGroups.TryGetValue(draftRegion, out var splittedRegionCoords))
                {
                    var splittedCoords = splittedRegionCoords.Select(x => x.Coords).ToArray();

                    var newCoordsOfCurrentRegion = draftRegion.Coords
                        .Except(splittedCoords)
                        .ToArray();

                    var recreatedRegionDraft = new RegionDraft(newCoordsOfCurrentRegion);
                    newDraftRegionList.Add(recreatedRegionDraft);

                    foreach (var splittedCoord in splittedCoords)
                    {
                        var newRegionDraft = new RegionDraft(new[] { splittedCoord });
                        newDraftRegionList.Add(newRegionDraft);
                    }
                }
                else
                {
                    newDraftRegionList.Add(draftRegion);
                }
            }

            return newDraftRegionList.ToArray();
        }

        public IEnumerable<RegionDraft> Process(IEnumerable<RegionDraft> sourceRegions)
        {
            var regionCountRule = _mapRuleManager.GetRuleOrNull<IRegionMinCountRule>();
            if (regionCountRule is null)
            {
                return sourceRegions;
            }

            var draftRegionArray = sourceRegions.ToArray();
            return SplitRegionsForTransitions(draftRegionArray, regionCountRule.Count);
        }

        private sealed class RegionCoords
        {
            public RegionCoords(OffsetCoords coords, RegionDraft region)
            {
                Coords = coords;
                Region = region ?? throw new ArgumentNullException(nameof(region));
            }

            public OffsetCoords Coords { get; }

            public RegionDraft Region { get; }
        }
    }
}

1	using System;
2	using System.Collections.Generic;
3	using System.Linq;
4
5	using Zilon.Core.CommonServices.Dices;
6
7	namespace Zilon.Core.MapGenerators.CellularAutomatonStyle
8	{
9	/// <summary>
10	/// Try to split regions to smaller.
11	/// </summary>
12	internal class SplitToTargetCountRegionPostProcessor : IRegionPostProcessor
13	{
14	private readonly IDice _dice;
15	private readonly IMapRuleManager _mapRuleManager;
16
17	public SplitToTargetCountRegionPostProcessor(IMapRuleManager mapRuleManager, IDice dice)	7✔
18	{
19	_mapRuleManager = mapRuleManager;	7✔
20	_dice = dice;	7✔
21	}	7✔
22
23	/// <summary>
24	/// Метод отделяет от существующих регионов ячйки таким образом,
25	/// чтобы суммарно на карте число регионов равнялось числу переходов + 1 (за стартовый).
26	/// Сейчас все отщеплённые регионы в первую отщепляются от произвольных.
27	/// Уже одноклеточные регионы не участвуют в расщеплении.
28	/// </summary>
29	/// <param name="draftRegions"> Текущие регионы на карте. </param>
30	/// <param name="targetRegionCount"> Целевое число регионов. </param>
31	/// <returns> Возвращает новый массив черновиков регионов. </returns>
32	private RegionDraft[] SplitRegionsForTransitions(
33	RegionDraft[] draftRegions,
34	int targetRegionCount)
35	{
36	if (draftRegions == null)	7!
37	{
38	throw new ArgumentNullException(nameof(draftRegions));	×
39	}
40
41	if (targetRegionCount <= 0)	7!
42	{
43	throw new ArgumentException("Целевое количество регионов должно быть больше 0.",	×
44	nameof(targetRegionCount));	×
45	}
46
47	var regionCountDiff = targetRegionCount - draftRegions.Length;	7✔
48	if (regionCountDiff <= 0)	7✔
49	{
50	return (RegionDraft[])draftRegions.Clone();	4✔
51	}
52
53	var availableSplitRegions = draftRegions.Where(x => x.Coords.Count() > 1);	3✔
54	var availableCoords = from region in availableSplitRegions	3✔
55	from coord in region.Coords.Skip(1)	3✔
56	select new RegionCoords(coord, region);	3✔
57
58	if (availableCoords.Count() < regionCountDiff)	3!
59	{
60	// Возможна ситуация, когда в принципе клеток меньше,
61	// чем требуется регионов.
62	// Даже если делать по одной клетки на регион.
63	// В этом случае ничего сделать нельзя.
64	// Передаём проблему вызывающему коду.
65	throw new CellularAutomatonException(	×
66	"Невозможно расщепить регионы на достаточное количество. Клеток меньше, чем требуется.");	×
67	}
68
69	var openRegionCoords = new List<RegionCoords>(availableCoords);	3✔
70	var usedRegionCoords = new List<RegionCoords>();	3✔
71
72	for (var i = 0; i < regionCountDiff; i++)	28✔
73	{
74	var coordRollIndex = _dice.Roll(0, openRegionCoords.Count - 1);	11✔
75	var regionCoordPair = openRegionCoords[coordRollIndex];	11✔
76	openRegionCoords.RemoveAt(coordRollIndex);	11✔
77	usedRegionCoords.Add(regionCoordPair);	11✔
78	}
79
80	var newDraftRegionList = new List<RegionDraft>();	3✔
81	var regionGroups = usedRegionCoords.GroupBy(x => x.Region)	3✔
82	.ToDictionary(x => x.Key, x => x.AsEnumerable());	3✔
83
84	foreach (var draftRegion in draftRegions)	14✔
85	{
86	if (regionGroups.TryGetValue(draftRegion, out var splittedRegionCoords))	4!
87	{
88	var splittedCoords = splittedRegionCoords.Select(x => x.Coords).ToArray();	4✔
89
90	var newCoordsOfCurrentRegion = draftRegion.Coords	4✔
91	.Except(splittedCoords)	4✔
92	.ToArray();	4✔
93
94	var recreatedRegionDraft = new RegionDraft(newCoordsOfCurrentRegion);	4✔
95	newDraftRegionList.Add(recreatedRegionDraft);	4✔
96
97	foreach (var splittedCoord in splittedCoords)	30✔
98	{
99	var newRegionDraft = new RegionDraft(new[] { splittedCoord });	11✔
100	newDraftRegionList.Add(newRegionDraft);	11✔
101	}
102	}
103	else
104	{
UNCOV 105	newDraftRegionList.Add(draftRegion);	×
106	}
107	}
108
109	return newDraftRegionList.ToArray();	3✔
110	}
111
112	public IEnumerable<RegionDraft> Process(IEnumerable<RegionDraft> sourceRegions)
113	{
114	var regionCountRule = _mapRuleManager.GetRuleOrNull<IRegionMinCountRule>();	7✔
115	if (regionCountRule is null)	7!
116	{
117	return sourceRegions;	×
118	}
119
120	var draftRegionArray = sourceRegions.ToArray();	7✔
121	return SplitRegionsForTransitions(draftRegionArray, regionCountRule.Count);	7✔
122	}
123
124	private sealed class RegionCoords
125	{
126	public RegionCoords(OffsetCoords coords, RegionDraft region)	3,124✔
127	{
128	Coords = coords;	3,124✔
129	Region = region ?? throw new ArgumentNullException(nameof(region));	3,124!
130	}	3,124✔
131
132	public OffsetCoords Coords { get; }
133
134	public RegionDraft Region { get; }
135	}
136	}
137	}

kreghek / Zilon_Roguelike / 904919801

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous