Topics

C#•6mo ago

Joreyk ( IXLLEGACYIXL )

How to work better with records

https://gist.github.com/IXLLEGACYIXL/e3c5575656de0bebf314096699e9d42c I have this and want the modified item after breakthrough is applied, but i dont want to modify the original item but it seems like this shouldnt be the way to work with records... can i simplify that syntax?

Gist

gist:e3c5575656de0bebf314096699e9d42c

GitHub Gist: instantly share code, notes, and snippets.

42 Replies

Pobiega•6mo ago

your Apply method should return the new record instance so it can't be a void

Joreyk ( IXLLEGACYIXL )•6mo ago

yes it was more about the changing of the record itself

Pobiega•6mo ago

you don't change records. you can't without doing some quite evil stuff also, why are you using full syntax instead of record syntax?

Joreyk ( IXLLEGACYIXL )•6mo ago

like line 10-13 should this "be the way" its intended to be? thats the question that i wanted to ask how to write it in better syntax whats the record syntax? or how could i write the lines i have "better" in a better syntax

Pobiega•6mo ago

internal record ComboDice(int Count, int Sides, int Fix)
{
    public virtual int Roll()
    {
        int total = 0;
        for (var i = 0; i < Count; i++)
        {
            total += IDice.Random.Next(1, Sides + 1);
        }
        return total + Fix;
    }
}

internal record ComboDice(int Count, int Sides, int Fix)
{
    public virtual int Roll()
    {
        int total = 0;
        for (var i = 0; i < Count; i++)
        {
            total += IDice.Random.Next(1, Sides + 1);
        }
        return total + Fix;
    }
}

thats how you declare a record. if you're going to manually create the props, then why bother with records in the first place?

Joreyk ( IXLLEGACYIXL )•6mo ago

the reason i dont have it this way is that the engine cant handle it

Pobiega•6mo ago

I don't really understand what you are trying to do, because what you have right now does not work

internal record class Breakthrough : ComboDice
{
    public void Apply(Character target, Character source, Item targetItem, Item sourceItem)
    {
        var x = targetItem with {
            Base = targetItem.Base with {
                Fix = targetItem.Base.Fix - Roll()
            }
        };
    }
}

internal record class Breakthrough : ComboDice
{
    public void Apply(Character target, Character source, Item targetItem, Item sourceItem)
    {
        var x = targetItem with {
            Base = targetItem.Base with {
                Fix = targetItem.Base.Fix - Roll()
            }
        };
    }
}

this does absolutely nothing

Joreyk ( IXLLEGACYIXL )•6mo ago

imagine it returns targetitem XD

Pobiega•6mo ago

but why is this its own class?

Joreyk ( IXLLEGACYIXL )•6mo ago

how do you mean?

Pobiega•6mo ago

its just a method. so add it as an extension method, or a normal method Breakthrough doesn't add or change anything in ComboDice

Joreyk ( IXLLEGACYIXL )•6mo ago

because i want a List of "buffs" with for example breakthrough to reduce armor and imagine that i have return targetItem.... so i have the original one and a "modified" version thats an example armor how it should look like in yaml

Armor :
    Base : { Rolls : 4 , Side : 8 FIx: 12 }
    Buffs:
        - BreakThrough : { Rolls : 0 , Sides : 0 Fix: 4 }
        - ArmorIncrease : { Rolls : 1 , Sides : 6 Fix: 0 }

Armor :
    Base : { Rolls : 4 , Side : 8 FIx: 12 }
    Buffs:
        - BreakThrough : { Rolls : 0 , Sides : 0 Fix: 4 }
        - ArmorIncrease : { Rolls : 1 , Sides : 6 Fix: 0 }

breakthrough reduces armor and armorincrease increases it but i dont want to modify "the original armors base" as i apply the buffs in each combat thats why records as they are good for "keeping the original" and getting a copy that i can modify with the buffs

Pobiega•6mo ago

okay, sure you're gonna have some issues with how the code is written thou. public ComboDice Base { get; init; } = new ComboDice(); for example that new ComboDice does nothing, if Item is intended to be deserialized

Joreyk ( IXLLEGACYIXL )•6mo ago

oh

Pobiega•6mo ago

in general, records without constructors are not going to work very well here

Joreyk ( IXLLEGACYIXL )•6mo ago

what would you advise for "modification" and keeping the original

Pobiega•6mo ago

im not sure about yaml deserializers specifically, but the json ones create the object (either via a parameterless ctor or the "best" ctor match) then use reflection to access the property setters

Joreyk ( IXLLEGACYIXL )•6mo ago

we have yamlsharp from 2008 and my yaml serilaizer that i wrote cant handle it either

Pobiega•6mo ago

how would the API look for this? ie, how do you "activate" the buffs?

Joreyk ( IXLLEGACYIXL )•6mo ago

how do you mean? if i get the collision event from the weapon, i take the source ( attacking wepaon and character ) and calculate it against the targets armor and character now all buffs get applied and at the end its just damage - armor armor would be modified if the weapon has breakthrough so in the last step it would resist less thats the plan to do

Pobiega•6mo ago

