[Fehler-Code]Zahlentripel

[Daxter]

Hi,

sitze gerade in der Schule und verzweifle gerade an der Suche meines Fehlers in meinem Programm. Jeder von euch kennt den Satz des Pythagoras und dies sollen wir programmieren. Zum Beispiel 3² + 4² = 5². Wenn ich als oberen Grenzwert zum Beispiel 25 eingebe erscheint bei mir in der Ausgabe:

Programm zur Berechnung des pythagoräischen Zahlentriple.
Geben Sie eine obere Grenze ein: 25
28 27 26

Mein Code:

Spoiler:

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;


public class Pytho 
{


	public static void main(String[] args) throws IOException 
	
	{
		System.out.println("Programm zur Berechnung des pythagoräischen Zahlentriple.");
		String str;
		int a, b, c, n;
		BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
		System.out.print("Geben Sie eine obere Grenze ein: ");
		str = in.readLine();
		n = Integer.parseInt(str);
		for (c=2; c<=n;c++);
		{
			for (b=1;b<=c; b++);
			{
				for(a=1; a<=b; a++);
				{
					if(a*a+b*b==c*c);
					{
						System.out.println(a+"\t"+b+"\t"+c);
					}
				}
			}
		}
	}
}

MfG

[SteffenHill]

Ich verstehe nicht ganau deine Frage, aber nach einer for schleife bitte keine ;

Also mach die mal weg, dann sollte es deiner Lösung entsprechen. Wenn dies nicht so ist, bitte die Frage genauer ausschreiben...

[Ta1lor]

Hab das gerade gefunden und etwas umgeschrieben. Ist zwar in C++, aber wird dir bestimmt auch weiter helfen.

include <conio.h>
using namespace std;
bool teilerfremd(unsigned int u, unsigned int v)
{
  if (v==1) return (true);
  for (int i=2; i<=v; i++)
    if((v%i==0) && (u%i==0))
      return (false);
  return (true);
}

void berechnen()
{
   unsigned int  obere_grenze, u, v, a, b, c;
      cout << "obere  Grenze: ";
      cin >> obere_grenze;
      for (u=2; u<=obere_grenze; u++)
      for (v=1; v<u; v++)
        if(teilerfremd(u,v) && ((u-v)%2))
        {
          a = u*u - v*v;
          b = 2*u*v;
          c = u*u + v*v;
          cout   << u << " " << " A: "<< a << " B: " << b << " C: " << c << endl;
        }
      _getch();

}

int main()
{
   berechnen();
   return 0;
} 

[SteffenHill]

Also ta1lor, das ist nicht nötig! Dein Programm ist in einer anderen Sprache und sein Code funktionert auch. Habe hier noch ein anderes Programm gefunden wenn du mal rien schauen willst wie es gemacht ist:

Spoiler:

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

namespace CD_Lehre.Pythagoras
{
    /// <summary>
    /// <para>Dieses Beispiel stellt alle pythagoräischen Dreiecke bis zu einer
    /// einzugebenden Kathetenlänge fest.</para>
    /// <para>Ein pythagoräisches Dreieck ist ein Dreieck, für welches 
    /// mit a, b, c als ganze positive Zahlen gilt:</para>
    /// c<sup>2</sup> = a<sup>2</sup> + b<sup>2</sup>
    /// <para>Musterlösung zur Verschachtelung von Schleifen (do-while-Schleife
    /// innerhalb von 2 geschachtelten for-Schleifen).</para>
    /// </summary>
    class Program
    {
        /// <summary>
        /// Der Haupteinstiegspunkt für die Anwendung.
        /// </summary>
        static void Main()
        {
            int aMax;
            int a;
            int b;
            int c;
            int a2;
            int b2;
            int c2;

            do
            {
                Console.Write("Eine Kathetenlänge eingeben: ");
            } while (!Int32.TryParse(Console.ReadLine(), out aMax) || (aMax <= 0));

            for (a = 2; a < aMax; a++)
            {
                a2 = a*a;
                for (b = 1; b <= (a - 1); b++)
                {
                    b2 = b*b;
                    c2 = a2 + b2;
                    c = a;
                    do
                    {
                        if (c*c == c2)
                        {
                            Console.WriteLine("{0} {1} {2}", a, b, c);
                            break;
                        }
                        else
                        {
                            c++;
                        }
                    } while (c*c <= c2);
                }
            }
            Console.ReadKey();  
        }
    }
} 

[Ta1lor]

Also ta1lor, das ist nicht nötig! Dein Programm ist in einer anderen Sprache und sein Code funktionert auch. Habe hier noch ein anderes Programm gefunden wenn du mal rien schauen willst wie es gemacht ist:

Spoiler:

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

namespace CD_Lehre.Pythagoras
{
    /// <summary>
    /// <para>Dieses Beispiel stellt alle pythagoräischen Dreiecke bis zu einer
    /// einzugebenden Kathetenlänge fest.</para>
    /// <para>Ein pythagoräisches Dreieck ist ein Dreieck, für welches 
    /// mit a, b, c als ganze positive Zahlen gilt:</para>
    /// c<sup>2</sup> = a<sup>2</sup> + b<sup>2</sup>
    /// <para>Musterlösung zur Verschachtelung von Schleifen (do-while-Schleife
    /// innerhalb von 2 geschachtelten for-Schleifen).</para>
    /// </summary>
    class Program
    {
        /// <summary>
        /// Der Haupteinstiegspunkt für die Anwendung.
        /// </summary>
        static void Main()
        {
            int aMax;
            int a;
            int b;
            int c;
            int a2;
            int b2;
            int c2;

            do
            {
                Console.Write("Eine Kathetenlänge eingeben: ");
            } while (!Int32.TryParse(Console.ReadLine(), out aMax) || (aMax <= 0));

            for (a = 2; a < aMax; a++)
            {
                a2 = a*a;
                for (b = 1; b <= (a - 1); b++)
                {
                    b2 = b*b;
                    c2 = a2 + b2;
                    c = a;
                    do
                    {
                        if (c*c == c2)
                        {
                            Console.WriteLine("{0} {1} {2}", a, b, c);
                            break;
                        }
                        else
                        {
                            c++;
                        }
                    } while (c*c <= c2);
                }
            }
            Console.ReadKey();  
        }
    }
} 

Deins ist übrigens C#.

[SteffenHill]

Deins ist übrigens C#.

Das was ich gepostet habe ist C#. C# ist identisch mit Java.

C ist die für dieses beispiel blöd, da er sicher keine Erfahrung mit C hat und C Programmcode sehr schwer zu lesen ist (für mich aufjeden Fall). Spielt ja auch keine Rolle, C# ist einfach die leichtere und für ihn bessere Sprache es schnell und korrekt zu überprüfen.