Victoria Zotova

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

Вот эти две сточки

uint _50 = address(this).balance/100*50; //50% of balance of contract
uint _25 = address(this).balance/100*25; //25% of balance of contract

будут выполняться во время работы конструктора. Т.е. если заранее на адрес контракта (до его создания) не отослать ничего - то address(this).balance будет равен 0. Эти переменные надо вычислять каждый раз, когда нужно переводить eth.
Также рекомендую сначала делать все умножение, а потом все деление: меньше проблем с округлением.
Чтобы контракт срабатывал автоматически, метод перевода средств должен быть в receive() external payable{}. И лучше не использовать balance, т.к. в него будут попадать прошлые остатки. В этом плане подойдет msg.value (если говорить об автоматическом переводе).
address payable addr1 и вторую переменную стоит сделать константой, если не планируется обновлять эти адреса в будущем. А если говорим о промышленном коде с тестированием, то неплохой вариант сделать их immutable и задавать в конструкторе.
Помимо этого необходим метод для вывода оставшихся 25% иначе застрянут на адресе контракта на совсем.
И наверное добавлю еще одну мелочь: transfer не самый идеальный способ переводить средства (если говорим о реальном контракте), из-за ограничения по газу на такой методу, рекомендую глянуть Address.sol (кажется метод sendValue) из библиотеки openzeppelin.

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

Начиная с какой-то версии (не помню точно какой), transfer работает только с address payable.
Попробуйте такой вариант:

function pay(address payable _user, uint _value) payable public {
        _user.transfer(_value);
 }

Еще хочу добавить, что transfer не самый предпочтительный способ перевода средств из-за ограничения в газе (23 000). Хотя, если уверены, что принимающая сторона не имеет fallback функции или она очень простая, то пойдет.