Ваш rollSim()
функция должна возвратить прокрученные числа вместо того, чтобы установить некоторые переменные и пытаться использовать их в Вашей другой функции. Я возвратил бы их в ассоциативном массиве, как это:
function rollSim() {
$roll['hAttack'] = rand(1,20);
$roll['mDefend'] = rand(1,20);
$roll['mAttack'] = rand(1,20);
$roll['hDefend'] = rand(1,20);
$roll['mDamage'] = rand(10,25);
$roll['hDamage'] = rand(1,20);
return $roll;
}
function combatSim() {
$roll = rollSim();
if ($roll['hAttack'] > $roll['mDefend']) {
print "Hero hit monster for {$roll['hDamage']} damage.<br>";
} else if ($roll['hAttack'] <= $roll['mDefend']) {
print "Hero missed monster.";
}
}
Нет, Вы не можете действительно сделать то, что Вы спрашиваете. Даже если Вы встроили объявление rollSim()
в определении combatSim()
(который можно сделать, это законно, но не имеет никаких реальных эффектов), переменные, Вы начинаетесь rollSim()
все еще было бы локально для него и недоступен combatSim()
.
Вам нужен лучший способ раздать информацию, в которой Вы обеспокоены. Jeremy Ruten детализирует хороший путь. Иначе должен был бы определить объект, это ответственно за моделирование Вашего боевого события, и иметь rollSim()
и combatSim()
оба быть методами на нем.
class myCombat {
private $hAttack;
private $mDefend;
private $mAttack;
private $hDefend;
private $mDamage;
private $hDamage;
function rollSim() {
$this->hAttack = rand(1, 20);
$this->mDefend = rand(1, 20);
$this->mAttack = rand(1, 20);
$this->hDefend = rand(1, 20);
$this->mDamage = rand(10, 25);
$this->hDamage = rand(1, 20);
}
function combatSim() {
$this->rollSim();
if($this->hAttack > $this->mDefend)
echo 'Hero hit monster for ' . $this->hDamage . ' damage.<br />';
else
echo 'Hero missed monster';
}
}
$combat = new myCombat;
$combat->combatSim();
Можно ли вызвать функции друг от друга, конечно..., но я думаю, что Вы хотите использовать объем одного в другом, корректном?
Совместное использование объема является грязным бизнесом. Вы - более обеспеченные передающие аргументы.