• supprimer N unités
  
• augmenter la nourriture
  

describe TestEggSacrifice do

  before do

    @corwin = Player.create(name: "Corwin")

    @montpellier = @corwin.cities.create(name: "Montpellier")

  end





  it "should explode if player doesn't exist" do

    assert_raises Player:layerNotFound do 

      EggSacrifice.new(nil, nil, 42)

    end

  end





  it "should explode if city doesn't belong to player" do

    mandor = Player.create(name: "Mandor")

    lyon = mandor.cities.create(name: "Lyon")



    assert_raises City::CityNotFound do 

      EggSacrifice.new(@corwin.id, lyon.id, 42)

    end

  end





  it "should explode when city doesn't have enough eggs" do

    assert_raises EggSacrifice::NotEnoughEggs do 

      EggSacrifice.new(@corwin.id, @montpellier.id, 42)

    end

  end





  it "should kill 5 eggs and add X resources to the city" do

    City.spawn_eggs(@montpellier.id, 10)



    result = EggSacrifice.new(@corwin.id, @montpellier.id, 5).execute()

    assert results.city_id == @montpellier.id

    assert results.remaining_eggs == 5

    assert results.added_resources == ... # Whatever the rules say.

    assert results.new_resources == ... # Whatever the rules say.



    assert_raises EggSacrifice::NotEnoughEggs do 

      EggSacrifice.new(@corwin.id, @montpellier.id, 6).execute()

    end



    assert City.resources(@montpellier.id) == ... # Whatever the rules say.

  end

end

class EggSacrifice

  def initialize(city_id, count)

    @city_id = city_id

    @count = count



    raise City::CityNotFound unless current_player.cities.exists?(city_id)

    raise City::NotEnoughEggs unless City.has_enough_eggs?(city_id, count)

  end





  def execute()

    City.transaction do

      result = Egg.remove_n_from_city(@city_id, @count)

      new_resources = City.add_resources(@city_id, result.resources)

      

      {

        city_id: @city_id,

        remaining_eggs: result.remaining_eggs,

        added_resources: result.resources,

        new_resources: new_resources

      }

    end

  end

end

CREATE PROCEDURE game_action_city_sacrifice(

IN idPlayer INT UNSIGNED,

IN idCity INT UNSIGNED,

IN idSacrifiedUnit SMALLINT UNSIGNED

) BEGIN



INSERT INTO city_resources (id_resource, id_city, quantity)

(SELECT

    uic.id,

    ur.id_resource,

    0.5 * ur.construct_quantity * uic.units_count

FROM units_resources AS ur

INNER JOIN units_in_city AS uic ON uic.id_unit = ur.id AND uic.id_player = idPlayer AND uic.id = idCity

WHERE id_unit = idSacrifiedUnit)

ON DUPLICATE KEY UPDATE

city_resources.id_city = city_resources.id_city,

city_resources.id_resource = city_resources.id_resource,

city_resources.quantity = city_resources.quantity + VALUES(city_resources.quantity)

;



DELETE FROM units_in_city

WHERE id_unit = idSacrifiedUnit

AND id_city = idCity;



END

$$

(22-03-2020, 12:26 PM)Sephi-Chan a écrit : Si ça peut t'aider à le concevoir, tu as le droit d'avoir des modèles qui ne correspondent pas à ta base de données. Généralement on appelle ça des service objects.

(22-03-2020, 12:26 PM)Sephi-Chan a écrit : Tu noteras que je fais appel à de nombreuses méthodes statiques. Je trouve que dans les jeux, passer des ID a souvent de bien meilleurs résultats que des objets complets (souvent issus de l'ORM) qui nécessitent plein d'appels inutiles à la base. Ça colle parfaitement avec ta volonté d'utiliser ta base de données directement plutôt qu'à travers l'ORM.

(22-03-2020, 01:01 PM)Xenos a écrit : Pour MVC, je ne sais pas si tu classes ça dans "controlleur" ou dans "modèle" (je serait tenté de dire "controlleur" car cela contient de la logique) mais je rejoins l'approche de faire une autre "classe" (ou autre concept du langage) controlleur appelant les deux "classes" de modèles pour que ce controlleur dise aux deux modèles de faire la suppression d'unité et l'ajout de ressources

(22-03-2020, 01:01 PM)Xenos a écrit : Et du côté de mes archis, la page "/game/action/city/sacrifice" serait un endpoint qui appelle sa procédure stockée dédiée, dans laquelle on aurait bêtement:

CREATE PROCEDURE game_action_city_sacrifice(

IN idPlayer INT UNSIGNED,

IN idCity INT UNSIGNED,

IN idSacrifiedUnit SMALLINT UNSIGNED

) BEGIN



-- Code raccourci



END

$$

Discord a écrit :Maz:
  oui, on est d'accord, quand je dis "supprimer le modèle"
  c'est genre

Code PHP :

<?php 

eatEggs($nb_fourmis) {

Ant::delete($b_fourmis);

City::increaseFoodFor($nb_fourmis);

// Ici j'envoie la vue

} 

  je vais pas aller taper un delete direct en sql dans le controlleur on est d'accord sur ça ^^
Sephi-Chan:
  non
  c'est pas bon ça
  tu dois avoir une méthode eatEggs
  c'est elle qui sait que ça doit kill des oeufs et ajouter de la bouffe
Maz:
  donc toi: depuis le modèle Ant, tu modifierais la table cities pour augmenter la nourriture? ou éventuellement depuis le modèle Ant tu appele le modèle City pour augmenter sa nourriture
Sephi-Chan:
  et donc tu testes cette méthode pour t'assurer qu'il y a bien assez d'œufs (donc des fourmis dans cette phase en assez grand nombre)
Maz:
  le truc c'est que je dois modifier 2 modèles, c'est pour ça que ça me fait chier de faire que l'un appel l'autre.
Sephi-ChanHier à 23:1
  on s'en fout ça existe pas cette restriction
Maz
ou alors je cré une procédure SQL genre eatEggsFromCity(nb_ants) et qui modifieras la table... (joke: structure de ouff ;p)

Citation :un contrôleur différent (ni Unity, ni City) qui appelerais directement les deux modèles depuis le contrôleur pour leur demander d'exécuter le traitement des données.

Citation : je trouve que l'on y perds en visibilité dans le code du moteur du jeu.

Citation :il doit donc aller chercher dans les migrations