Calling Method

ah jag önskar jag kunde såna här grejer..det är var komplett meningslöst ville bara visa min avensjuka.

public class BlackCards{
	int pointz;

	
	
	public void methodB(String CardReturn){
		
			if (pointz == 1) {	
				CardReturn = "Ess.JPG";
				return CardReturn;
			}
			else if(pointz ==2){
				CardReturn ="Two.JPG";
				return CardReturn;
			}
			
	}
	
}

Ska det inte va typ
public string methodB(String CardReturn)?
Void betyder väl att du inte vill ha nåt return value?
(Va längesedan jag höll på med JAVA så lite osäker )

Pröva göra pointz till public int pointz

Vill du skicka en int (pointz) till metoden methodB, är det de du menar? Och att methodB sedan retunerar en sträng?

i sånna fall borde väl något sånt här fungera:

public string methodB(int x)
{
if (x== 1) 
{	
	CardReturn = "Ess.JPG";
	return CardReturn;
}
else if(x ==2)
{
	CardReturn ="Two.JPG";
	return CardReturn;
}
else
return null;

Sen om du ska använda methodB så skickar du bara med pointz, methodB(pointz)
}

du har nog inte förstått hur det funkar med metoder än, här kommer ett exempel:
Main:

{
   int points = 2;
   BlackCrap blackcrap= new BlackCrap();
   String returnValue = blackcrap.methodName(points);
}

BlackCrap

{
   public String methodName(int parameter){
      if(parameter == 2)return "two.crap";
      return "crap.crap";
   }
}

EDIT: Lite efter

public string methodB(int CardReturn)

Public betyder att den är tillgänglig "överallt".
String är det du vill ha TILLBAKA från metoden (typen du skickar tillbaka med return).
(int CardReturn) är det du skickar IN till metoden, alltså en int med namnet CardReturn.

Så den korrekta metoden vill ha är:

public string methodB(int points){
	if (points == 1) {	
		return "Ess.JPG";
	}
	else if(points ==2) {
		return "Two.JPG";
	}
}

Du skickar alltså in en integer (points) som du kör lite if satser på och returnerar sedan en sträng.

Notera även att en metod och klass inte är samma sak. Varför ha en klass med endast en metod? Du kan lika bra ha den metoden i din main klass.

Gnejs skrev redan det.

Hade du gjort det i PHP eller javascript istället, så hade jag kunnat hjälpa dig.
Behöver det vara en desktop-applikation?

          int points 
                BlackCards blackcrap= new BlackCards();
		String returnValue = BlackCards.methodB(points);

CardReturn = BlackCards.methodB(points);

Bättre å hålla sig till riktig programmering än PHP/JS skräp

Alltså du kan inte använda variabler utanför en metod som du gör där (då finns det ingen anledning att ha en return). Du måste säga att CardReturn i metoden ska vara en string.

public String methodB(int points){
    if (points == 1) {    
        String CardReturn ="Ess.JPG";
        return CardReturn;
    } else if(points ==2) {
        String CardReturn ="Two.JPG";
        return CardReturn;
    }
}

har inte orkar läsa alla svar, men du kan iaf inte ha "void" och return "string"

denna rad är fel med tanke på att du vill returna string: public void methodB(String CardReturn){

borde vara: public STRING methodB(String CardReturn){

fast STRING -> String då..

public String methodB(int points){
    if (points == 1) {    
        String CardReturn ="Ess.JPG";
        return CardReturn;
    } else if(points ==2) {
        String CardReturn ="Two.JPG";
        return CardReturn;
    }
}

Det kan han, även om han inte borde det Då det den returnerar kommer vara en referens till samma Sträng som den publika variabeln refererar till.
Däremot till problemet:
Du behöver en return efter if-satserna då det finns utfall som inte passar i någon av de två if satserna. Om du hade kollat på mitt exempel istället så hade det blivit rätt

public String methodB(int points){
    if (points == 1) {    
        String CardReturn ="Ess.JPG";
        return CardReturn;
    } else if(points ==2) {
        String CardReturn ="Two.JPG";
        return CardReturn;
    }
}

Han behöver väl inte skicka med int points om han bara har public int points?
Här är ju ett exempel på det http://docs.oracle.com/javase/tutorial/displayCode.html?code=...

Han ska inte ha en public int points då det är idioti att ha det
Om det var ett C program på 80-talet hade det kanske varit ok med 'globala variabler'

points[i] = null; sätter endast points för index i till null. Observera null ej kan användas för primitiva typer.
Om du har en array som du vill rensa är det nog lättast att ersätta den med en ny array bara.

Är den fortfarande public när den inte ligger i main?
Eller kan man kanske köra private istället?
(Som sagt var ett tag sen sist)

Om man skall hålla sig till "reglerna" kring encapsulation så skall samtliga class medlemmer vara private såvida det inte finns väldigt bra skäl till att inte ha det. T.ex. protected vid Arv osv. Public = no no då man skall använda sig av de eventuella get/set metoderna för att manipulera värdena på ett mer kontrollerat sätt.

edit: Lite förklaring, riktigt "global variabel" blir det ju inte då den inte är static, men att sätta den till public betyder av "vem som helst" som har tillgång till ett objekt av den typen kan ändra värdet direkt. Dvs om ett blackcards objekt hade hittat sig till en annan del av applikationen, eller om en annan utvecklare använt samma klasser så hade han när som helst, var som helst kunnat komma åt pointz via objekt.pointz, och skulle t.ex. kunna säga objekt.pointz = 23487828737; sen vet man aldrig vad för kul som skulle kunna inträffa

Jag skulle nog föredra att kasta ett exception om inget av villkoren stämmer. Det låter troligt att det normala är att något av villkoren ska stämma.

Sedan kan man ju fundera på "if - else if" vs switch vs map-datastruktur.

vs Enum då det verkar handla om spelkort (även om jag inte riktigt hänger med den mindre logiska kopplingen till "pointz").
Går ju inte att misslyckas om man måste välja Enum konstant^^ Card.ACE;
Designvalet har nog ingen större betydelse i TS lilla App iaf