Михаил Ливач

В общем случае избежать "цикла в цикле" позволяет использование индексации массива идентификатором и - соответственно - обращение по индексу вместо перебора.
В данном случае вторую коллекцию проиндексировать можно с помощью метода keyBy('id');

И тогда останется только один цикл по первому массиву, в котором обращаться ко второму по id.
То есть сложность будет длина массива * 2 вместо длина массива * длина массива

Если же данные берутся из базы, то, как правильно заметили в комментариях, там же и надо сразу считать

Потому что у строки не может быть строкового смещения.

В ответе выше написана чушь.
Обращаться к строке по индексу можно. И в РНР7, и в РНР 8. Без всяких предупреждений.
К отдельным байтам (но не символам) в строке можно обращаться, указывая смещение в квадратных скобках:
echo 'hello'[0];
Но в сообщении об ошибке говорится конкретно про строковые ключи. А это уже действительно бессмыслица, никаких строковых смещений в строке быть не может. Что и написано черным по белому в ошибке.

И проблема действительно находится выше, в совершенно дурацком присвоении пустой строки массиву.
Если мы хотим использовать массив, то и присваивать начальное значение надо в виде пустого массива, а не строки.

При этом отдельно проверять наличие $aUnit[$server['unit']] и $aUnit[$server['unit']][$server['game']] не нужно. Достаточно проверить на существование сразу конечный элемент.
В 8.2 это можно красиво написать одной строчкой
$aUnit[$server['unit']][$server['game']] ??= '';
Но если почитать код дальше, то станет видно, что и здесь нам строка тоже не нужна. А снова нужен массив.
Потому что в коде ниже эта строка разбивается через explode(???).
То есть, надо написать

$aUnit[$server['unit']][$server['game']] ??= [];
$aUnit[$server['unit']][$server['game']][] = $server['id'];

А ниже выкинуть всё от explode() до unset()

Третий вариант, задавать переменные через map

map $uri $APP_SETTINGS {
  default "";
  ~^/path1/ app-settings-12;
  ~^/path2/ app-settings-100500;
}

server {
  location ~\.php$ {
    fastcgi_pass ...;
    fastcgi_param APP_SETTINGS $APP_SETTINGS if_not_empty;
    ...
  }
}

Конечно IP-адрес удалённого сервера, с которого создалось соединение. Посетитель удалённого сервера к такому соединению имеет отношение только в том, что соединение создаётся для получения данных для этого посетителя.

Можно используя функцию eval().

console.log(eval(x + y + w));

Но в случае, если получение действия будет происходить из пользовательского ввода, использование eval() может быть опасным.
Тогда стоит ограничить отведенный пользователю набор действий:

if (y === '+') {
    console.log(x + w);
}

По моему опыту, API для пополнения счета вам все равно не поможет.
Опсосы (а конкретно - МТС и Мегафон) теперь для дозволения любой, хоть технической, рассылки в своих сетях требуют предоставить им гарантийное письмо от того, кто эту рассылку делает, и оплатить ежемесячную дань в 2000 - 2500 за эту роскошную привилегию.
Без этих телодвижений вы ничего на них не отправите, а делать это за своих клиентов вы не имеете права.

Вообще, имхо, рассылка СМС просто-напросто устарела так же, как и оплата по СМС, и примерно по тем же причинам.

Вы так все усложнили...
Назначаете вручную одному пользователю нужные вам интервалы, затем берете его как шаблон и распространяете на нужную вам группу, либо берете значение из этого шаблона и вставляете в свой скрипт
Вот простенький скрипт, который делает то, что вам нужно:

$templateuser='a.khramov'
$templatehours= Get-ADUser -Identity $templateuser -properties logonHours
Get-ADGroupmember "msk-VPN_Users" |foreach {Set-ADUSer $_.samaccountname -Replace @{logonHours = $templatehours.logonHours} }

Уфф, 5 раз прочитал, пока понял. Второе.

Метод класса вписывается в класс который над которым осуществляют действие.
Собственно, определение класса - это "данные и методы для работы с ними". С ними. А не с данными чужих объектов.

Сущность Модератор в данном случае вообще не нужна, достаточно свойства в сущности Пользователь (кстати, откуда вы все берете этих "участников"? Участник бывает у события, а не у сущности. Вы же не говорите "участник 5А класса?")

Связей здесь должно быть значительно меньше.
Если модератор не упоминается ни в опросе, ни в добавленной книге, то и связи между ними никакой нет.
В текущей схеме должна быть только одна связь: опроса с прошедшим его пользователем. Одним из свойств опроса будет экземпляр класса Пользователь. Причем это касается класса РезультатОпроса, которого нет на диаграмме. А он должен быть. И вот он будет связан с классом Опрос.

Чего здесь не хватает - это интерфейса. Классов-контроллеров. Которые собственно эти методы и вызывают.
Какой-нибудь BookController.add(), который принимает, скажем, два объекта, Пользователь и Инпут, и проверяет, является ли пользователь модератором. И если да, то создает объект Book, и используя данные из инпута, вызывает Book.add()
И то же самое с опросами

Еще по-хорошему должен быть BookRepository. Потому что если книга умеет сама себя добавлять в хранилище, то это считается плохим тоном. Должен быть отдельный объект Хранилище Книг, в котором и будут методы добавления и поиска.