beginners vraag c#

[Verwijderd]

Hallo allemaal,

ik heb even een stomme beginnersvraag met c#. Ik ben er net mee begonnen dus let niet op de eenvoud van de code

Ik heb de volgende code:

C#:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
    class Program
    {

        public class collega
        {
            public string sNaam;
            public int nLeeftijd;
        }

        static void Main(string[] args)
        {

            Console.WriteLine("Hoeveel collega's heb je :");
            string sAantal = Console.ReadLine();
            int nAantal = Convert.ToInt32(sAantal);

            collega[] collegas = new collega[nAantal];

            for (int i = 0; i < nAantal; i++)
            {
                Console.WriteLine("Wat is de naam van collega " + (i + 0) + " :");
                string sNaam = Console.ReadLine();

                Console.WriteLine("Wat is de leeftijd van " + sNaam + " :");
                string sLeeftijd = Console.ReadLine();
                int nLeeftijd = Convert.ToInt32(sLeeftijd);

                collega DezeCollega = new collega();

                DezeCollega.sNaam = sNaam;
                DezeCollega.nLeeftijd = nLeeftijd;

                collega[i] = DezeCollega;
            }
        }

    }

}

Ik krijg over de regel: collega[i] = DezeCollega; de volgende foutmelding:
Error 1 'ConsoleApplication1.Program.collega' is a 'type' but is used like a 'variable'

Hoe krijg ik deze melding weg?

Bij voorbaat dank

[ Voor 0% gewijzigd door RobIII op 03-10-2011 22:46 . Reden: Code tags toegevoegd. ]

[YopY]

Door je collega-variabele (die dezelfde naam heeft als je class collega) te hernoemen. Ook is het vaak normaal om classnamen met een hoofdletter te laten beginnen, en variabelen (zoals DezeCollega) met een kleine letter.

Ten tweede, je kunt je code netjes (laten) formatteren door de [code=c#]-bbcode te gebruiken.

[ValHallASW]

Beginnen met goed lezen: 'ConsoleApplication1.Program.collega' is a 'type' but is used like a 'variable'. Op je regel staat

code:

collega[i] = DezeCollega

maar collega is een type (nl. de naam van de klasse die je net hebt gemaakt). Je array heeft een andere naam!

YopY schreef op maandag 03 oktober 2011 @ 21:33:
Door je collega-variabele (die dezelfde naam heeft als je class collega) te hernoemen.

Ook jij moet beter lezen ;-) Het is een array van type collega[], maar hij heet geen collega ;-)

[KoW]

Hehe, inderdaad.

Je mist een s

[RobIII]

Volgende keer als je code post, gebruik dan code tags wil je? Verder zou je dit prima moeten kunnen oplossen door gewoon überhaupt eens te kijken over welk regelnummer VS klaagt en dan eens wat verder dan je neus te kijken

[Verwijderd]

Bedankt voor de reacties, was achteraf inderdaad erg simpel gewoon die s erachter...
Begin net en dan is het best lastig om erachter te komen (ook met google erbij )

In ieder geval bedankt en volgende keer zal ik de code tags goed gebruiken

[Wijnbo]

Tips voor TS:

Let op je naamgeving.
Voorkomt veel van zulke problemen :-)

Dus :

C#:

Class Pietje
{
    private string _pietje 
    public string EenMooieString {get;set;}
}

Verder zou ik ook niet met prefixes om aan te geven wat voor type iets is, aangezien een fatsoenlijke IDE dit zelf wel aangeeft.

En nog een tip: Wen je vanaf het begin aan om alles in het engels te doen, ook voor eigen testprojectjes die toch nooit iemand te zien krijgt.

Succes met programmeren!


Toch even niets te doen :

C#:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
    class Program
    {

        public class Colleague
        {
            public string Name { get; set; }
            public int Age { get; set; }
        }

        static void Main(string[] args)
        {
            Console.WriteLine("Hoeveel collega's heb je :");
            int amount = Convert.ToInt32(Console.ReadLine());
            Colleague[] colleagues = new Colleague[amount];

            for (int i = 0; i < amount; i++)
            {
                Console.WriteLine("Wat is de naam van collega " + (i + 0) + " :");
                string name = Console.ReadLine();
                Console.WriteLine("Wat is de leeftijd van " + name + " :");
                int age= Convert.ToInt32(Console.ReadLine());
                Colleague newColleague = new Colleague();
                newColleague.Name = name;
                newColleague.Age = age;
                colleagues[i] = newColleague;
            }
        }
    }
}

Zelfde code, alleen dan met de naamgeving/hoofdlettergebruik wat redelijk algemeen is. Hier en daar zie je soms wat verschillen, maar als je zoiets als dit doet zit je redelijk goed.

Zie je code er gelijk een stuk professioneler uit mochten anderen het ooit onder ogen krijgen. En als je werkt in .NET 3.5 of hoger kun je "var " gebruiken en vereenvoudigd properties zetten bij het constructen van een object:

C#:

var newColleague = new Colleague {Name = name, Age = age};

[ Voor 86% gewijzigd door Wijnbo op 04-10-2011 08:39 ]

[dj_vibri]

Wijnbo schreef op dinsdag 04 oktober 2011 @ 08:28:
Tips voor TS:

Let op je naamgeving.
Voorkomt veel van zulke problemen :-)


Verder zou ik ook niet met prefixes om aan te geven wat voor type iets is, aangezien een fatsoenlijke IDE dit zelf wel aangeeft.

