654548066

Build Type

github

Committed by GitHub

Commit Message

Merge de8e1e19a into 54b1be009

Pull Request Pull Request #1056: Fix 967 transition break game

Run Details

2166 of 3426 branches covered (63.22%)

Branch coverage included in aggregate %.

7 of 8 new or added lines in 2 files covered. (87.5%)

11 existing lines in 6 files now uncovered.

5903 of 7658 relevant lines covered (77.08%)

39675.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 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)	7✔
20	{
21	_mapRuleManager = mapRuleManager;	7✔
22	_dice = dice;	7✔
23	}	7✔
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)	7!
39	{
40	throw new ArgumentNullException(nameof(draftRegions));	×
41	}
42
43	if (targetRegionCount <= 0)	7!
44	{
45	throw new ArgumentException("Целевое количество регионов должно быть больше 0.",	×
46	nameof(targetRegionCount));	×
47	}
48
49	var regionCountDiff = targetRegionCount - draftRegions.Length;	7✔
50	if (regionCountDiff <= 0)	7✔
51	{
52	return (RegionDraft[])draftRegions.Clone();	2✔
53	}
54
55	var availableSplitRegions = draftRegions.Where(x => x.Coords.Count() > 1);	5✔
56	var availableCoords = from region in availableSplitRegions	5✔
57	from coord in region.Coords.Skip(1)	5✔
58	select new RegionCoords(coord, region);	5✔
59
60	if (availableCoords.Count() < regionCountDiff)	5!
61	{
62	// Возможна ситуация, когда в принципе клеток меньше,
63	// чем требуется регионов.
64	// Даже если делать по одной клетки на регион.
65	// В этом случае ничего сделать нельзя.
66	// Передаём проблему вызывающему коду.
67	throw new CellularAutomatonException(	×
68	"Невозможно расщепить регионы на достаточное количество. Клеток меньше, чем требуется.");	×
69	}
70
71	var openRegionCoords = new List<RegionCoords>(availableCoords);	5✔
72	var usedRegionCoords = new List<RegionCoords>();	5✔
73
74	for (var i = 0; i < regionCountDiff; i++)	38✔
75	{
76	var coordRollIndex = _dice.Roll(0, openRegionCoords.Count - 1);	14✔
77	var regionCoordPair = openRegionCoords[coordRollIndex];	14✔
78	openRegionCoords.RemoveAt(coordRollIndex);	14✔
79	usedRegionCoords.Add(regionCoordPair);	14✔
80	}
81
82	var newDraftRegionList = new List<RegionDraft>();	5✔
83	var regionGroups = usedRegionCoords.GroupBy(x => x.Region)	5✔
84	.ToDictionary(x => x.Key, x => x.AsEnumerable());	5✔
85
86	foreach (var draftRegion in draftRegions)	20✔
87	{
88	if (regionGroups.TryGetValue(draftRegion, out var splittedRegionCoords))	5!
89	{
90	var splittedCoords = splittedRegionCoords.Select(x => x.Coords).ToArray();	5✔
91
92	var newCoordsOfCurrentRegion = draftRegion.Coords	5✔
93	.Except(splittedCoords)	5✔
94	.ToArray();	5✔
95
96	var recreatedRegionDraft = new RegionDraft(newCoordsOfCurrentRegion);	5✔
97	newDraftRegionList.Add(recreatedRegionDraft);	5✔
98
99	foreach (var splittedCoord in splittedCoords)	38✔
100	{
101	var newRegionDraft = new RegionDraft(new[] { splittedCoord });	14✔
102	newDraftRegionList.Add(newRegionDraft);	14✔
103	}
104	}
105	else
106	{
UNCOV 107	newDraftRegionList.Add(draftRegion);	×
108	}
109	}
110
111	return newDraftRegionList.ToArray();	5✔
112	}
113
114	public IEnumerable<RegionDraft> Process(IEnumerable<RegionDraft> sourceRegions)
115	{
116	var regionCountRule = _mapRuleManager.GetRuleOrNull<IRegionMinCountRule>();	7✔
117	if (regionCountRule is null)	7!
118	{
119	return sourceRegions;	×
120	}
121
122	var draftRegionArray = sourceRegions.ToArray();	7✔
123	return SplitRegionsForTransitions(draftRegionArray, regionCountRule.Count);	7✔
124	}
125
126	private sealed class RegionCoords
127	{
128	public RegionCoords(OffsetCoords coords, RegionDraft region)	4,726✔
129	{
130	Coords = coords;	4,726✔
131	Region = region ?? throw new ArgumentNullException(nameof(region));	4,726!
132	}	4,726✔
133
134	public OffsetCoords Coords { get; }
135
136	public RegionDraft Region { get; }
137	}
138	}
139	}

kreghek / Zilon_Roguelike / 654548066

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