Hallo allemaal,

Ik moet voor school een klein programmaatje schrijven die alle oneven getallen van de invoer bij elkaar optelt... Alleen kom ik er maar niet uit.

Ik heb zelf de volgende code:

import java.util.Scanner;


public class opdrachtE {


	public static void main(String[] args) {
		
		Scanner in = new Scanner(System.in);


		System.out.print("Getal:");
	
		int nummers = in.nextInt();
		int som = 0;
		int antwoord = 0;
		
		String nummersLengte = String.valueOf(nummers);
		for(int x = 0; x < nummersLengte.length(); x++) {
				
			int getal = nummersLengte.charAt(x);
			
			if(getal%2 == 1) {
				som = som + getal;
			}
			antwoord = som;
		}
		
		System.out.println("antwoord is:" + antwoord);
		
			
	}


}

Echter wanneer je nu bijvoorbeeld 1234 invult, komt er als output: antwoord is: 100. Terwijl het antwoord zou moeten zijn 1+3= 4. Iemand enig idee wat ik fout doe?

Hallo Stefan,

Hier zit inderdaad de fout in. Wanneer ik namelijk 1234 invul, krijg ik als waardes: 49,50,51,52 terug. Echter is bij mij nu helemaal de klomp gebroken, want nu begrijp ik niet waar die waardes vandaan komen?

Over de andere opmerkingen:

- Antwoord heeft inderdaad geen toegevoegde waarde, dit is eigenlijk een overblijfsel van andere pogingen die ik heb gedaan. Deze heb ik nu ook verwijderd
- Op school hebben we geleerd dat je begint met een kleine letter en elk ander woord met een hoofdletter. Dus bijv.: classJavaVoorSchool
- Ik gebruik momenteel Eclipse (moet van school). Daar heb ik echter geen fout melding gekregen.

De Scanner gebruik ik om de gebruiker een waarde in te laten vullen. Hier staat het wat duidelijker beschreven:
http://www.java-made-easy.com/java-scanner.html

som += getal;

Hier heb ik nog niet eerder van gehoord, handig! Dank je

Super jongens, de laatste tip van je stefan... Dat ik daar zelf niet op kwam ! Maar in ieder geval hartstikke bedankt jongens!

Hier mijn volledig werkende code, mocht iemand er nog belang bij hebben:

import java.util.Scanner;


public class opdrachtE {


	public static void main(String[] args) {
		
		Scanner in = new Scanner(System.in);


		System.out.print("Getal:");
	
		int nummers = in.nextInt();
		int som = 0;
		
		String nummersLengte = String.valueOf(nummers);
		
		for(int x = 0; x < nummersLengte.length(); x++) {
				
			int getal = Character.getNumericValue(nummersLengte.charAt(x));
			System.out.println(getal);
			
			if(getal%2 == 1) {
				som = som + getal;
			}


		}
		
		System.out.println("antwoord is:" + som);
		
			
	}


}

PS: Als ik de class verander in CamelCase, dus naar OpdrachtE, geeft hij de volgende error:
The public type OpdrachtE must be defined in its own file.

EDIT: Maar dat komt natuurlijk omdat ik hem opdrachtE heb genoemd, ik zal in het vervolg rekening houden met CamelCase