Wat betreft de prefixes, buiten het feit dat er steeds weer discussie's over ontstaan , kan het altijd zijn dat de TS dit momenteel in school/les aan het leren is. Zelfs in "mijn tijd" werden we verplicht van nog te werken met " txtVoornaam", "cmdSave", ... etc....

Toegegeven: ééns je van school bent en je gaat werken leren ze je dat wel snel af

[Caelorum]

Hungarian notatiln op de goede manier gedaan is helemaal niet slecht en kan zeker wat aan de leesbaarheid van de code toevoegen. Waar je alleen voor moet opletten is dat je niet namen gaat maken als 'sNaam' of 'int32Leeftijd', maar namen gebruikt die beschrijven wat iets is niet hoe iets is.

[Cyphax]

dj_vibri schreef op dinsdag 04 oktober 2011 @ 08:48:
[...]


Wat betreft de prefixes, buiten het feit dat er steeds weer discussie's over ontstaan , kan het altijd zijn dat de TS dit momenteel in school/les aan het leren is. Zelfs in "mijn tijd" werden we verplicht van nog te werken met " txtVoornaam", "cmdSave", ... etc....

Toegegeven: ééns je van school bent en je gaat werken leren ze je dat wel snel af

Oh, in dat geval: Wikipedia: Hungarian notation: Systems vs. Apps Hungarian
Kan je het maar het beste gelijk even weten, 't stelt geen zak voor namelijk.

[RobIII]

^ Dat, maar lees ook meteen Making Wrong Code Look Wrong even

[ Voor 12% gewijzigd door RobIII op 04-10-2011 09:22 ]

[Cyphax]

Dat is helemaal een goed idee. Good ol' Spolsky.

[RobIII]

Cyphax schreef op dinsdag 04 oktober 2011 @ 09:41:
Dat is helemaal een goed idee. Good ol' Spolsky.

offtopic:
Nou ja, Spolsky is niet heilig, maar hij zegt schrijft wel eens slimme dingen Wanneer in twijfel: eerst naar RobIII luisteren, dan naar Spolsky

[Wijnbo]

dj_vibri schreef op dinsdag 04 oktober 2011 @ 08:48:
[...]


Wat betreft de prefixes, buiten het feit dat er steeds weer discussie's over ontstaan , kan het altijd zijn dat de TS dit momenteel in school/les aan het leren is. Zelfs in "mijn tijd" werden we verplicht van nog te werken met " txtVoornaam", "cmdSave", ... etc....

Toegegeven: ééns je van school bent en je gaat werken leren ze je dat wel snel af

Daar ben ik het dan weer niet mee eens, ui elementen moet je vind ik juist weer WEL te prefixen... Zeker als je vaak een label hebt en dan een textbox er bij ofzo.

[Guillome]

Dit ging toch over variabelen en properties, en niet over elementen/IDs van ui-elementen

[Caelorum]

Ui elementen zijn toch ook gewoon objecten waar variabelen naar kunnen verwijzen?

[Guillome]

Jep en zo`n object heeft niet als property sName, dat is gewoon object.name
Dat je de waarde daarvan nou lblUserName maakt heeft volgens mij niets hiermee te maken

[Caelorum]

Guillome schreef op dinsdag 04 oktober 2011 @ 16:14:
Jep en zo`n object heeft niet als property sName, dat is gewoon object.name
Dat je de waarde daarvan nou lblUserName maakt heeft volgens mij niets hiermee te maken

Het ging dan ook om het volgende:

C#:

var admbtnCancel = new Button(); //Admin Button

Om even een extreem slecht voorbeeld te geven.

Dit ging toch over variabelen en properties, en niet over elementen/IDs van ui-elementen

Een variabele kan net zo goed verwijzen naar een ui-element en de naam van die variabele kan dus zoals in het brakke voorbeeld hierboven genaamd zijn.

Of begrijp ik Wijnbo nou verkeerd?

[ Voor 3% gewijzigd door Caelorum op 04-10-2011 16:38 ]

[Wijnbo]

Caelorum schreef op dinsdag 04 oktober 2011 @ 16:38:
[...]

Het ging dan ook om het volgende:

C#:

var admbtnCancel = new Button(); //Admin Button

Om even een extreem slecht voorbeeld te geven.

[...]

Een variabele kan net zo goed verwijzen naar een ui-element en de naam van die variabele kan dus zoals in het brakke voorbeeld hierboven genaamd zijn.

Of begrijp ik Wijnbo nou verkeerd?

Eh, geen idee

Maar wat ik over het algemeen zie (en zelf ook gebruik):

C#:

int day;
double amount ;
string userName;

//Maar:

RequiredFieldValidator rfvNameRequired;
Button btnSave;
TextBox tbUserName

//Maar ook:
Banana banana;
SomeInstanceOfHelperClass helper
List<Banana>  bananas;

[Woy]

voor UI elementen gebruik ik inderdaad ook altijd prefixes met het type, maar dat heeft een heel andere reden dan "normale" hungarian notation. Ten eerste voorkom je er naming collisions mee ( lblName, txtName ), ten tweede is het IMHO makkelijker in het gebruik, omdat je meestal toch op zoek bent naar een bepaald type element. Als je bijvoorbeeld een textbox wil hebben begin je met "txt" typen en heb je meteen een overzicht van alle textboxen.

Ik moet wel zeggen dat ik het sinds WPF eigenlijk niet meer doe, maar dat komt omdat je daar toch op een heel andere manier met een UI omgaat, en meestal niet zo direct in code met je UI elementen aan het werk bent. ( En je vaak niet eens een naam aan een UI element hoeft te geven )

[ Voor 4% gewijzigd door Woy op 05-10-2011 09:04 ]