proggen.org

Schrieb mir gerade ein Freund von mir... wenn also ein Hobbyprogrammierer sich fragt, wie man professionell Probleme löst, siehe hier:

ich habe ein Code-Sahnestück gefunden

Code: Alles auswählen

        page.ContentRange.Tables
        public static void EnumerateSteps()
        {
            int lastStepNr = 0;
            foreach (ContentPage page in ScreenPlayTemplate.ContentPages)
            {
                int tableCount = 1;
                foreach (Word.Table table in page.ContentRange.Tables)
                {

                    if (tableCount == 3)
                    {
                        foreach (Word.Row row in table.Rows)
                        {
                            foreach (Word.Cell cell in row.Cells)
                            {
                                if (cell.ColumnIndex == 1 && cell.RowIndex > 1)
                                {
                                    lastStepNr++;
                                    cell.Range.Text = String.Format("{0}.", lastStepNr);
                                }
                            }
                        }
                    }
                    tableCount++;
                }
            }
        }

Jemand 'ne Idee, was hier passiert?


(PS: Erinnert mich an ein Fundstück, dass ich selbst gefunden habe - nicht in meinem Code(!): Noch ein Sahnestück)

Xin hat geschrieben: ...

Code: Alles auswählen

                            foreach (Word.Cell cell in row.Cells)
                            {
                                if (cell.ColumnIndex == 1 && cell.RowIndex > 1)
                                {

Jemand 'ne Idee, was hier passiert?

Höchstwahrscheinlich was reichlich Ineffizientes. Warteschleife?
Warum iteriert man über alle Zeilen und Spalten (ist doch ein GridView, oder?), wenn man sowieso nur die Spalte 1 anfassen will?

Sieht aus wie der dümmste Code der Welt um die erste Spalte einer Tabelle fortlaufend durchzunummerieren (also Zeilenbeschriftungen zu erzeugen).

Wer hat das verbrochen?

fat-lobyte hat geschrieben:Wer hat das verbrochen?

Ich würde - ohne das geringste Vorurteil - spontan auf einen Inder tippen...