4. Danke für die klasse Zusammenarbeit!

Danke für die klasse Zusammenarbeit!

  • _

    Also ich will dir diesmal nicht schon wieder die Lösung vorkauen. Du willst ja (hoffentlich) auch was lernen. Wenn du die horizontale Spiegelung verstanden hast, dann kannst du auch ganz einfach die vertikale Spiegelung realisieren. Es wird dir also von großem Nutzen sein, wenn du den bisherigen Code mal schrittweise durchspielst (der Debugger hilft dir dabei mit Haltepunkten und schrittweiser Ausführung) und die Berechnungen nachvollziehst. Das solltest du unbedingt tun. Einfach mal 2 Minuten auf den Code zu starren, bringt dir vermutlich wenig...

    Aber zumindest ein paar Tipps:

    Stuttgart schrieb:

    Also ich hab die oberen beiden Bedingungen vertauscht und hab beim Speicher [] als dann 1+y gemacht!

    Wie kommst du auf +1? Erklär bitte mal, was du denkst, was du damit erreichst?

    [quote="Stuttgart"]

    for(x=0;x<tagbmp.bmWidth/2;x++) {
  for(y=0;y<tagbmp.bmHeight;y++) {

    So sorgst du dafür, dass nur die Hälfte der Spalten abgearbeitet werden! Du willst aber jede Spalte bis zur Hälfte bearbeiten...

    Stuttgart schrieb:

    Visual Studio findet keinen Fehler aber es kommt wieder zu einem abrupten Ende.

    Das ist eben der Unterschied zwischen einem syntakischen Fehler, den der Compiler sofort bemerkt (ähnlich einem Rechtschreibfehler), und einem semantischen Fehler, der in der Regel viel schlimmer und viel schwerer zu finden ist (auch ohne Rechtschreibfehler kann man Blödsinn schreiben 😉 ).

    Übrigens: poste bitte jedesmal die Fehlermeldung mit, auch wenn ich mir die Ursache diesmal denken kann (du überschreitest wieder Feldgrenzen).

    0
  • S

    Da hast du allerdings recht! Aber was mir jetzt aufgefallen ist. Ich darf nie nur das eine Problem anschauen und es immer übertragen sondern des ganze Programm komplett betrachten. Und dann muss es natürlich heißen:

    for(x=0;x<tagbmp.bmWidth;x++) {
  for(y=0;y<tagbmp.bmHeight/2;y++) {

    So wird über gesamt x die Spalten bis zur Hälfte "angefahren". Und das reicht ja!

    Und das +1 meinte ich weil wir ja jetzt nicht mehr "linke" und "rechte" Pixel vertauschen sondern "oben" und "unten". Und als oberen pixel sage ich der hat die Y-Koordinate 1+y und als x-Wert x und der untere pixel eben nur y und x.

    Liebe Grüße

    0
  • _

    😕

    Der erste Pixel in Zeile 0, Spalte 0 hat die Koordinaten 0,0. Wenn du da nochmal +1 rechnest, gehst du in der Zeile einen nach rechts, d.h. du würdest das Bild um 1px nach rechts verschieben. Ist ja nicht in deinem Sinne...

    Zur Fehlersuche: bitte rechne mal nach, welche Werte (Indizes für Speicher) du berechnest, wenn die Schleifen im letzten Durchlauf sind, also x=maximale Breite-1 und y=maximale Höhe-1 sind. Bedenke: der höchste, gültige Index für Speicher ist Breite*Höhe.

    EDIT: Die Schleifenköpfe stimmen jetzt aber.

    0
  • S

    Servus Matze,

    for(x=0;x<tagbmp.bmWidth;x++) {                         
        }
    }

    Stimmt es so? Bekomm genau die von dir beschriebene Fehlermeldung! Und das mit dem Nachrechnen hab ich eigentlich so verstanden, dass es halt maximal Höhe*Breite sein kann und somit ja des 1+ sinnlos.
    Ich weiß nur nicht wie ich die oberen Pixel auswählen kann. In einer Zeile hab ichs jetzt verstanden aber wie geht das in einer Spale? 1-y? (Siehe C-Code oben)

    Grüße

    0
  • _

    Stuttgart schrieb:

    du musst Nerven haben 😃 Mit jemandem über C reden der es grade mal fertig bekommt Visual Studio zu öffnen, musss echt hart sein 😉

    Ich staune über mich selbst... 😉

    Stuttgart schrieb:

    Speicher[1-y*tagbmp.bmWidth+x]= ...

    Was denkst du, welche Pixel du hier adressierst? 1-[...]?? Ist dir nicht klar, dass du hier einen negativen Index errechnest, sobald y>0 ist? Was denkst du passiert, wenn du an Stelle Speicher[-1279] schreiben willst?

    Stuttgart schrieb:

    Und das mit dem Nachrechnen hab ich eigentlich so verstanden, dass es halt maximal Höhe*Breite sein kann und somit ja des 1+ sinnlos.
    Ich weiß nur nicht wie ich die oberen Pixel auswählen kann. In einer Zeile hab ichs jetzt verstanden aber wie geht das in einer Spale? 1-y?

    Ich verstehe deinen Satz nicht. Das mit dem Nachrechnen habe ich so gemeint, dass du dir ein Blatt und einen Stift nimmst und das Ganze mal nachrechnest! Du sagtest doch, dass du eher der Mathematiker bist. Wenn du eine Wertetabelle erstellst, fütterst du auch eine Funktion mit Werten und rechnest die jeweiligen Ergebnisse aus. Genauso musst du das auch machen. Was passiert bei x=0 und y=0? Und was passiert bei x=1,y=0 oder x=0,y=1? Ich denke nämlich nicht, dass du diese Adressierung bis jetzt verstanden hast. Das beweisen dein letzter Code und deine Frage, wie du das in einer Spalte machen musst. Koordinaten beziehen sich immer auf Zeile und Spalte, und die Formel y*breite+x ist unter allen Umständen gültig und ohne Änderungen anwendbar, um zu Pixel x,y zu gelangen. Alles was du nun machen musst, ist zu entscheiden, welche Pixel du adressieren willst. Und das heißt, du musst nur gedanklich nachvollziehen, welche Pixel du beim Spiegeln tauschen musst.

    0
  • S

    Hi Matze,

    bin nur wieder aus dem Urlaub zurück gekehrt! Bin leider recht müde und schau mir deshalb die "Addressierung" erst Morgen an.
    Das Horizontale habe ich eigentlich verstanden aber wie gesagt: Ich schaus mir morgen mal an und melde mich

    Grüße aus Stuttgart

    0
  • S

    Servus,

    also ich hab mal, so wie du mir geraten hast, ein Blatt genommen und mir den Spiegelvorgang nochmal klargemacht.
    Damit du siehst wo vielleicht mein Fehler ist oder ob es gar richtig ist, mache ich ein Beispiel:

    Nehmen wir an mein Bild ist 10px hoch und 3px breit.

    Und x bleibt ja gleich! Ist ja immer in der selben Spalte

    **
    Die Probe:**

    Ich will, beim oben genannten Bild, den Pixel mit y=2 vertikal spiegeln.
    Die Y-Koordinate des "entsprechenden oberen" Pixel wäre dann (10-2+1)=9!

    Für y=3 ergibt sich 8 usw. das passt doch, oder?

    Belehre mich eines Besseren, ich freu mich drauf 😃

    0
  • _

    Tach!

    Deine Überlegung ist ja eigentlich schon richtig. Nur, dass ich die Bezeichnungen ändern würde, denn wenn dein 'oberer Pixel' 9 ist, ist es ja eigentlich der untere Pixel (0,0 ist die obere, linke Ecke). Außerdem ist natürlich alles null-basiert (dein Pixel 1 ist also Pixel 0).

    Pixely=Pixel(Höhe-y-1)

    und umgedreht:

    Pixel(Höhe-y-1)=Pixely

    So tauschst du (bei Höhe 10) Pixel 0 mit Pixel 9, Pixel 1 mit Pixel 8 usw.

    Vermisch das mit der genannten Adressierung y*Breite+x und zwei Schleifen, und schon hast du deine vertikale Spiegelung!

    0
  • S

    Abend Matze!

    Also hab mich jetzt nochmal versucht und ich habe das Gefühl, dass ich die "Addressierung" verstanden habe. Mir war nicht bewusst, dass es links oben anfängt! Danke!

    Leider bringt Visual Studio eine Fehlermeldung mit Zugriffsverletzung. Schade 😞

    Grüße aus dem jetzt verregnetem Stuttgart

    0
  • _
     

    Speicher[(tagbmp.bmHeight-y+1)*tagbmp.bmWidth+tagbmp.bmWidth+x]

    Das ist ja auch Quatsch. Damit kommst du aus dem gültigen Bereich raus (hatte ich auch nicht so geschrieben). Wenn y=0 ist, dann kommst du bei einer Höhe von 10 (also 0-9, maximale Zeile ist also 9) auf 10-0+1, also 11. Ist ein bisschen zuviel, oder? 😉 Außerdem kommst du, unabhängig von der Zeile 👍 auch in die falsche Spalte. Wenn beispielsweise x auf Maximum steht, addierst du nicht nur einmal x, sondern zweimal (+tagbmp.bmWidth+x).

    Ersetze das mal an beiden Stellen durch das hier:

    Speicher[(tagbmp.bmHeight-y-1)*tagbmp.bmWidth+x]

    Macht mehr Sinn, denke ich. Klappt?

    0
  • S

    Servus Matze!

    Danke. Ich war mir eigentlich so sicher aber du hattest natürlich recht! Macht ja anders gar keinen Sinn! Aber das hab ich jetzt echt verstanden. Hab jetzt mal noch ein bisschen was gemacht:

    Hinweis: Hab die Variable Spig oben deklariert.
    Weiß nicht ob das elegant ist oder nicht! Was sagst du dazu?

    Außerdem ist was ganz komisches passiert: Wenn ich ganz am Ende der Main ge hab dann schließt sich die Konsole nach der Eingabe gleich! Wenn ich ch() hab dann bleibt es wie gewünscht offen! Komisch oder?

    Muss ich auch was machen, dass wenn man was anderes als 1/2 eingibt, dass es dann eine Fehlermeldung oder so gibt? Da komm ich noch nicht dahinter! Hast du da einen Tipp?

    Grüße

    0
  • _

    Stuttgart schrieb:

    Danke. Ich war mir eigentlich so sicher aber du hattest natürlich recht! Macht ja anders gar keinen Sinn! Aber das hab ich jetzt echt verstanden. Hab jetzt mal noch ein bisschen was gemacht:

    Das heißt, das Spiegeln funktioniert jetzt?

    Stuttgart schrieb:

    Hinweis: Hab die Variable Spiegelung oben deklariert.
    Weiß nicht ob das elegant ist oder nicht! Was sagst du dazu?

    Globale Variablen sollte man eher vermeiden. Deklariere die Variable lieber am Anfang der main-Funktion.

    Stuttgart schrieb:

    Muss ich auch was machen, dass wenn man was anderes als 1/2 eingibt, dass es dann eine Fehlermeldung oder so gibt? Da komm ich noch nicht dahinter! Hast du da einen Tipp?

    Da musst du die Eingabe in eine Schleife packen, die solange läuft, bis eine gültige Eingabe gemacht wurde. Eine do-while-Schleife eignet sich da am besten. Zusätzlich solltest du dem Benutzer noch eine dritte Eingabemöglichkeit, nämlich zum Abbruch, bieten.

    0
  • S

    Abend Matze

    Hab gerade noch den Teil mit dem Abbruch rein gebracht!

    Passt also meiner Meinung nach alles. Funktioniert auch ganz gut!

    Wenn du damit einverstanden bist 😉

    Hab danach noch ein paar Fragen wegen "Vereinfachung" an dich

    Grüße

    0
  • _

    Schön, dass es klappt. Du solltest aber dringend mal an deiner Einrückung arbeiten. Das sollte alles schön strukturiert sein, so dass man Blockanfang und -ende auch erkennen kann.

    http://de.wikipedia.org/wiki/Einrückungsstil

    0
  • D

    sorry für off-topic aber wieso kann ich nicht auf die 5te seite von JAMLEGEND BOT (http://www.c-plusplus.net/forum/viewtopic-var-t-is-241595-and-start-is-40.html)

    0
  • _

    drachlen schrieb:

    sorry für off-topic aber wieso kann ich nicht auf die 5te seite von JAMLEGEND BOT (http://www.c-plusplus.net/forum/viewtopic-var-t-is-241595-and-start-is-40.html)

    😕
    Das ist allerdings off-topic. offer gehts ja gar nicht. Warum platzierst du deine Frage nicht hier?

    http://c-plusplus.net/forum/viewforum-var-f-is-39.html

    0
  • S

    Hi Matze,

    also ich danke dir nochmal für deine Tips! Hab mir über die Woche nochmal Grundlagen von C angeschaut und bin im Unterricht heute voll gut mit gekommen! Ich denke das habe ich teilweise auch deinem Durchhaltevermögen zu verdanken.

    Das Programm ist funktionstechnisch fertig aber ich verstehe folgendes nicht:

    ES KOMMT EIN UPDATE HIER REIN!

    Kann man diesen Teil auch anders schreiben? Ich versteh zum Beispiel nicht warum man die Farben der Pixel aufrufen muss und in Bites speichern muss.
    Wenn nicht könnte ich dann zu einigen Zeilen etwas fragen?

    Grüße aus Stuttgart

    0
  • _

    Da ich diesen Programmteil irgendwo im Internet gefunden habe, weil ich eine schnelle Möglichkeit gesucht habe, eine Bitmap-Datei zu schreiben, kann ich dazu nix großartig sagen, ohne mir das genauer anzusehen (und dazu fehlt mir im Moment die Zeit, sorry). Ich nutze für sowas eine andere Bibliothek, daher kenne ich mich mit den WinAPI-Funktionen für solchen Kram nicht aus.

    Ich finde aber auch, dass das für dein Vorankommen im Moment wirklich nicht nötig ist. Du solltest dich besser mit C-Grundlagen beschäftigen (das Erlernen der Sprache ist doch dein Ziel, oder? Welche Ausbildung machst du eigentlich?), z.B. indem du den Programmablauf in der main (Interaktion mit dem Benutzer) optimierst.

    Was passiert, wenn der Benutzer eine falsche Eingabe macht (hast du ja schon selbst angesprochen)?

    Wie halte ich die Konsole am Programmende korrekt geöffnet (wenn du kurz vor Schluss den Eingabepuffer leerst, dann klappt auch ein getchar wie gewünscht)?

    Wie kann ich den Code besser lesbar gestalten (Tipps: korrekte Einrückung ist ganz wichtig; if-else-if Konstrukt durch switch-Anweisung ersetzen).

    Welche zusätzlichen Optionen könnte ich dem Benutzer bieten (z.B. beide Spiegelungen anwenden)? Und wie setze ich das um (Struktur müsste ein wenig angepasst werden, redundanten Code hierbei unbedingt vermeiden!)?

    ...und was dir selbst noch so einfällt...

    Trotzdem kannst du natürlich auch zu dem Code gerne einzelne Fragen stellen. Nur versuch jetzt nicht, den bis ins kleinste Detail zu verstehen. Es gibt Wichtigeres zu lernen.

    0
  • S

    Abend Matze,

    wollt mich nur kurz melden und bescheid geben, dass es mich noch gibt. Hatte zur Zeit viel zu tun und werd mich morgen nochmal mit dem Programm beschäftigen!

    Grüße

    0
  • S

    Hi Matze,

    hab jetzt das Programm nochmal durchgearbeitet und habs auch ganz gut verstanden!

    Ich bedanke mich abschließend noch mal ganz herzlich bei dir! Vielen Dank für deine Nervenstärke 😉

    Grüße und wir "schreiben" uns wahrscheinlich im Forum!

    Grüße

    0
Anmelden zum Antworten