Eerste getal weghalen (C#)

[Verwijderd]

Beste allemaal,

voor mijn studie ben ik bezig met opdrachten in C#, echter loop ik tegen een probleem aan.

Ik moet vanuit een array alle getallen bij elkaar optellen, maar ik mag het EERSTE even getal niet meetellen.

Het optellen van alle getallen is uiteraard geen probleem, en in principe lukt het vinden van het eerste even getal (volgens mij) ook, alleen wanneer ik deze van het gehele getal af wil trekken krijg ik een error terug.
Misschien dat iemand er even met een frisse blik naar zou willen kijken

Dit is de vraagstelling + mijn stuk code:

code:

        //Sum all the elements in an array up to but not including the first even number. 
        //(Write your unit tests. What about the case when there is no even number?)
        [Test]
        public void TestExercise5()
        {
            Programmeren2Tests.Chapter11Test.TestExercise5(Exercise5);
        }

        public int Exercise5(int[] xs)
        {
            int count = 0;


                foreach (var x in xs)
            {

                for (int i = 0; i < xs.Length; i++)
                {
                    if (xs[i] % 2 == 0)
                    {
                        return i;

                    }


                if (x != 0)
                {
                    count = count + x - i;
                }
            }
            return count;

}
        }

Ik hoop dat jullie mij hierbij kunnen helpen.

Groetjes

[Rannasha]

Je kunt de code het beste binnen [ code ] tags plaatsen, dat bevordert de leesbaarheid. Bijvoorbeeld:

C#:

foo.Bar("Hello World!");

Specifiek over jouw code-voorbeeld, je hebt een dubbele lus. Eerst loop je middels de foreach() lus over alle x in xs. En vervolgens doe je binnen die lus hetzelfde maar dan met de for() lus. Zet in Visual Studio eens een breakpoint (F9) op de eerste regel van het programma en loop vervolgens stap voor stap (F11) door het programma heen. Bij iedere stap kun je kijken wat er wordt uitgevoerd en welke waarde iedere variabele heeft.

Je kunt ook een andere aanpak overwegen:

Hou in een aparte variabele bij of je al een even getal tegen bent gekomen. Zolang deze variabele aangeeft dat je nog geen even getal hebt gehad, controleer je bij ieder getal of het even is. Is het even en ben je nog niet eerder een even getal tegen gekomen, sla het dan over en update de eerdergenoemde variabele.

edit:

Om verder te gaan met je originele aanpak, het loont vaak om de code op te delen in verschillende functies met elk een eigen taak. Zo kun je iedere functie apart ontwikkelen en testen en loop je minder risico dat dingen door elkaar heen gaan lopen.

Bijvoorbeeld:

C#:

int FindFirstEvenNumber(int[] numbers)
{
    // Code om het eerste even getal te vinden
}

int ArraySum(int[] numbers)
{
    // Code om de som van de getallen te berekenen
}

[ Voor 89% gewijzigd door Rannasha op 29-01-2018 16:35 ]

[downtime]

Verwijderd schreef op maandag 29 januari 2018 @ 16:06:
Beste allemaal,

voor mijn studie ben ik bezig met opdrachten in C#, echter loop ik tegen een probleem aan.

Ik moet vanuit een array alle getallen bij elkaar optellen, maar ik mag het EERSTE even getal niet meetellen.

Het optellen van alle getallen is uiteraard geen probleem, en in principe lukt het vinden van het eerste even getal (volgens mij) ook, alleen wanneer ik deze van het gehele getal af wil trekken krijg ik een error terug.
Misschien dat iemand er even met een frisse blik naar zou willen kijken

Dit is de vraagstelling + mijn stuk code:


[...]


Ik hoop dat jullie mij hierbij kunnen helpen.

Groetjes

Jouw opdracht komt niet overeen met het commentaar. Je zegt dat je het eerste even getal niet mag meerekenen. Maar het commentaar spreekt over de getallen tot het eerste even getal, en alles daarna mag niet meegeteld worden. Da’s niet hetzelfde.

[Stoelpoot]

Dit lijkt een schoolopdracht, en in principe geven we op t.net niet 1 op 1 het antwoord voor een schoolopdracht. Daarom houd ik het voor nu bij wat guidance over een mogelijke aanpak.

2 problemen. Het eerste: Ik mis je error. Met 'krijg ik een error terug' kunnen we vrij weinig helpen.

Het tweede (misschien ook een hint waar je naar moet kijken):

en in principe lukt het vinden van het eerste even getal (volgens mij) ook

Vanwege je 'geloof ik' krijg ik direct al het idee dat hier het probleem zit. Het beste plan van aanpak is wat mij betreft dus eerst uitwerken hoe je het eerste even getal vind, en dan het optellen en aftrekken uitwerken.

[RobIII]

Rannasha schreef op maandag 29 januari 2018 @ 16:10:
Bijvoorbeeld:

Dat is geen voorbeeld meer, dat is 't (zo goed als) compleet voorgekauwd-en-wel de oplossing aanreiken

Give a man a fish and feed him for a day. Teach a man how to fish and feed him for a lifetime.

[Rannasha]

RobIII schreef op maandag 29 januari 2018 @ 16:33:
[...]

Dat is geen voorbeeld meer, dat is 't (zo goed als) compleet voorgekauwd-en-wel de oplossing aanreiken

[...]

True. Heb even de bodies van de functies vervangen door een enkele regel commentaar over de gewenste invulling

[Verwijderd]

Excuses voor het niet gebruiken van code tags in eerste instantie. Ik was ook niet op zoek naar het exacte antwoord alleen een duw in de goede richting, uiteindelijk wist ik het aan de hand van de gegeven informatie hier op te lossen:

code:

        public int Exercise5(int[] xs)
        {
            int count = 0;
            
                foreach (int x in xs)
                {
                    if (x % 2 == 0)
                    {
                        return count;
                    }
               
                else
                {
                    count = count + x;
                }

                }
            return count;

        }

Mijn dank is groot

[erik23]

Verwijderd schreef op maandag 29 januari 2018 @ 16:55:
Excuses voor het niet gebruiken van code tags in eerste instantie. Ik was ook niet op zoek naar het exacte antwoord alleen een duw in de goede richting, uiteindelijk wist ik het aan de hand van de gegeven informatie hier op te lossen:

code:

        public int Exercise5(int[] xs)
        {
            int count = 0;
            
                foreach (int x in xs)
                {
                    if (x % 2 == 0)
                    {
                        return count;
                    }
               
                else
                {
                    count = count + x;
                }

                }
            return count;

        }

Mijn dank is groot

Weet je zeker dat dit klopt? Dit is namelijk niet het doel dat je aangaf:

Verwijderd schreef op maandag 29 januari 2018 @ 16:06:
Ik moet vanuit een array alle getallen bij elkaar optellen, maar ik mag het EERSTE even getal niet meetellen.

Nu stop je met tellen zodra je een even getal tegenkomt.

[Caelorum]

@erik23 het klopt nu iig wel met het commentaar dat bij en zijn testfunctie staat (zoals al aangegeven was)

[ThoNohT]

De code die je hier als laatste postte lijkt inderdaad te werken. Niks mis mee dus (alhoewel de indentation wat verwarrend is).

Deze manier van programmeren is ook zeker goed om te doen, en om te leren algoritmisch te denken, maar toch wil ik je even wijzen op een van de features van C#, namelijk Linq. Hiermee is de hele opdracht in een enkele regel, goed leesbaar op te schrijven. Het loont zeker om hier ook wat ervaring mee te krijgen.

Aangezien je toch al een correct antwoord hebt zal ik mijn versie maar gewoon geven, het is dan aan jou om uit te zoeken waarom en hoe dit werkt, mocht je hier meer over willen leren.

C#:

return xs.TakeWhile(x => x % 2 != 0).Sum();

[ Voor 1% gewijzigd door ThoNohT op 01-02-2018 12:09 . Reden: Nog korter na review van RobIII ]

[RobIII]

ThoNohT schreef op donderdag 1 februari 2018 @ 10:05:

C#:

return xs.AsEnumerable().TakeWhile(x => x % 2 != 0).Sum();

Vanwaar de AsEnumerable? xs is al een int array en is dus al enumerable.

[ Voor 10% gewijzigd door RobIII op 01-02-2018 10:43 ]

[ThoNohT]

Dat is waar inderdaad. Het is door die aanroep wellicht wel iets duidelijker dat de Linq operaties primair op IEnumerable zijn gedefinieerd, en niet iets array specifieks zijn, maar het is geheel niet nodig.

[RobLemmens]

ThoNohT schreef op donderdag 1 februari 2018 @ 12:14:
Dat is waar inderdaad. Het is door die aanroep wellicht wel iets duidelijker dat de Linq operaties primair op IEnumerable zijn gedefinieerd, en niet iets array specifieks zijn, maar het is geheel niet nodig.

Je voorbeeld klopt toch niet? Takewhile pakt alle elementen tot de conditie true is. Dus tot het eerste even cijfer. Dat is niet wat er gevraagd werd: som minus eerste even getal.

[ThoNohT]

RobLemmens schreef op donderdag 1 februari 2018 @ 13:48:
[...]
Je voorbeeld klopt toch niet? Takewhile pakt alle elementen tot de conditie true is. Dus tot het eerste even cijfer. Dat is niet wat er gevraagd werd: som minus eerste even getal.

Dat is wat de titel en zijn inleiding vraagt. Maar als je naar het commentaar boven de eerste test kijkt, en zijn uiteindelijke implementatie, dan is mijn voorbeeld exact waar hij naar op zoek was.

C#:

        //Sum all the elements in an array up to but not including the first even number. 
        //(Write your unit tests. What about the case when there is no even number?)

[RobLemmens]

Oei excuus