602612224

Build Type

github

Committed by GitHub

Commit Message

Merge a14c2b82b into 8cc6744b8

Pull Request Pull Request #1047: Refactor

Run Details

2203 of 3420 branches covered (64.42%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

14 existing lines in 7 files now uncovered.

5954 of 7648 relevant lines covered (77.85%)

78292.03 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 JetBrains.Annotations;

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

kreghek / Zilon_Roguelike / 602612224

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