but.. your armor value isnt fixed. its not part of the record its made by calling the Roll method

Joreyk ( IXLLEGACYIXL )•6mo ago

if my current armor has this as base { Rolls : 4 , Side : 8 FIx: 12 } then the new record will have { Rolls : 4 , Side : 8 FIx: 8 } with a breakthrough of a roll of 4 if i would calculate now the armors value it would be 4 dices a 8 sides + 8 so its reduced

Pobiega•6mo ago

Ah I see, so they modify the base roll they dont make their own rolls

Joreyk ( IXLLEGACYIXL )•6mo ago

yes

Pobiega•6mo ago

I see.

Joreyk ( IXLLEGACYIXL )•6mo ago

a 1 roll of 6 sides for armor increase with a result of 3 would increase the armors fix value by 3 after all modifications, the armor rolls with the end result

Pobiega•6mo ago

well then the result of applying all the buffs should be an IDice I guess?

Joreyk ( IXLLEGACYIXL )•6mo ago

oh ye it should be

Pobiega•6mo ago

but even then this seems a bit weird each buff shouldnt care about the target etc, no? it should only care about the dice its modifying, and itself?

Joreyk ( IXLLEGACYIXL )•6mo ago

the idea was if you have high fire resistance then ice damages you more

Pobiega•6mo ago

is there an individual stat for ice resistance? or is fire resistance actually... "heat modifier"? ie, low values block ice, but make you vulnerable to fire and vice versa

Joreyk ( IXLLEGACYIXL )•6mo ago

there is not yet an extra stat for ice didnt think that far so an armor would have Base Ice Fire ... and the buffs as values hmmm

Pobiega•6mo ago

public record Item(Dice Base, List<IBuff> Buffs);

internal record class Breakthrough(int Count, int Sides, int Fix) : Dice(Count, Sides, Fix), IBuff
{
    public Dice Apply(Dice baseDice)
        => baseDice with { Fix = baseDice.Fix - this.Fix };
}

public interface IBuff
{
    public Dice Apply(Dice baseDice);
}

public record Item(Dice Base, List<IBuff> Buffs);

internal record class Breakthrough(int Count, int Sides, int Fix) : Dice(Count, Sides, Fix), IBuff
{
    public Dice Apply(Dice baseDice)
        => baseDice with { Fix = baseDice.Fix - this.Fix };
}

public interface IBuff
{
    public Dice Apply(Dice baseDice);
}

im playing around with something like this atm

Joreyk ( IXLLEGACYIXL )•6mo ago

okeeee

Pobiega•6mo ago

public int Attack(Character target)
{
    // this applies all the buffs in order, giving us the final modified value
    var dice = Weapon.Buffs.Aggregate(Weapon.Base, (current, buff) => buff.Apply(current));

    var roll = dice.Roll();

    // TODO: modify the damage by armor

    return roll;
}

public int Attack(Character target)
{
    // this applies all the buffs in order, giving us the final modified value
    var dice = Weapon.Buffs.Aggregate(Weapon.Base, (current, buff) => buff.Apply(current));

    var roll = dice.Roll();

    // TODO: modify the damage by armor

    return roll;
}

lets you apply buffs like this

Joreyk ( IXLLEGACYIXL )•6mo ago

okeee i gotta play with that

Pobiega•6mo ago

you could easily send in the source and target char/items here too

Joreyk ( IXLLEGACYIXL )•6mo ago

falsely understood XD i have to play with it i didnt use aggregate too much

Pobiega•6mo ago

the idea is that it takes the base then sends that to the first buff, gets a result sends that result to the next buff repeat until done

Joreyk ( IXLLEGACYIXL )•6mo ago

that sounds good yep thats what im going with

Pobiega•6mo ago

ok let me implement another buff and see how it goes also, currently breakthrough ONLY affects Fix so if you give it a value for side or count, it wont do anything

Joreyk ( IXLLEGACYIXL )•6mo ago

yes

We are a programming server aimed at coders discussing everything related to C# (CSharp) and .NET.

52KMembers

View on Discord

Want results from more Discord servers?

Add your server

More Posts

Unable to configure values in DI using the extension methodI'm having difficulty binding config values through... Details in the following SO question: https:Need help with pokemon game for schooli only have 2 days left been working on this stuff but its so much and very hard can someone help me Windows hosting .net 7.0 web api project doesn't connection mssql databaseHello, i am using .net 7.0 for web api. I built and published project on windows hosting. I tried co what is this?API Authentication via Generated URLs onlyFirst of all, I know this is terrible practice. I am prefacing this by saying its to work around the ✅ calling functionsim trying to test my functions but it keeps giving errors anyone that could help ✅ Microsoft.Interop.Excel version?I have this error and after some research the version of Microsoft.Interop.Excel NuGet Package shoul Is it possible to host my Telegram bot app using Arduino, if yes, how?title is there a point of a one to one relationship.Is there ever a use case to create a seperate table for a one to one relationship. Because what is t aaaa java0000, type is defined multiple timesandroidx.collection.ArrayKtClass is defined multiple times it says, this only happens to this specif